View source for Module talk:Infobox gene/Archive 1



== Targeted by drug ==

Hi {{ping|Julialturner}}  While I am highly sympathetic to allowing links to "targeted by drug" (this is what I do in real life), I think the display of this section of the infbox should be suppressed unless there are relevant entries in Wikidata.  It appears that this is what you had in mind since you added the comment "''check if any drugs have references if not then don't render the headers''" to the template code, but for some reason, the drug header is displayed whether or not there any drugs targeting this protein.  I would appreciate if you would check this when you get a chance.  Cheers. [[User:Boghog|Boghog]] ([[User talk:Boghog|talk]]) 19:39, 15 July 2016 (UTC)

{{ping|Boghog}} Thank you.  I just noticed this wasn't being suppressed yesterday on some pages and I am working on a solution.  Best, [[User:Julialturner|Julialturner]] ([[User talk:Julialturner|talk]]) 21:20, 15 July 2016 (UTC)

: Julia, thanks for [https://en.wikipedia.org/w/index.php?title=Module:Infobox_gene&diff=prev&oldid=729978900 fixing] the suppression of the section.  Much appreciated. [[User:Boghog|Boghog]] ([[User talk:Boghog|talk]]) 14:33, 2 August 2016 (UTC)

== Wikidata citations ==

Hi {{ping|Julialturner}}.  Another request.   I noticed that infobox gene is now staring to display citation data (see for example [[Cyclic nucleotide-gated channel alpha 1|CNGA1]]).  Unfortunately bare URLs are displayed which is far from ideal.  They are not very readable and lead to link rot. They are also ugly.  Furthermore I can't locate the citation in Wikidata to see if it is even possible to add a formatted citation. Do you know if formatted citations are supported in Wikidata?  If not, I humbly suggest that it may be premature to display this type of data.  Cheers. [[User:Boghog|Boghog]] ([[User talk:Boghog|talk]]) 14:47, 2 August 2016 (UTC)
:Some kind of formatted citations you can see [[:d:Q2866016#P19|here]]. If that's not what you're thinking about, then you can simply ignore me :) --'''[[User:Edgars2007|<span style="color:#FF6600;">Edgars2007</span>]]''' <small>([[User talk:Edgars2007|talk]]/[[Special:Contributions/Edgars2007|contribs]])</small> 16:18, 2 August 2016 (UTC)
:: {{ping|Julialturner|Boghog|Daniel_Mietchen}} Wikidata won't hold formatted references as a string if that is what you are thinking.  There is ongoing work to hold all the components of a reference (authors, journal, title, etc.) as structured data there such that a script could automatically compose a formatted string for references.  So.. what to do now?  The URLs you are referring to appear as references within the corresponding wikidata item.  For your example, you can see them at https://www.wikidata.org/wiki/Q17907825 under the corresponding property - e.g. genetic association.  In many cases, it seems the link to the database where the information was collected from is the appropriate reference.  When the appropriate reference is indeed an online resource - which admittedly is subject to linkrot - what is an appropriate referencing pattern?  --[[User:I9606|Benjamin Good]] ([[User talk:I9606|talk]]) 21:59, 2 August 2016 (UTC)
::: {{ping|I9606}} Thanks for the response.  Actually some of the data database links look OK (e.g., human and mouse PubMed references).  At a minimum, the database references should display like these PubMed references where the displayed link is replaced by readable text. The OMIM reference ideally should look like what {{tl|OMIM}} produces. Also In the [[CNGA1]] example, targeted by drug, [[Dequalinium]] cites https://www.ncbi.nlm.nih.gov/pubmed/12508052.  This reference as well as the references for Genetically Related Diseases ([[Retinitis pigmentosa]]) are  not contained in https://www.wikidata.org/wiki/Q17907825. [[User:Boghog|Boghog]] ([[User talk:Boghog|talk]]) 06:11, 3 August 2016 (UTC)
:::: {{ping|Boghog}} Thanks, clearly we need to work on the presentation of the references.  I really hope we can get some help from the people focused on citation like {{ping|Daniel_Mietchen}}.  To clarify where things are coming from, the drug interaction and its reference are located on the wikidata item associated with the protein product for that gene https://www.wikidata.org/wiki/Q21105636 under the 'physically interacts with' property.  The Lua script that builds the template pulls such interactions by traversing from the gene item to the protein item in the same way that it gets the Gene Ontology annotations.  The references for Genetically Related Diseases ([[Retinitis pigmentosa]]) are indeed coming directly from https://www.wikidata.org/wiki/Q17907825 .  Look for the 'genetic association' property on that item.  --[[User:I9606|Benjamin Good]] ([[User talk:I9606|talk]]) 18:32, 3 August 2016 (UTC)
::::: {{ping|I9606}} OK, thanks. I now see the reference links for Dequalinium and Retinitis pigments (in Wikidata pages, the Safari search tool doesn't seem to search beyond what is visible in the currently displayed browser window).  I hope someone is able to come up with a better long term solution to the display of the citations.  A partial solution for the PubMed citation would be to parse the bare url https://www.ncbi.nlm.nih.gov/pubmed/12508052 for the pmid and replace the displayed url with <nowiki>"PMID 12508052"</nowiki> that renders as "{{PMID|12508052}}". [[User:Boghog|Boghog]] ([[User talk:Boghog|talk]]) 19:49, 3 August 2016 (UTC)
:::::: Hi, {{ping|Boghog}} I have been adjusting the citation format for genetically related diseases.  I created an example here that I think would be a better solution (https://en.wikipedia.org/wiki/User:Julialturner/RELN).  What are your thoughts? [[User:Julialturner|Julialturner]] ([[User talk:Julialturner|talk]]) 04:42, 27 September 2016 (UTC)

* Would someone please point me to the RfC that allowed integration of Wikidata into this template for distribution throughout Wikipedia's gene articles?  Thanks. [[User:Jytdog|Jytdog]] ([[User talk:Jytdog|talk]]) 21:45, 27 February 2017 (UTC)

== renderCaption  ==

{{ping|Julialturner}} The not yet implemented renderCaption function will indeed have a problem trying to get information dynamically from comments in Commons. Wikidata provides a property {{Q|P2096}} that is monolingual text designed to carry the image caption and it would be better to add that property manually (or perhaps by a bot fetching comments from Commons?) to each gene entry, as it would then make the image legend available programmatically to all. There's an example of using the property to fetch the image caption in {{tl|Infobox telescope}}, which relies on the getImageLegend call in [[Module:Wikidata]] (currently lines 890–963). It implements arbitrary access, preferred ranks (to cope with multiple images) and uses the local wiki language unless an language iso-code is given in the call:
* <code><nowiki>{{#invoke:Wikidata |getImageLegend |FETCH_WIKIDATA |id=Q1513315}}</nowiki></code> → {{#invoke:Wikidata |getImageLegend |FETCH_WIKIDATA |id=Q1513315}}
* <code><nowiki>{{#invoke:Wikidata |getImageLegend |FETCH_WIKIDATA |lang=lt |id=Q1513315}}</nowiki></code> → {{#invoke:Wikidata |getImageLegend |FETCH_WIKIDATA |lang=lt |id=Q1513315}}
It should be easy for you to adapt the Wikidata module code to fill in renderCaption so that it gets a {{Q|P2096}}, leaving editors with the job of updating the relevant Wikidata entries in their own time. Hope that helps. --[[User:RexxS|RexxS]] ([[User talk:RexxS|talk]]) 23:10, 28 February 2017 (UTC)

:{{ping|RexxS}} I appreciate your suggestion very much.  I will definitely try to adapt the wikidata module code here and see if I can get the captions into the infobox.  [[User:Julialturner|Julialturner]] ([[User talk:Julialturner|talk]]) 05:20, 1 March 2017 (UTC)

== Invalid HTML ==
{{ping|Julialturner}} It seems this infobox, outputs same incorrect HTML. The parser mostly cleans this up, but it fails sometimes and then the infobox breaks in VisualEditor for instance as it does for [[BCL2-related_protein_A1]]. If you go to [https://en.wikipedia.org/wiki/Special:ExpandTemplates?wpContextTitle=BCL2-related_protein_A1&wpInput={{Infobox%20gene}} this page], you can see that the td, and tr elements are sometimes not closed, or closed too often, causing the parser having to go into a bit of a guessing game. —[[User:TheDJ|Th<span style="color: green">e</span>DJ]] ([[User talk:TheDJ|talk]] • [[Special:Contributions/TheDJ|contribs]]) 14:21, 16 December 2016 (UTC)
:{{ping|TheDJ}} Thank you for notifying me of this issue I will look into correcting the code. [[User:Julialturner|Julialturner]]
:Hi, {{ping|TheDJ}} I did some code cleanup and now all the tag elements should be closed. [[User:Julialturner|Julialturner]] ([[User talk:Julialturner|talk]]) 06:55, 6 January 2017 (UTC)
:I ran into this via some testing as well. The problem is on line 1603 in the module. It emits a table directly inside a tr tag. You are missing a required td tag to wrap the table - only td, th, and captiont can include content in a table. That is at least one thing that I ran into that needs fixing. [[User:SSastry (WMF)|SSastry (WMF)]] ([[User talk:SSastry (WMF)|talk]]) 18:17, 21 March 2017 (UTC)
::{{ping|Julialturner}} In case you didn't see my earlier message. Pardon me if you have seen that and just haven't gotten around to looking into it yet. Thanks. [[User:SSastry (WMF)|SSastry (WMF)]] ([[User talk:SSastry (WMF)|talk]]) 16:19, 13 April 2017 (UTC)
:::{{ping|SSastry (WMF)}} Thank you for reminding me.  I haven't had a chance to fix this yet, but I will try to fix it in the next week or so.[[Special:Contributions/76.167.64.98|76.167.64.98]] ([[User talk:76.167.64.98|talk]]) 20:09, 13 April 2017 (UTC)

== Biomedical content  - disease associations and interactions ==

Please see [[Wikipedia_talk:WikiProject_Medicine#More_Wikidata_funk_-_infobox_gene]] [[User:Jytdog|Jytdog]] ([[User talk:Jytdog|talk]]) 01:28, 5 April 2017 (UTC)
:Not seeing the funk discussion there now, assuming its been resolved.  --[[User:I9606|Benjamin Good]] ([[User talk:I9606|talk]]) 16:54, 26 April 2017 (UTC)
::Nope it was not.   A means of addressing the problem is being discussed below. [[User:Jytdog|Jytdog]] ([[User talk:Jytdog|talk]]) 17:45, 26 April 2017 (UTC)

== Seriously, how do we suppress individual fields ==

So at [[KCNB1]] the infobox had several pieces of content that was [[WP:Biomedical information]] that was not sourced per MEDRS, and I couldn't remove it, so I went into Wikidata and removed it there.  Some of it has now been restored in Wikidata.  I have '''no desire''' to get into arguments at Wikidata about content that is appearing in Wikipedia. 

So how do we suppress individual fields at a local article?  One of the bad fields was "genetic association".  The other bad field is more complex as I mentioned above... 

I don't want to go nuclear and call for this infobox to be nuked but if we cannot selectively control what comes in, that will be the only option.  [[User:Jytdog|Jytdog]] ([[User talk:Jytdog|talk]]) 06:29, 16 April 2017 (UTC)
: Adding the ability to suppress individual fields based on passing in a parameter like 'nodisease' etc. seems doable.  A more general solution would be to consistently identify which kinds of sources were acceptable for the MEDRS folks, tag them as such within Wikidata, and then add code that would act accordingly without the requirement to touch individual infoboxes.  We will have a look at both options.  By the way, its grossly inappropriate to threaten 'nuclear options' in your discussions here.  Please keep your tone under control and we will work together to continue to improve Wikipedia together.  --[[User:I9606|Benjamin Good]] ([[User talk:I9606|talk]]) 16:54, 26 April 2017 (UTC)
::Thanks for replying!  It would not be appropriate to try to enforce en-WP standards in Wikidata, and if someone did, there is no policy basis there to object if someone should revert.  I also have no desire to try; I have no desire to edit Wikidata on any kind of regular basis.   If folks want to bring in Wikidata for some things in en-WP via infoboxes that is fine of course, but there must be  a way to exclude unreliable data from appearing in en-WP, from within en-WP, that is reasonably easy to implement at the template level or on a per article basis.   There is nothing inappropriate (much less "grossly") about proposing to delete a template, which is what I will indeed do, if there is no way to control things at field level.   I can wait a while but I hope that field-level control will be introduced soon.  Thanks again for replying!   [[User:Jytdog|Jytdog]] ([[User talk:Jytdog|talk]]) 17:42, 26 April 2017 (UTC)
:::I looked into it and it is indeed fairly straightforward to adjust this template to take in a parameter like |showdisease=false and hide a section.  But looking at what you are attempting to conceal, I'm not really convinced its a good idea.  Looking at [[ATG16L1]] for example, there is a line in the infobox saying that there is a genetic association between the gene and Crohn's disease.  That relationship is supported by a link to the  [http://chibi.ubc.ca/Gemma/phenotypes.html?phenotypeUrlId=DOID_8778&ncbiId=55054 database entry where this information was gathered] that in turn links to several journal articles and entries in other databases that support the claim.  I think this is useful information to people interested in this gene.  I don't see how it is medically dangerous.  I'm concerned that adding a 'hide' function to this module will result in good information like that being hidden from view.  I'd like to hear more from the biology community e.g. {{ping|Boghog}} before making a change.  --[[User:I9606|Benjamin Good]] ([[User talk:I9606|talk]]) 05:22, 2 May 2017 (UTC)
:::: {{reply to |I9606}} Unfortunately in the example you give, the association between [[ATG16L1]] and [[Crohn's disease]] is supported in the Gemma database by three primary studies and one secondary (a meta-analysis). It is not acceptable per [[WP:MEDRS]] to have a situation where biomedical claims are made though an indiscriminate process that does not guarantee that secondary sources are used. You will note that the article [[ATG16L1]] does not present an association as fact, although the [[Crohn's disease]] article discusses associations in greater detail. A bald assertion that gene X is genetically related to disease Y, without any assurance that the conclusion is based on anything more than a primary study, breaches MEDRS and is completely unsuitable for inclusion in an infobox where information must be clear-cut and not subject to caveats. --[[User:RexxS|RexxS]] ([[User talk:RexxS|talk]]) 12:12, 2 May 2017 (UTC)
::::::[[User:RexxS]] do you think we should just exclude the disease field from this template?  Checking its output article by article is unreasonable, probably. [[User:Jytdog|Jytdog]] ([[User talk:Jytdog|talk]]) 13:22, 2 May 2017 (UTC)
:::::::The Gemma database does have a "Quality code" that might help us restrict the associations to only those supported by high quality sources. Unfortunately they don't seem to discriminate between primary studies and meta-analyses which all received one star. The OMIM tertiary source received three stars and I am not sure how critical OMIM is. [[User:Boghog|Boghog]] ([[User talk:Boghog|talk]]) 13:48, 2 May 2017 (UTC)
:::::::: {{reply to |Jytdog}} I'd prefer to develop methods of marking the Wikidata with a field indicating quality of evidence, so we could filter the data returned, but we're a long way from that at present. I'd agree that in the meantime, we ought to seriously consider leaving out of infoboxes any claims that don't meet our own policies of V, RS and MEDRS. It's just making a rod for own backs when we're trying to import reliable data. {{reply to |Boghog}} You only have to look at the 7 sources that [https://www.ncbi.nlm.nih.gov/pubmed/18849966,18438405,17200669,18852889,24553140,17435756,19337756 OMIM cites]: 1 mouse study, 2 comparative studies, 3 research studies and 1 meta-analysis. The meta-analysis concluded "the ATG16L1 T300A polymorphism was associated with CD risk in Caucasians ... no significant association was found in Asians." How are you going to fit that can of worms into an infobox? --[[User:RexxS|RexxS]] ([[User talk:RexxS|talk]]) 16:22, 2 May 2017 (UTC)
::::::::: I wasn't suggesting that we cite OMIM, just suppress associations with low quality as a potential way of increasingly the reliability. But it looks like Gemma database quality ratings are too crude for that purpose. [[User:Boghog|Boghog]] ([[User talk:Boghog|talk]]) 16:35, 2 May 2017 (UTC)
:::::::::: [[User:RexxS]] For what its worth, its far from an indiscriminate process and that information is vastly more likely to be of use to people looking for information about that gene than it is to cause some sort of medical harm - which as I understand it is the main motivation behind the MEDRS approach.  But i'm not going to continue that particular argument here.  I concur with your sentiment about developing "methods of marking the Wikidata with a field indicating quality of evidence" and I don't see why its so far off.  In fact we already have code in place that prevents unreferenced statements from appearing in the infobox.  The claim in question here has references saying the claim is stated in [https://www.wikidata.org/wiki/Q22330995 Phenocarta] and in a [https://www.wikidata.org/wiki/Q22251069 journal article], both of which have items in Wikidata.  All we would need to do would be to add a property to those items indicating that they were 'MEDRA approved references' or something like that and then we could build a filter into the template that would automatically hide any claims that did not meet the criteria.  Another approach that could be implemented right now without creating any new properties or tagging anything would be to use the [https://www.wikidata.org/wiki/Property:P459 determination method] qualifier property on the claim.  In this case, the claim is made based on evidence from a [https://www.wikidata.org/wiki/Q1098876 genome wide association study] or 'GWAS'.  If we can decide on what forms of evidence are suitable for inclusion of a given kind of claim - e.g. '[https://www.wikidata.org/wiki/Property:P2293 genetic association]', we could use that as a filter immediately.  --[[User:I9606|Benjamin Good]] ([[User talk:I9606|talk]]) 17:03, 2 May 2017 (UTC)
::::::::::: The Gemma database ratings are precisely what I was describing as indiscriminate - which for our purposes, they are if a 3 star rating means "1 mouse study, 2 comparative studies, 3 research studies and 1 meta-analysis with conflicting conclusions". Without taking the time to evaluate each entry in the Gemma database against MEDRS, how do you propose we eliminate the possibility that a Wikidata claim may be insufficient to support a biomedical claim ''by our standards''? If people find the information in {{Q|Q22330995}} interesting, then they can read it there. We have no need to include information indirectly via Wikidata that falls short of what we require from any source used directly in an article. I've invested a lot of time and effort over the last few years in creating and refining tools to facilitate the import of information from Wikidata, and the last thing I want to see is it being thrown away because of knee-jerk reactions by the community to accusations that "Wikidata is not a reliable source" and "Using Wikidata breaches our Verifiability policy". To get an idea of what we're going to be up against, take a look at [[Wikipedia:Wikidata/2017 State of affairs #Perceived disadvantages of using Wikidata on enwiki]] – or at [https://en.wikipedia.org/w/index.php?title=Wikipedia:Wikidata/2017_State_of_affairs&offset=201701122050&limit=100&action=history the edit war] to force inclusion of the phrase "{{tq|The lack of reliable sourcing means that imported Wikidata text violates [[WP:V]] and [[WP:BLP]]}}". Are you sure it's a good idea to add "violates MEDRS" as ammunition for the nay-sayers? --[[User:RexxS|RexxS]] ([[User talk:RexxS|talk]]) 18:02, 2 May 2017 (UTC)
:::::::::::: [[User:RexxS]] why is it better, in your opinion, to put out a blanket deletion of content than to formalize and clarify the reasons for not showing it?  How else do you imagine bridging the gap between structured content and Wikipedians?  Wikidata has, and will continue to have, a variety of different sources for the information it contains.  We need to develop mechanisms for taking advantage of its infrastructure to provide the community of data consumers (Wikipedia here) with ways to use it as they see fit.  I would much rather have the MEDRS people make a definitive statement like "we don't think GWAS studies provide enough evidence to justify showing the resultant content in Wikipedia" and encode that logic in a combination of wikidata statements and template code then I would like to see people simply deleting infoboxes or data in wikidata.  Do you have an alternative plan for the future here? --[[User:I9606|Benjamin Good]] ([[User talk:I9606|talk]]) 19:20, 2 May 2017 (UTC)   
::::::::::::: It is better, in my opinion, not to display a field in an infobox than to display information that falls short of what is required by our guidelines on sourcing. I have difficulty understanding your reasons for taking the opposite view; perhaps you could explain them to me? If you would kindly elucidate what you mean by "the gap between structured content and Wikipedians", I might better be able to see why you are taking your position.
::::::::::::: [[WP:MEDRS]] is not a person, but a [[WP:PAG|Wikipedia guideline]] that enjoys project-wide community support. We are all "MEDRS people" just as we are all "NPOV people", etc. so there's no reason why you can't make the definitive statement yourself: it would carry the same weight (i.e. one person's opinion).
::::::::::::: If you want to "encode that logic in a combination of wikidata statements and template code", then I'm all in favour: please go ahead and do it. Let me know as soon as you've finished, and you'll have my support for importing the filtered data into {{tl|Infobox gene}}. In the meantime, how do you suggest we suppress individual fields in this infobox, as the topic of this thread asks? If we can't ensure that the infobox doesn't include dud information from Wikidata, nor remove the offending claim from Wikidata, what solution are you offering ''now''? --[[User:RexxS|RexxS]] ([[User talk:RexxS|talk]]) 20:25, 2 May 2017 (UTC)
:::::::::::::As someone working here in en-Wikipedia, [[User:I9606]] you are '''obligated''' to follow en-WP policies and guidelines and as RexxS noted MEDRS has broad and deep consensus in en-WP.  We all know that there is no similar thing in Wikidata and as I noted way above, I have zero desire to try to change Wikidata or enforce en-WP policies and guidelines there. But the differing policy/guideline environments, is why care must be taken moving data between the two projects. [[User:Jytdog|Jytdog]] ([[User talk:Jytdog|talk]]) 20:31, 2 May 2017 (UTC) 

* so at this point i am asking that the "diseases" field and the field that produces "interacts with" be removed from this template.  Both get into health content that we cannot rely on to be MEDRS-sourced in Wikidata. The purely biochem data, I don't mind.  But these two should go. Would you please remove from them from the output?  ([https://en.wikipedia.org/wiki/Wikipedia_talk:WikiProject_Medicine/Archive_95#More_Wikidata_funk_-_infobox_gene here] is the corresponding thread at WT:MED that I had opened.  There have been a bunch of discussions at WT:MED about these kinds of fields in Wikidata.  See for example [https://en.wikipedia.org/wiki/Wikipedia_talk:WikiProject_Medicine/Archive_92#Wikidata_for_pneumonia_.28failed_experiment_maybe.29 this one] which was probably the most clear about the concerns).  Thanks. [[User:Jytdog|Jytdog]] ([[User talk:Jytdog|talk]]) 17:50, 2 May 2017 (UTC)

I removed the offending information from the infobox.  The change should percolate.  More later on a strategy for moving forward and why I think this was not such a great move.  --[[User:I9606|Benjamin Good]] ([[User talk:I9606|talk]]) 21:07, 2 May 2017 (UTC)
:Thank you!!!  That was a great move. [[User:Jytdog|Jytdog]] ([[User talk:Jytdog|talk]]) 22:23, 2 May 2017 (UTC)
::That change just made more than 11,000 articles less informative, losing the community value produced by a lot of people's hard work here, in wikidata, and elsewhere.  I personally don't think that was a positive change.  I did it as a temporary patch while a pattern that is more satisfactory to the majority of WP editors can be constructed.  [[User:I9606|Benjamin Good]] ([[User talk:I9606|talk]]) 03:36, 3 May 2017 (UTC)
:::That change just removed unreliable information from 11,000 articles.  en-WP =/= Wikidata and I appreciate you respecting the policies and guidelines of en-WP in deed, if not in word.  Thanks again.  (I ''really'' mean that. I did not want this to become a dramafest.  So thanks. )   [[User:Jytdog|Jytdog]] ([[User talk:Jytdog|talk]]) 05:12, 3 May 2017 (UTC)

== Linking to the UCSC genome browser for mouse chromosomes is broken ==

See e.g. the link at [[Peripheral myelin protein 22]], which goes to human chromosome 11 instead of mouse chromosome 11. Substituting 'hg38' with 'mm10' in the browser URL field seems to do the trick (in the final URL); as does substituting 'mm0' with 'mm10' in the URL provided in the article. --[[User:Njardarlogar|Njardarlogar]] ([[User talk:Njardarlogar|talk]]) 17:11, 11 June 2017 (UTC)
:{{Done}} Fixed[https://en.wikipedia.org/w/index.php?title=Module:Infobox_gene&diff=792766535&oldid=784324140] --[[User:Was a bee|Was a bee]] ([[User talk:Was a bee|talk]]) 14:57, 28 July 2017 (UTC)

== Multiple errors ==

I just undid the most recent changes as they seem to be breaking the infobox on a number of pages, in particular [[MT-ND4]], [[MT-CYB]], [[MT-ND3]], [[MT-ND1]], [[MT-ND5]], [[MT-ND6]]. [[MT-ND4L]]. At time of writing they have the error {{Error|Lua error in Module:Infobox_gene at line 927: attempt to perform arithmetic on local 'chrLength_mm' (a nil value).}} in place of the infobox. As the box contains no parameters I cannot fix it in the articles, and do not feel confident fixing it in the module as it unclear what it is trying to do with that data.--<small>[[User:JohnBlackburne|JohnBlackburne]]</small><sup>[[User_talk:JohnBlackburne|words]]</sup><sub style="margin-left:-2.0ex;">[[Special:Contributions/JohnBlackburne|deeds]]</sub> 18:28, 19 August 2017 (UTC)
:{{Ping|JohnBlackburne}} Sorry for your worries. These are [[mtDNA]] pages which need a bit special treatment, among around ten thousand of gene pages. I have already written new code which doesn't generate error. I'm fixing details and related settings now. Thank you. --[[User:Was a bee|Was a bee]] ([[User talk:Was a bee|talk]]) 19:17, 19 August 2017 (UTC)
:::{{Done}} Fixed. ([https://en.wikipedia.org/w/index.php?title=Module%3AInfobox_gene&type=revision&diff=796353023&oldid=796245683 diff from former revision]) --[[User:Was a bee|Was a bee]] ([[User talk:Was a bee|talk]]) 07:48, 20 August 2017 (UTC)
::::Thanks. I wish there was some better way to handle this. From your description this is some edge case you were not aware of, so did not test against. but that happens all the time with infoboxes and templates. Often the only way to test them fully is publish your changes and see what breaks.
::::The problem is with the way this template is implemented. Other templates that go wrong are small, and so have little impact if they break. Or if they are large like {{tl|infobox settlement}} they are mostly Wikitext, so if a script goes wrong it only affects one element of it. But this template/module is entirely implemented in code. If it goes wrong nothing is rendered except the error. And as many articles containing it contain little else it effectively wrecks them.
::::Often if a template breaks in a few articles it’s often easy to identify the problem with those articles; perhaps a particular parameter is missing or incorrect, and it can be fixed in the articles. But this template normally takes no parameters, so even when the error message suggests some data is missing (as this did) it’s of no help.
::::It would be better perhaps if this template worked more like other infobox templates, with the layout done in Wikitext, with individual fields implemented as calls to templates/modules. Then if one part of it fails the rest of it still works. It would also be much easier to identify what the problem is, if it were with just one part of it. Those individual parts would be much simpler, much easier to fix if they were broken. Long term we can’t assume someone familiar with the code such as yourself will always be available, so having it as maintainable as possible is also important.--<small>[[User:JohnBlackburne|JohnBlackburne]]</small><sup>[[User_talk:JohnBlackburne|words]]</sup><sub style="margin-left:-2.0ex;">[[Special:Contributions/JohnBlackburne|deeds]]</sub> 12:19, 20 August 2017 (UTC)
:::::Certainly. You got a point. This template is complex. But at the same time, it seems there is a reason for that. It is Wikidata data structure is not simple. For example, genomic data of [[ABO (gene)]] about human is stored at [[:d:Q14839826]] (this page is connected to Wikipedia page). But genomic data of ABO (gene) about mouse is stored at [[:d:Q14839892]]. And these two pages are interconnected each other through "ortholog" (P684) property entry. So this module searches mouse data through "ortholog" property in the human genomic data page and shows it. I don't know whether it is possible to simplify the code processing this kind of task. But if possible, it's nice :) --[[User:Was a bee|Was a bee]] ([[User talk:Was a bee|talk]]) 19:41, 23 August 2017 (UTC)

== Breaks when wikipedia article renamed ? ==

I renamed [https://en.wikipedia.org/w/index.php?title=CKLF_like_MARVEL_transmembrane_domain_containing_7&oldid=798021738 old revision] and it [https://en.wikipedia.org/wiki/CKLF_like_MARVEL_transmembrane_domain_containing_7 now] gives an infobox error. Looked at the template talk page (and two wikidata items [https://www.wikidata.org/wiki/Q18048859 CMTM7] and [https://www.wikidata.org/wiki/Q18271556 cmtm7]) but Can't see what is wrong or how to fix it. - Do I have to change something in wikidata or should the template still work ? - [[User:Rod57|Rod57]] ([[User talk:Rod57|talk]]) 15:41, 3 September 2017 (UTC)

== Gene location column added ==

[[File:Infobox gene location before after.png|thumb|New column added]]
Even though there are gene location data at Wikidata, the data was not shown in Wikipedia.  So I  implemented the gene location column to the gene infobox ([https://en.wikipedia.org/w/index.php?title=Module:Infobox_gene&diff=796086073&oldid=792766535 diff]).

Addition in a nutshell:
* Genetic location is shown in graphically and numerically with the source information. Gene position data is coming from Wikidata. The location data have been maintained by [[:wikidata:User:ProteinBoxBot]] (operated by Su team). Thank you team!
* Cytogenetic band data is not yet implemented (shown as ''No data available''). Inclusion is now under discussion at [[:wikidata:Wikidata:Property proposal/Cytogenetic location]]. Need more participant.
* Some bit technical memos about newly added column is here ([[User:Was a bee/Gene]]).

Though I suppose newly added part works properly in all gene article pages, if there are error/bug, and any comment, feel free to post here! Thanks!--[[User:Was a bee|Was a bee]] ([[User talk:Was a bee|talk]]) 11:40, 18 August 2017 (UTC)

Some page links for your convenience when checking
* [[Sonic hedgehog|SHH]] - typical? gene
* [[ABO (gene)|ABO]] - blood type
* [[RUNX1]] - large gene
* [[Interleukin-9 receptor|IL9R]] - gene on [[pseudoautosomal region]]

:@[[User:Was a bee|Was a bee]]: A problem is appearing in several articles, for example, [[CALM2]] and [[HSPA1A]]. I had a very quick look at the module to see what was happening. If you wanted an alternative to <code>unpack(args)</code>, see function <code>wikidata_call</code> in [[Module:Convert]]. That won't help with the error but <code>...</code> is recommended. [[User:Johnuniq|Johnuniq]] ([[User talk:Johnuniq|talk]]) 07:52, 19 August 2017 (UTC)
:I cleaned out a few unrelated articles and [https://en.wikipedia.org/w/api.php?action=query&list=categorymembers&cmprop=title&cmlimit=500&cmnamespace=0&format=xml&cmtitle=Category:Pages_with_script_errors this API call] lists articles with a script error. [[User:Johnuniq|Johnuniq]] ([[User talk:Johnuniq|talk]]) 08:04, 19 August 2017 (UTC)
::{{ping|Johnuniq}} Thank you. That API link is very helpful. I investigate the cause of errors. --[[User:Was a bee|Was a bee]] ([[User talk:Was a bee|talk]]) 09:29, 19 August 2017 (UTC)
{{ping|Was a bee}} The cytogenic location for human genes are now added. [[User:Gstupp|Gstupp]] ([[User talk:Gstupp|talk]]) 16:52, 18 September 2017 (UTC)
:{{ping|Gstupp}} oh, amazing! When I see this page ([[:d:Wikidata:Database reports/Constraint violations/P4196]]), I was surprised about its page numbers. Current usage of cytogenic location property is 55k pages in gene category. Its astonishing number. Thank you very much! --[[User:Was a bee|Was a bee]] ([[User talk:Was a bee|talk]]) 11:39, 19 September 2017 (UTC)

== Add an optional autocollapse parameter to collapse "Gene location (Human)" and "RNA expression pattern"? ==

Wondering what others think about this.  Autocollapsing those sections would improve page formatting in some gene articles where the infobox breaks into sections where images are located and pushes them downward as a result.  [[User:Seppi333|'''<span style="color:#32CD32;">Seppi</span>''<span style="color:Black;">333</span>''''']]&nbsp;([[User Talk:Seppi333|Insert&nbsp;'''2¢''']]) 20:14, 25 November 2017 (UTC)

== Infobox gene mysteriously forgets to add commas between aliases ==

In [[Special:Diff/830125782]], I tried to add a (rather incorrect) infobox for Dll on the DLX family page by manually specifying a root_qid, and was greeted by a very wide infobox. It appears that all the commas between aliases are lost for some reason. Manually fetching the entity and its aliases via REPL seems to give normal results, so something else must be wrong. Can it be the "get rid of gene name if in aliases list" part? --[[User:Artoria2e5|Artoria]][[User talk:Artoria2e5|2e5]] <small style="font-weight:lighter">[[Special:Contributions/Artoria2e5|contrib]]</small> 22:30, 12 March 2018 (UTC)

Bingo! It seems that gene_symbol is nil for Dll. Perhaps we should make this thing a bit less human-centric starting on April Fool's Day. --[[User:Artoria2e5|Artoria]][[User talk:Artoria2e5|2e5]] <small style="font-weight:lighter">[[Special:Contributions/Artoria2e5|contrib]]</small> 22:32, 12 March 2018 (UTC)

== Linking "Ortholog" ==
The rendered output uses (where appropriate) the term "orthologs", which is a term a general reader is unlikely to have heard of before. We have a redirect at [[ortholog]] that seems to remedy that; I believe we should link the term in the rendered output. I've tried this in the sandbox, and it seems to work, so I've implemented it here. Please back the change off if it breaks anything. -- [[User:The Anome|The Anome]] ([[User talk:The Anome|talk]]) 16:56, 5 June 2018 (UTC)

== IUPHAR links ==

The links to IUPHAR have recently been split into targets and ligands. Since proteins can behave either as targets ([[Dopamine receptor D1]]) or ligands ([[insulin]]), Infobox gene should be updated to reflect the new IUPHAR database structure. A new wikidata property has been created for this purpose ([https://www.wikidata.org/wiki/Wikidata:Property_proposal/Guide_to_Pharmacology_Target_ID Guide to Pharmacology Target ID]).

I modified the the sandbox ([https://en.wikipedia.org/w/index.php?title=Special%3AComparePages&page1=Module%3AInfobox+gene&page2=Module%3AInfobox+gene%2Fsandbox diff]) to display this IUPHAR data and the test cases appear to work (see [[Template:Infobox gene/testcases]], compare the External IDs section where IUPHAR IDs now appear in the sandbox version).  Does this look OK?

The second question is how to upload the data into wikidata.  {{tl|IUPHAR}} has a mapping between HUGO gene symbol and the IUPHAR target IDs (although some of these need to be updated).  I am not very familiar with wikidata, but I have added the data manually (e.g., [https://www.wikidata.org/w/index.php?title=Q21110867&diff=prev&oldid=716694797 diff]) for the test cases.  I can provide an Excel file with the required data, but is HUGO gene symbol sufficient to locate the required wikidata entry (e.g., {{Q|21110867}})? I assume that file a [https://www.wikidata.org/wiki/Wikidata:Bot_requests bot request] to get this done, but I am insure that the HUGO gene symbol is sufficient information. Suggestions would be appreciated. [[User:Boghog|Boghog]] ([[User talk:Boghog|talk]]) 20:27, 27 July 2018 (UTC)

== EC links ==

I noticed that wikidata is not populated with the enzyme commission numbers, but the infobox can display them if the data is loaded into wikidata (see for example [https://www.wikidata.org/w/index.php?title=Q21155117&diff=prev&oldid=716427532 diff] and [[ADH1A]]).  As with the IUPHAR request above, I can supply an Excel sheet with the required data, but I am unsure about which identifier I should use to locate the required human protein page.  Is the Hugo Gene symbol sufficient? [[User:Boghog|Boghog]] ([[User talk:Boghog|talk]]) 20:34, 27 July 2018 (UTC)

== Making Infobox gene more understandable/useful for the general reader ==

There has been some rather passionate dicussion about making Infobox gene more accessible for the general reader (see [[WT:PHARMA#Readability_of_medical_physiology_articles|here]] and [[WT:MCB#Improvements_to_infobox_gene|here]]).  One idea that had gained some support was promoting a subset of the most important GO data for display in a uncollapsed form near the top of the infobox.  These key properties would provide answers to basic questions such as what is the function, mechanism, and subcellular location of the protein.

Using the Gene Ontology section in [[Beta-2 adrenergic receptor]] as an example, if we were to [[d:HELP:Ranking|rank]] a subset of GO data as "preferred" for the wikidata {{Q|287961}} data set (see [https://www.wikidata.org/w/index.php?title=Q287961&type=revision&diff=708012317&oldid=697583204 diff]), the following could be displayed in the infobox:

{| class="wikitable"
|+ Key Properties
|-
! Property !! Description
|-
| [[Biological process|Biological function]] || regulation of smooth muscle contraction
|-
| [[Protein#Cellular_functions|Molecular mechanism]] || G-protein coupled receptor activity, epinephrine binding, norepinephrine binding
|-
| [[Cellular component|Subcellular location]] || membrane
|}

For [[BRCA1]] the promoted data could look like:
{| class="wikitable"
|+ Key Properties
|-
! Property !! Description
|-
| [[Biological process|Biological function]] || DNA repair
|-
| [[Protein#Cellular_functions|Molecular mechanism]] || damaged DNA binding
|-
| [[Cellular component|Subcellular location]] || plasma membrane, cytoplasm, cell nucleus
|}

First question: Is there support for such a proposal? Second question: if so, would someone be willing to make a mockup in a sandbox?  I would do this myself, but I am not very familiar with lua. [[User:Boghog|Boghog]] ([[User talk:Boghog|talk]]) 20:59, 27 July 2018 (UTC)

Promoting the data would be a lot of work, but because of similarities of proteins within the same family, the data were processed family wise, the process could be speed up considerably.  The process could be partially automatted:
# down load the GO data from wikidata into a spread sheet (automated)
# organize by family (automated)
# select preferred data (manual)
# adjust the wikidata ranking of the selected data (automated)
[[User:Boghog|Boghog]] ([[User talk:Boghog|talk]]) 21:12, 27 July 2018 (UTC)
:{{ping|Boghog}} That is interesting feature. At first, technically implementing "''filtering by rank''" in Lua is possible by [[:mw:Extension:Wikibase_Client/Lua#mw.wikibase.entity.claimRanks|claimRanks]]. So I think I can implement that.
:One question I have is "''how to select preferred data''"? Surely I agree that "Key Properties" feature is good and very helpful. But at the same time, I feel that we have to be able to show the reason or algorithm that "''how did we choose this as Key property''", especially for function field. For example, [[HBA1]] (building block of [[hemoglobin]] which is important content of [[red blood cell]]) has various GO functions. How do you choose one key property among those....? Is there any algorithm or idea or something? --[[User:Was a bee|Was a bee]] ([[User talk:Was a bee|talk]]) 12:49, 28 July 2018 (UTC)
::{{ping|Was a bee}} Thanks for your response and for confirming that is technically possible to implement this proposal. One thing that I wanted to make absolutely clear is that there is no reliable algorithm to select key properties. This subjective selection must rely on human editors.  The problem with GO descriptions is that for many proteins, the list of terms is so overwhelming to the point where is useless to the average reader.  We need to filter the list to highlight most important GO terms.  For well understood protein families, this is fairly straight forward.  For less studied proteins, it may not be so clear.  Hence promotions of GO terms should only be done by editors that that have taken the time to read the relevant literature.  Another important point is that this is an editor controled "opt-in" scheme. If an editor does not make a conscious decision to promote a GO term, it will not be displayed.  If another editor objects to the decision, it can be reverted.  In short, the selection is editor driven, not bot driven. Finally I wanted to emphasize that there is no rush to promote GO terms.  We can start with well understood protein families where the assignment of primary function is uncontroversial.  More difficult cases can wait. [[User:Boghog|Boghog]] ([[User talk:Boghog|talk]]) 16:01, 28 July 2018 (UTC)
:::{{ping|Boghog}} Thank you for answering. 
:::*I think it is OK. I think it is something like commonsensical summary among researchers who know that topic and share contexts. When I am asked "Where is [[Los Angeles]]?", I'll answer "California" or "West coast" or "America". I don't answer "On surface of the earth" or "At east side of Pacific ocean" or "Milky-way galaxy". I can't say clearly why this is so, but shared context forcing me to do so.  
:::*I did simple test edit at sandbox[https://en.wikipedia.org/w/index.php?title=Module:Infobox_gene/sandbox&diff=prev&oldid=852563853] which shows only "preferred" GO items. Here is sample page which uses sandbox module ([[User:Was a bee/gene sandbox]]). --[[User:Was a bee|Was a bee]] ([[User talk:Was a bee|talk]]) 22:09, 29 July 2018 (UTC)
::::{{ec}} The analogy between city/geographical location on protein/subcellular location is not a good one since a city has only one instance and hence only one location whereas there can be many copies of the same protein that be distributed in more than one location.  In biology, there are at least three parts to the location question: (1) species (specified by External IDs/[[HomoloGene]]), (2) tissue (specified indirectly by "[[RNA expression]] pattern"), and (3) subcellular location (specified by the GO cellular component term). The best way to give context in internal wikilinks that define what the subcellular location means. Perhaps we should add "species distribution" (e.g., primate, mammals, vertebrates, eukaryotes, etc.)"tissue distribution" captured from for example the [[Human Protein Atlas]] as a key fact, but that would be two new projects.
:::: Thanks for your test edit that displays only preferred GO data.  Just to clarify what I had intended is that the all the GO data be retained in a collapsed form as is now done.  But then in addition, insert a new "key facts" section in an uncollapsed state. [[User:Boghog|Boghog]] ([[User talk:Boghog|talk]]) 05:10, 30 July 2018 (UTC)
:::: I think the key word here is "context".  For Los Angeles, California is the proper context, for earth, the solar system is the proper context, for a protein, a species, tissue or part of a cell is the proper context. [[User:Boghog|Boghog]] ([[User talk:Boghog|talk]]) 05:57, 30 July 2018 (UTC)

Just sharing some quick thoughts here. Overall, I'm super supportive of this infobox finally getting a refresh.  I've been meaning to chime in on the other longer discussion, but just haven't found the time.  On this specific proposal, I have two questions/concerns.  First, is the plan that when preferred rank statements exist, the normal rank statements are not shown?  I'm not sure I support that.  As an alternative, perhaps those statements get bolded at put at the top of each section?  Second, I'm not sure I like the idea of using the wikidata rank system.  Since WD changes do not show up in WP edit history, changes in WD don't trigger WP watchlists.  And I know this has been a source of friction from the WP community in the past.  As an alternative, perhaps the preferred GO terms can be indicated in the <code>#invoke</code> statement?  Anyway, again, just throwing out some ideas for discussion.  Unfortunately I'm a bit tied up over the next 2 weeks so forgive me if I can't stay engaged here.  But definitely don't let that stop you from being bold and moving forward!  Best, [[User:Andrew Su|Andrew Su]] ([[User talk:Andrew Su|talk]]) 04:44, 30 July 2018 (UTC)
: Thanks Andrew for your reply.  Just to clarify, my proposal was to display all the GO data in a collapsed state as is done now, but only display "preferred" GO data in an uncollapsed state. [[User:Boghog|Boghog]] ([[User talk:Boghog|talk]]) 05:12, 30 July 2018 (UTC)
: Also thanks for pointing out the issue of using the wikidata ranking system. Specifying preferred data using  <code>#invoke</code> statements might be better from a documentation standpoint, although not as clean.  I could support either solution. [[User:Boghog|Boghog]] ([[User talk:Boghog|talk]]) 06:08, 30 July 2018 (UTC)

== Italics ==

It seems that gene symbols are italicized, and should be when they're the title of the infobox invoked here. See [[DGCR2]]. Can this be implemented? If this is the wrong place to ask, can someone point me in the right direction for the request? -- [[User:JHunterJ|JHunterJ]] ([[User talk:JHunterJ|talk]]) 19:21, 18 September 2018 (UTC)

== Image caption ==

Wikidata image (P18) can be accopanied by a caption (P2096) to annotate the image, which is especially useful for protein/receptor-ligand complexes because it helps to know which is which. Maybe add it to getImage()? CC [[User:Was a bee]]. --[[User:Artoria2e5|Artoria]][[User talk:Artoria2e5|2e5]] <small style="font-weight:lighter">[[Special:Contributions/Artoria2e5|🌉]]</small> 20:46, 11 April 2019 (UTC)
:Hi {{ping|Artoria2e5}}. That sounds good idea. From your [[User:Artoria2e5#Infobox_gene|user page]], I think you can implement new code by yourself. So if you make test version, I'll test the new code at various pages :) --[[User:Was a bee|Was a bee]] ([[User talk:Was a bee|talk]]) 07:54, 12 April 2019 (UTC)

== PDB links ==

Hi {{ping|Julialturner}}  The old {{tl|GNF Protein box}} produced PDB links (PDB Ortholog search: PDBe RCSB) based on {{tl|Homologene2uniprot}} so that crystal structures for both human and orthologs in other species were returned.  The new {{tl|Infobox gene}} returns crystal structures only for mouse. At a bare minimum, the link should be changed to human from mouse since there are far more human crystal structures ([http://www.rcsb.org/pdb/statistics/histogram.do?mdcat=entity_src_gen&mditem=pdbx_gene_src_scientific_name&numOfbars=50&name=Source%20Organism%20(Gene%20Source) currently] 35929 human vs. 5474 mouse). Ideally of course, the crystal structures for all the orthlogs should be returned. Thanks. [[User:Boghog|Boghog]] ([[User talk:Boghog|talk]]) 12:54, 9 August 2016 (UTC)
::Thanks, {{ping|Boghog}} I will look into the differences between old and new and see if I can make adjustments to the new infobox. [[User:Julialturner|Julialturner]] ([[User talk:Julialturner|talk]]) 19:41, 9 August 2016 (UTC)
::{{ping|Boghog}} the PDB link and RCSB should now have both the human and mouse structures returned.  Currently, we are only displaying mouse data, but maybe future development could include other species.  [[User:Julialturner|Julialturner]] ([[User talk:Julialturner|talk]]) 21:33, 9 August 2016 (UTC)
::: The current version once again only does it for the human entity_protein. I am thinking about iterating through all of <code>entity:getAllStatements(ortholog_propertyID)</code>, grabbing <code>protein_propertyID</code> from genes, and storing the orthologs so getPDB can use them. I guess storing QIDs for <code>mw.wikibase.getAllStatements(id, prop)</code> would be good enough, since we don't need the whole entity (expensive). --[[User:Artoria2e5|Artoria]][[User talk:Artoria2e5|2e5]] <small style="font-weight:lighter">[[Special:Contributions/Artoria2e5|🌉]]</small> 20:51, 11 April 2019 (UTC)

{{Edit template-protected|ans=y}}
Hi! Revision 892056776 in the sandbox is the new version for displaying human, mouse, and for trying to find more orthologs ([https://en.wikipedia.org/w/index.php?title=Module%3AInfobox_gene%2Fsandbox&type=revision&diff=892056776&oldid=892048115 diff]). If you think it's OK, please merge it into the current Module. --[[User:Artoria2e5|Artoria]][[User talk:Artoria2e5|2e5]] <small style="font-weight:lighter">[[Special:Contributions/Artoria2e5|🌉]]</small> 22:14, 11 April 2019 (UTC)
::[[File:Red information icon with gradient background.svg|20px|link=|alt=]] '''Not done:''' please establish a [[Wikipedia:Consensus|consensus]] for this alteration '''[[Wikipedia:Edit requests|before]]''' using the {{tlx|edit template-protected}} template.<!-- Template:ETp --> {{u|Artoria2e5}}, the template asks for ''"a complete and specific description of the request, so that an editor unfamiliar with the subject matter could complete the requested edit immediately"''. You're certainly not going to get an opinion [[Of Mice and Men|on humans, mice]] and orthologs from template editors. Please gather some consensus from folks who may have a clue like {{u|Boghog}}, {{u|Julialturner}}, or probably better still at [[WT:GEN]]. A more complete description of what your changes are intended to achieve would be a good starting point. [[User:Cabayi|Cabayi]] ([[User talk:Cabayi|talk]]) 12:21, 19 April 2019 (UTC)
::: I agree that it would have been best to provide a link to all the available orthologs (just to note, the current version of the infobox does provide links to both human and mouse orthologs, see for example [[Estrogen receptor beta]], [http://www.rcsb.org/pdb/results/results.do?tabtoshow=Current&qrid=B3F42575 RCSB PDB link]).  Before implementing this, we need to test the sandbox version in [[Module:Infobox gene/testcases]]. I am not at all sure how to do this for a [[Lua (programming language)]] module. Perhaps {{ping|Julialturner}} or one of the other maintainers of this module could implement test cases? [[User:Boghog|Boghog]] ([[User talk:Boghog|talk]]) 05:09, 20 April 2019 (UTC)
::: The sandbox version doesn't seem to return other orthlogs (see for example [[User:Boghog/sandbox]] for estrogen receptor beta).  The PDB contains structures for human and rat, while the links produced by the sandbox version only searches for human, mouse, but not rat. [[User:Boghog|Boghog]] ([[User talk:Boghog|talk]]) 05:27, 20 April 2019 (UTC)
::: There is also a bug in the sandbox. For example, in this [[User:Boghog/sandbox|test case]]:  If one uncollapses "List of PDB id codes [show]", the following error message is displayed: VALUE_ERROR (bad argument #1 to 'getAllStatements' (string expected, got boolean)).  Compare with the infobox in [[Estrogen receptor beta]] where there is no error.
::::{{ping|User:Andrew Su}} Would you be able to recommend next steps in testing this implementation?

== Template-protected edit request on 27 May 2019: duplicate ref tags ==

{{edit template-protected|Module:Infobox gene|answered=yes}}
The module currently always emits the same-named ref tags over multiple invocations, causing problems when a page is used for different genes. There should be a suffix added to remedy this issue. Please do the following:

* For every instance of {{code|"refGRCh38Ensembl"}}, change to {{code|"refGRCh38Ensembl" .. ensembl}}. Likewise for {{code|"refGRCh37Ensembl"}}.
* For every instance of  {{code|"refGRCm38Ensembl"}}, change to {{code|"refGRCh38Ensembl" .. ensembl_mm}}.

Here I am using ensembl's IDs as suffixes, which nicely enough does encode the species and its ensembl origin in it. And honestly I don't get why it is putting "ref" as a prefix. You might want to change "refGRCh38Ensembl" to "GRCh38_" to make things shorter and vice versa.

[[User:Artoria2e5|Artoria]][[User talk:Artoria2e5|2e5]] <small style="font-weight:lighter">[[Special:Contributions/Artoria2e5|🌉]]</small> 01:53, 27 May 2019 (UTC)
:Agreed!  But where does "ensembl" come from? Will it be unique when invoked for different QIDs on the same page? -- [[User:Mikeblas|Mikeblas]] ([[User talk:Mikeblas|talk]]) 00:32, 28 May 2019 (UTC)
:: [[User:Mikeblas]], the ensembl variables are the gene identifiers used by Ensemble to uniquely identify a gene. It is the long ENSG00000 something you see in these link names. Even if someone erroneously calls the template for the same gene twice, there will be no reference error reported since the content of the identically-named ref tags will be the same anyways. --[[User:Artoria2e5|Artoria]][[User talk:Artoria2e5|2e5]] <small style="font-weight:lighter">[[Special:Contributions/Artoria2e5|🌉]]</small> 00:56, 28 May 2019 (UTC)
:[[File:Red information icon with gradient background.svg|20px|link=|alt=]] '''Not done:''' please make your requested changes to the module's sandbox first; see [[Wikipedia:Template sandbox and test cases|WP:TESTCASES]].<!-- Template:ETp --> &mdash;&nbsp;Martin <small>([[User:MSGJ|MSGJ]]&nbsp;·&nbsp;[[User talk:MSGJ|talk]])</small> 08:05, 4 June 2019 (UTC)

:Is it really not possible to fix this problem? It's been broken for more than a month. -- [[User:Mikeblas|Mikeblas]] ([[User talk:Mikeblas|talk]]) 23:56, 14 June 2019 (UTC)
::{{not done}} {{Re|Mikeblas}} it can certainly be discussed still, but as was already requested above, this needs to be carefully sandboxed first. — [[User:Xaosflux|<span style="color:#FF9933; font-weight:bold; font-family:monotype;">xaosflux</span>]] <sup>[[User talk:Xaosflux|<span style="color:#009933;">Talk</span>]]</sup> 16:21, 15 June 2019 (UTC)

== Fixing localization support ==

Instead of parsing for "MT" from chromosome name this should lookup P1813 which has it correctly: that way code would be easier to port without changing hard-coded names in strings. Lengths should be used from property P2043 instead of having hard-coded table of lengths in the code. Changing this second part would actually completely do away with the first part since the short name is not used anywhere else? [[User:Ipr1|Ipr1]] ([[User talk:Ipr1|talk]]) 07:54, 8 August 2019 (UTC)

== (un)required modules? ==

So why are those three other modules marked as required when they are not required and not used or referenced in any way in this module? [[User:Ipr1|Ipr1]] ([[User talk:Ipr1|talk]]) 17:24, 8 August 2019 (UTC)

== Causing reference errors on [[Cryptochrome]]? ==

I don't know if this actually a problem with the module, or with the data on Wikidata... But using it on [[Cryptochrome]] gives cite errors....

<pre>
 GRCh38: Ensembl release 89: ENSG00000008405 - Ensembl, May 2017 Cite error: Invalid <ref> tag; name "refGRCh38Ensembl" defined multiple times with different content
 GRCm38: Ensembl release 89: ENSMUSG00000020038 - Ensembl, May 2017 Cite error: Invalid <ref> tag; name "refGRCm38Ensembl" defined multiple times with different content
</pre>

[[User:Reedy|Reedy]] ([[User talk:Reedy|talk]]) 01:23, 10 December 2019 (UTC)
:I believe you are saying that if the contents of [[Cryptochrome]] is replaced with the following and previewed, then the error message is displayed.
<syntaxhighlight lang=moin>
{{#invoke:Infobox_gene|getTemplateData}}
{{#invoke:Infobox_gene|getTemplateData|QID=Q14866005}}
{{#invoke:Infobox_gene|getTemplateData|QID=Q17909946}}
</syntaxhighlight>
:The first line is inserted; the following two lines already exist in the article. Here are the corresponding Wikidata items:
:*{{Q|Q416345}}
:*{{Q|Q14866005}}
:*{{Q|Q17909946}}
:Searching the previewed page for <code>[1]</code> finds the places where the invalid ref is used.
:I don't know what is going on, but there appears to be a conflict between these items. Are they really wanted? If so, the next step might be to examine what generates the refs at the Wikidata links. [[User:Johnuniq|Johnuniq]] ([[User talk:Johnuniq|talk]]) 02:06, 10 December 2019 (UTC)
:: {{fixed}} {{tl|Infobox gene}} is meant to be used once and only once per article. If more than one {{tl|Infobox gene}} is used per article, a conflict between identical ref names with different content results. It appears that [[CRY1]] and [[CRY2]] were merged into one article which caused the conflict.  I have [https://en.wikipedia.org/w/index.php?title=Cryptochrome&type=revision&diff=930098131&oldid=927118712 replaced] {{tl|Infobox gene}} with the more compact {{tl|Infobox protein}} to removed the conflict. [[User:Boghog|Boghog]] ([[User talk:Boghog|talk]]) 06:29, 10 December 2019 (UTC)

== Propose addition of Expression Atlas links for human, mouse and other species' gene expression ==

Currently the infobox only shows expression data from BioGPS, which is a resource that according to its latest publication ("https://www.ncbi.nlm.nih.gov/pmc/articles/PMC4702805/"), mostly deals with microarray data. I would like to propose that in addition we add a link on the info box for the expression of a gene/protein to the EMBL-EBI Expression Atlas ("https://academic.oup.com/nar/article/48/D1/D77/5609521" "https://www.ebi.ac.uk/gxa/home"), which holds updated information of expression not only on Microarrays, but superseding sequencing technology RNA-Seq, for both baseline and differential data. Expression Atlas also holds baseline proteomics expression, further complementing the expression view (as gene and protein expression don't normally correlate well). All of the tissues and cell types on Expression Atlas are matched against ontologies, and the resource is updated 2 to 3 times per year with new studies, following ENSEMBL releases mostly. Releases include for many datasets re-alignments against newer ENSEMBL genome annotations, with a standardised pipeline. Recently Expression Atlas also started collecting Single Cell RNA-Seq data as well, in its sister resource "https://www.ebi.ac.uk/gxa/sc/home". I would be happy to do the legwork if I'm indicated where to do it. I honestly think that this would improve the infobox provisioning of gene/protein expression data. Disclosure: I am the Data Production Project Leader for Expression Atlas ("https://www.ebi.ac.uk/about/people/pablo-moreno").[[User:Pcm32|Pcm32]] ([[User talk:Pcm32|talk]]) 11:04, 14 March 2020 (UTC)

: I agree that links to protein expression databases would be helpful. In addition to the EMBL-EBI's [[Expression Atlas]] (btw, the corresponding Wikipedia article should be updated), there is also the [[Human Protein Atlas]] and probably several others. Comparing the [[insulin]] entry for the two databases:
:* [[Expression Atlas]]: [https://www.ebi.ac.uk/gxa/genes/ensg00000254647?bs=%7B%22homo%20sapiens%22%3A%5B%22ORGANISM_PART%22%5D%7D&ds=%7B%22kingdom%22%3A%5B%22animals%22%5D%7D#baseline Insulin (homo sapiens) ]
:* [[Human Protein Atlas]]: [https://www.proteinatlas.org/ENSG00000254647-INS Insulin (homo sapiens) ]
: shows that the two databases are only partially overlapping and therefore a case can be made for including links to both. [[User:Boghog|Boghog]] ([[User talk:Boghog|talk]]) 13:14, 14 March 2020 (UTC)

== Please update HGNC URL to current format ==

The current links to HGNC ("https://www.genenames.org/cgi-bin/gene_symbol_report?hgnc_id=") are out of date - currently a redirect is in place but this may not be the case in the long term. Please update to "https://www.genenames.org/data/gene-symbol-report/#!/hgnc_id/".
[[User:Aretaon|Aretaon]] ([[User talk:Aretaon|talk]]) 10:17, 16 December 2019 (UTC)
:Just re-upping this request with an additional note that since Wikidata stores the URLs for HGNC IDs (P353-> P1630), perhaps it would be more sustainable to retrieve this from Wikidata rather than hard coding it in this module? cc {{ping|Julialturner}} {{ping|Was a bee}} (Wasn't sure who to ping, hope this is okay and sorry to bother you!) [[User:Aretaon|Aretaon]] ([[User talk:Aretaon|talk]]) 10:32, 7 January 2020 (UTC)
:{{edit template-protected|answered=yes}}
:Re-upping this request. I do not have permissions to edit the template so please can someone review and make this URL change in the Infobox gene module: replace the  link to HGNC ("https://www.genenames.org/cgi-bin/gene_symbol_report?hgnc_id="), which is several years out of date, with the current link: "https://www.genenames.org/data/gene-symbol-report/#!/hgnc_id/". I made this change in Infobox gene/sandbox already and confirmed that it works fine in the testcase. Thank you! [[User:Aretaon|Aretaon]] ([[User talk:Aretaon|talk]]) 09:13, 1 July 2020 (UTC)
::[[File:Yes check.svg|20px|link=|alt=]] '''Done'''<!-- Template:ETp --> Is it working? I am unable to find an article where it is working. Should it be working at [[SRARP]], which appears to have an HGNC ID in Wikidata? Maybe I don't know what I'm looking for. – [[User:Jonesey95|Jonesey95]] ([[User talk:Jonesey95|talk]]) 00:59, 2 July 2020 (UTC)
::: {{ping|Jonesey95}} Thank you! It looks like it's working just fine to me! I checked in SRARP and also in BRAF and they both looked fine - it's the first link in the "Aliases" field and it now goes to the correct URLs at genenames.org. Thanks so much for your help. [[User:Aretaon|Aretaon]] ([[User talk:Aretaon|talk]]) 09:25, 2 July 2020 (UTC)

==  Locus ==

"Cytogenetic location" now links to https://en.wikipedia.org/wiki/Locus, please change that to https://en.wikipedia.org/wiki/Locus_(genetics), [[Special:Contributions/81.191.116.205|81.191.116.205]] ([[User talk:81.191.116.205|talk]]) 18:34, 16 November 2020 (UTC)

== Localize ==
{{edit template-protected|answered=yes}}
Please add comment --**lclz** to mark line 1223, the text "Sources:" should be localized. --[[User:Hidayatsrf|Hidayatsrf]] ([[User talk:Hidayatsrf|talk]]) 04:04, 2 February 2021 (UTC)
:[[File:Red information icon with gradient background.svg|20px|link=|alt=]] '''Not done for now:'''<!-- Template:ETp --> Please put this in the sandbox to wait for a more major update. [[User:Izno|Izno]] ([[User talk:Izno|talk]]) 04:09, 2 February 2021 (UTC)
::{{Ping|Izno}} okay, this is indeed a very minor change, thanks. --[[User:Hidayatsrf|Hidayatsrf]] ([[User talk:Hidayatsrf|talk]]) 10:36, 2 February 2021 (UTC)

== i18n refine, adding l10n note and cleanup ==

{{edit template-protected|answered=yes}}
See the [https://en.wikipedia.org/w/index.php?title=Module%3AInfobox_gene%2Fsandbox&type=revision&diff=1008455144&oldid=1008450698 changes] and the history of the sandbox page for all changes. You can rule out the unwanted, but I wish to get the reasons. Some changes have been deployed on zhwiki.--[[User:YFdyh000|YFdyh000]] ([[User talk:YFdyh000|talk]]) 11:32, 23 February 2021 (UTC)
:{{u|YFdyh000}} can you please explain what changes you've made, why you made them, and if there are any breaking changes? [[User:Elliot321|Elliot321]] ([[User_talk:Elliot321|talk]] &#124; [[Special:Contributions/Elliot321|contribs]]) 19:07, 24 February 2021 (UTC)
::{{u|Elliot321}} 1. Sync the current version to the sandbox. 2. Added variables to help localization. 3. Clean up spaces at the end of lines. 4. Add more localization notes. 5. Move the colon outside the links. 6. Add useful internal links (like [[Human genome|Human]]). 7. No breaking changes, and the [[Template:Infobox_gene/testcases]] looks fine. --[[User:YFdyh000|YFdyh000]] ([[User talk:YFdyh000|talk]]) 22:08, 24 February 2021 (UTC)
:[[File:Yes check.svg|20px|link=|alt=]] '''Done'''<!-- Template:ETp --> Looks fine, if any issues arise, feel free to open another request (or to undo the changes, if you're a template-editor/sysop). [[User:Elliot321|Elliot321]] ([[User_talk:Elliot321|talk]] &#124; [[Special:Contributions/Elliot321|contribs]]) 00:58, 25 February 2021 (UTC)

== Template-protected edit request on 1 May 2021 ==

{{edit template-protected|Module:Infobox gene|answered=yes}}
To fix the RCSB PDB search, line 299 replace this URL prefix:

<syntaxhighlight lang="lua">
    local RCSB_base = 'http://www.rcsb.org/pdb/search/smartSubquery.do?smartSearchSubtype=UpAccessionIdQuery&amp;accessionIdList='
</syntaxhighlight>

by these 3 lines:

<syntaxhighlight lang="lua">
    local RCSB_base = 'https://www.rcsb.org/search?q='
					..'rcsb_polymer_entity_container_identifiers.reference_sequence_identifiers.database_name:UniProt%20AND%20'
					..'rcsb_polymer_entity_container_identifiers.reference_sequence_identifiers.database_accession:'
</syntaxhighlight>

The display here is not correct, at least on my screen, the strings are too long and get put on  lines separated from their indenting tabs, they're supposed to be aligned with the first line declaring the variable.
It might be safer to copy from the [[Module:Infobox_gene/sandbox]]. [[User:Bonob|bonob]] ([[User talk:Bonob|talk]]) 20:16, 1 May 2021 (UTC)
: {{Done}} [[User:Pppery|* Pppery *]] [[User talk:Pppery|<sub style="color:#800000">it has begun...</sub>]] 04:19, 2 May 2021 (UTC)

== Template-protected edit request on 13 October 2021 ==

{{edit template-protected|Module:Infobox gene|answered=yes}}

=== Changes to include gene expression calls via "expressed in" statements from Wikidata / Bgee database ===
The code below adds into the Wikipedia "infobox gene" the "expressed in" assertions of human genes available in Wikidata from the Bgee database ([https://www.bgee.org www.bgee.org]) to the "RNA expression pattern" section in the infobox. These assertions per gene are at most the top 10 anatomic entities (tissues) where a gene is expressed. 

The code is available in the sandbox at https://en.wikipedia.org/wiki/Module:Infobox_gene/sandbox and tested in https://en.wikipedia.org/w/index.php?title=Module_talk:Infobox_gene/testcases infobox gene testcase talk.

{{cot|Change preview}}
==== Change preview ==== 

Let us take as an example the Wikidata[https://www.wikidata.org/wiki/Q414043 RELN gene], the code below gets "expressed in" Wikidata property values and assign them to a "Bgee DB" row in the section "RNA expression pattern" of the infobox at [[Reelin]] gene.

{| class="wikitable"
|+ Click on show in the "RNA expression pattern" section to see changes
|-
! FROM !! TO
|-
| {{#invoke:Infobox gene|getTemplateData|QID=Q414043}} || {{#invoke:Infobox gene/sandbox|getTemplateData|QID=Q414043}}
|}


==== Change: add line into "if" clause around line 148 ==== 
Add the code below at line 148, for example.
<syntaxhighlight lang="lua">
local expressed_in_tissues = check_values(p.getValue, {entity, "P5572", localNotApplicableStr, localSeparatorStr}) --P5572: expressed in
</syntaxhighlight>

==== Change: add "sep" as an argument of getValue function  ====  
It also set <code>sep = " "</code> as default by replacing line 2149

'''CHANGE''' line 2145
<syntaxhighlight lang="lua">
p.getValue = function(entity, propertyID, return_val)
</syntaxhighlight>
'''TO''' 
<syntaxhighlight lang="lua">
p.getValue = function(entity, propertyID, return_val, sep)
</syntaxhighlight>
'''CHANGE''' line 2149
<syntaxhighlight lang="lua">
local sep = " " --could ad as input parameter if need be
</syntaxhighlight>
'''TO'''
<syntaxhighlight lang="lua">
if sep == nil then sep = " " end
</syntaxhighlight>

=== Change: lines 212-214, to include Bgee DB conditions and arguments === 
'''CHANGE'''
<syntaxhighlight lang="lua">
if expression_images ~= ""  then
	p.renderRNAexpression(expression_images, entrez_gene)
end
</syntaxhighlight>
'''TO'''
<syntaxhighlight lang="lua">
if expression_images ~= "" or expressed_in_tissues ~= localNotApplicableStr then
	p.renderRNAexpression(expression_images, entrez_gene, ensembl, p.renderBgeeExpression(expressed_in_tissues))
end
</syntaxhighlight>

==== Change: add new function to render Bgee DB "expressed in" data from Wikidata ==== 
<syntaxhighlight lang="lua">
--Get the render elements title, collapse option, collapsed anatomic entities,  
--and anatomic entity list for the Bgee gene expression rendering.
p.renderBgeeExpression = function(expressed_in_tissues)
	--create list of expressed in anatomic entities
	local bgee_title = "Bgee" 
	bgee_title = "[https://bgee.org/ " .. bgee_title .. "]"
	local bgee_links = localNotApplicableStr
	local bgee_collapse = "none"
	local bgee_default = "Top expressed in" 
	local split_bgee = mw.text.split(expressed_in_tissues, localSeparatorStr)
	local bgee_link_list = {}
	local results
	for k,v in ipairs(split_bgee) do
		if string.match(v, '%w+') and v ~= localNotApplicableStr then
			bgee_link_list[#bgee_link_list+1] = "\n* "..v
		end
	end
	--if less than 11 don't create collapsible list
	if  table.getn(bgee_link_list) < 11 then
		if bgee_default == nil and table.getn(bgee_link_list) == 0 then bgee_links = localNotApplicableStr end
	else
		bgee_collapse = "collapsible collapsed"
		bgee_default = bgee_default .. '<br>' .. table.remove(bgee_link_list, 1) .. '<br>' .. table.remove(bgee_link_list, 1) .. '<br>' ..table.remove(bgee_link_list, 1) .. '<br>' .. table.remove(bgee_link_list, 1) .. '<br>' .. table.remove(bgee_link_list, 1) .. '<br>'--get first 5 elements in table and use for display
	end
	if bgee_link_list[#bgee_link_list] then
		bgee_links = table.concat(bgee_link_list, "<br>")
	end
	results = {bgee_title, bgee_collapse, bgee_default, bgee_links}
	return results
end
</syntaxhighlight>

==== Change: add Bgee database to the RNA expression render function ==== 
This function should replace the current one starting at line 1104.
'''CHANGE'''
<syntaxhighlight lang="lua">
p.renderRNAexpression = function(expression_images, entrez_gene)
	local title = "[[Gene expression|RNA expression]] pattern" --**lclz**
	local biogps_link = "[http://biogps.org/gene/"..entrez_gene.."/ More reference expression data]" --**lclz**
	root
		:tag('tr')
        	:tag('td')
   				:attr('colspan', 4)
            	:css('text-align', 'center')
            	:css('background-color', rowBGcolor)
            	:tag('table')
            		:attr('class', 'collapsible collapsed')
            		:css('padding', '0')
            		:css('border', 'none')
            		:css('margin', '0')
            		:css('width', '100%')
            		:css('text-align', 'left')
					:tag('tr')
						:tag('th')
							:attr('colspan', '4')
							:css('text-align', 'center')
							:css('background-color', titleBGcolor)
							:wikitext(title)
							:done() --end th
						:done() --end tr
					:tag('tr')
						:tag('td')
							:attr('colspan', '4')
							:css('text-align', 'center')
							:css('background-color', rowBGcolor)
							:wikitext(expression_images)
							:done() --end td
						:done() --end tr
					:tag('tr')
						:tag('td')
							:attr('colspan', '4')
							:css('text-align', 'center')
							:css('background-color', rowBGcolor)
							:tag('span')
								:attr('class', 'plainlinks')
								:wikitext(biogps_link)
								:done() --end span
							:done() --end td
						:done() --end tr
					:done() --end table
				:done() --end td
			:done() --end tr
end
</syntaxhighlight>
'''TO'''
<syntaxhighlight lang="lua">
p.renderRNAexpression = function(expression_images, entrez_gene, ensembl, bgee_expression)
	local title = "[[Gene expression|RNA expression]] pattern" --**lclz**
	local biogps_link = ""
	local biogps_title = "BioGPS"
	biogps_title = "[http://biogps.org/ " .. biogps_title .. "]"
	-- If no expression image exist in BioGPS, the N/A is displayed
	if expression_images ~= ""  then
		biogps_link = "[http://biogps.org/gene/"..entrez_gene.."/ More reference expression data]" --**lclz**	  
	else
		expression_images = localNotApplicableStr
	end
	local bgee_title = bgee_expression[1]
	local bgee_collapse = bgee_expression[2]
	local bgee_default = bgee_expression[3]
	local bgee_links = bgee_expression[4]
	local bgee_more_link = ""
	local ensembl_id = string.gsub(ensembl, "%s", "")
	if bgee_links ~= localNotApplicableStr  then
		bgee_more_link = "[https://bgee.org/?page=gene&gene_id="..ensembl_id.." More reference expression data]" --**lclz**
		bgee_default = "[https://bgee.org/?page=gene&gene_id="..ensembl_id.." '''"..bgee_default.."'''] " --**lclz**
	else
		bgee_default = ""
	end
	root
		:tag('tr')
        	:tag('td')
   				:attr('colspan', 4)
            	:css('text-align', 'center')
            	:css('background-color', rowBGcolor)
            	:tag('table')
            		:attr('class', 'collapsible collapsed')
            		:css('padding', '0')
            		:css('border', 'none')
            		:css('margin', '0')
            		:css('width', '100%')
            		:css('text-align', 'left')
					:tag('tr')
						:tag('th')
							:attr('colspan', '4')
							:css('text-align', 'center')
							:css('background-color', titleBGcolor)
							:wikitext(title)
							:done() --end th
						:done() --end tr
					:tag('tr')
						:tag('th')
        					:attr('scope', 'row')
        					:css('background-color', sideTitleBGcolor)
        					:wikitext(bgee_title)
        					:done() --end th
        				:tag('td')
        					:tag('table')
            					:attr('class', bgee_collapse)
            					:css('padding', '0')
            					:css('border', 'none')
            					:css('margin', '0')
            					:css('width', '100%')
            					:css('text-align', 'left')
            					:tag('tr')
            						:tag('td')
            							:attr('colspan', '1')
            							:tag('span')
            								:attr('class', 'plainlinks')
            								:wikitext(bgee_default)
            								:done() --end span
            							:done() --end td
            						:done() --end tr
            					:tag('tr')
            						:tag('td')
            							:attr('colspan', '1')
            							:tag('span')
            								:attr('class', 'plainlinks')
            								:wikitext(bgee_links)
            								:done() --end span
            							:done() --end td
            						:done() --end tr
            					:tag('tr')
									:tag('td')
										:attr('colspan', '4')
										:css('text-align', 'center')
										:css('background-color', rowBGcolor)
										:tag('span')
											:attr('class', 'plainlinks')
											:wikitext(bgee_more_link)
											:done() --end span
										:done() --end td
									:done() --end tr
            					:done() --end table
            				:done() --end td
						:done() --end tr
					:tag('tr')
						:tag('th')
        					:attr('scope', 'row')
        					:css('background-color', sideTitleBGcolor)
        					:wikitext(biogps_title)
        					:done() --end th
						:tag('td')
							:tag('table')
            					:attr('class', bgee_collapse)
            					:css('padding', '0')
            					:css('border', 'none')
            					:css('margin', '0')
            					:css('width', '100%')
            					:css('text-align', 'left')
            					:tag('tr')
            						:tag('td')
										:attr('colspan', '4')
										:css('text-align', 'center')
										:css('background-color', rowBGcolor)
										:wikitext(expression_images)
										:done() --end td
									:done() --end tr
								:tag('tr')
									:tag('td')
										:attr('colspan', '4')
										:css('text-align', 'center')
										:css('background-color', rowBGcolor)
										:tag('span')
											:attr('class', 'plainlinks')
											:wikitext(biogps_link)
											:done() --end span
										:done() --end td
									:done() --end tr
								:done() --end table
							:done() --end td
						:done() --end tr
					:done() --end table
				:done() --end td
			:done() --end tr
end
</syntaxhighlight>
{{cob}}
[[User:BgeeDB|BgeeDB]] ([[User talk:BgeeDB|talk]]) 14:13, 13 October 2021 (UTC)
:{{to|BgeeDB}} {{done|done}}. '''''[[User:Paine Ellsworth|<span style="font-size:92%;color:darkblue;font-family:Segoe Script">P.I.&nbsp;Ellsworth</span>]]'''''&numsp;-&nbsp;[[Editor|<span style="color:black">ed.</span>]]&nbsp;[[User talk:Paine Ellsworth|<sup>put'r&nbsp;there</sup>]]&nbsp;<small>20:22, 23 October 2021 (UTC)</small>
:{{to|BgeeDB}} {{undone|undone}}. Had to revert for now; please see [[User talk:Paine Ellsworth#Infobox gene]], which shows that this edit caused linking to disambiguation pages from mainspace articles, and that's a no-no. So this edit needs to be fixed. '''''[[User:Paine Ellsworth|<span style="font-size:92%;color:darkblue;font-family:Segoe Script">P.I.&nbsp;Ellsworth</span>]]'''''&numsp;-&nbsp;[[Editor|<span style="color:black">ed.</span>]]&nbsp;[[User talk:Paine Ellsworth|<sup>put'r&nbsp;there</sup>]]&nbsp;<small>20:56, 24 October 2021 (UTC)</small>
: Reping {{u|Tarsmf}} since the user was renamed. [[User:Pppery|* Pppery *]] [[User talk:Pppery|<sub style="color:#800000">it has begun...</sub>]] 21:02, 24 October 2021 (UTC)
::Curious, {{u|Pppery|&#42; Pppery *}}, if red links notify, then why wouldn't redirects notify? '''''[[User:Paine Ellsworth|<span style="font-size:92%;color:darkblue;font-family:Segoe Script">P.I.&nbsp;Ellsworth</span>]]'''''&numsp;-&nbsp;[[Editor|<span style="color:black">ed.</span>]]&nbsp;[[User talk:Paine Ellsworth|<sup>put'r&nbsp;there</sup>]]&nbsp;<small>21:07, 24 October 2021 (UTC)</small>
::{{to|Paine Ellsworth}}Thank you all very much for noticing the ambiguity issue. To avoid any kind of ambiguity I have removed the links to Wikipedia pages. Is it okay for you? Thanks for taking care!<small> [[User:Tarsmf|Tarsmf]] ([[User talk:Tarsmf|talk]]) 07:34, 25 October 2021 (UTC)</small> 
:::Wish I knew more detail about LUA, but if we have your assurance that the sandbox code has been improved and will not load dab pages with a lot of article links, then yes I'm okay with it. So to be clear, you're saying that the sandbox code is ready to go? '''''[[User:Paine Ellsworth|<span style="font-size:92%;color:darkblue;font-family:Segoe Script">P.I.&nbsp;Ellsworth</span>]]'''''&numsp;-&nbsp;[[Editor|<span style="color:black">ed.</span>]]&nbsp;[[User talk:Paine Ellsworth|<sup>put'r&nbsp;there</sup>]]&nbsp;<small>09:45, 25 October 2021 (UTC)</small>
::::Yes, I have changed the part concerning the wikipedia links of anatomical entities to be all a simple text instead of links. You can see in the example above of the RELN gene, how it is now in the sandbox, also here https://en.wikipedia.org/w/index.php?title=Module_talk:Infobox_gene/testcases . Thanks a lot! [[User:Tarsmf|Tarsmf]] ([[User talk:Tarsmf|talk]]) 11:58, 25 October 2021 (UTC) 
:::::Okay then, and {{done|done}} once again. '''''[[User:Paine Ellsworth|<span style="font-size:92%;color:darkblue;font-family:Segoe Script">P.I.&nbsp;Ellsworth</span>]]'''''&numsp;-&nbsp;[[Editor|<span style="color:black">ed.</span>]]&nbsp;[[User talk:Paine Ellsworth|<sup>put'r&nbsp;there</sup>]]&nbsp;<small>12:52, 25 October 2021 (UTC)</small>

{{ref-talk}}

== List construction ==

Please use {{tag|ul|o}} / {{tag|li|o}} to construct lists, rather than pseudo-lists with • and {{tag|br|o}}. The latter is semantically incorrect and may be inaccessible to screen readers. [[User:Hairy Dude|Hairy Dude]] ([[User talk:Hairy Dude|talk]]) 18:31, 4 April 2021 (UTC)
:The relevant code appears to be:
<syntaxhighlight lang="lua">
if sitelink and result_GOID ~= nil then
wiki_link = "<big>•</big> [" .. sitelink .. result_GOID .. " " .. label .."]<br>"
else
wiki_link = "<big>•</big> [[:d:Q" .. v.mainsnak.datavalue.value["numeric-id"] .. "|" .. label .. "]]<br>"
end
overall_results[#overall_results+1] = wiki_link
</syntaxhighlight>
:Yes, that code should be converted to a normal list format. See [[Help:List]]. I have attempted conversion in the module's sandbox and tried it at [[Calreticulin]]. It appears to work fine. [https://en.wikipedia.org/w/index.php?title=Special%3AComparePages&page1=Module%3AInfobox+gene&page2=Module%3AInfobox+gene%2Fsandbox Here's the difference between the live code and the sandbox]. If someone familiar with Lua could check my work, that would be helpful. I tried using the testcases page, but I could not get it to produce anything for me. – [[User:Jonesey95|Jonesey95]] ([[User talk:Jonesey95|talk]]) 01:28, 5 April 2021 (UTC)
:: This modification was taken into account when deploying the changes requested in section "Template-protected edit request on 13 October 2021", {{to|Paine Ellsworth}} you may set this issue as done too :), thanks. [[User:Tarsmf|Tarsmf]] ([[User talk:Tarsmf|talk]]) 10:30, 19 November 2021 (UTC)
:{{to|Hairy Dude}} {{done|done}} per above notification by editor {{u|Tarsmf}}. '''''[[User:Paine Ellsworth|<span style="font-size:92%;color:darkblue;font-family:Segoe Script">P.I.&nbsp;Ellsworth</span>]]'''''&numsp;-&nbsp;[[Editor|<span style="color:black">ed.</span>]]&nbsp;[[User talk:Paine Ellsworth|<sup>put'r&nbsp;there</sup>]]&nbsp;<small>02:40, 22 November 2021 (UTC)</small>

== Required modules ==

Does this module really
<syntaxhighlight lang="lua">
local navbar = require('Module:Navbar')._navbar
local infobox = require('Module:Infobox3cols').infobox
local infoboxImage = require('Module:InfoboxImage').InfoboxImage
</syntaxhighlight>
I do not see anywhere in the module where these local variables are used using Cntl + F. Am I blind? [[User:Izno|Izno]] ([[User talk:Izno|talk]]) 05:25, 1 November 2021 (UTC)
::I agree with you they are not necessary, the gene infobox works fine without them as done in the sandbox in my opinion too. [[User:Tarsmf|Tarsmf]] ([[User talk:Tarsmf|talk]]) 10:36, 19 November 2021 (UTC)
:::{{Done}} —[[User:Uzume|Uzume]] ([[User talk:Uzume|talk]]) 01:49, 1 December 2021 (UTC)

== Template-protected edit request on 3 February 2022 ==

{{edit template-protected|Module:Infobox gene|answered=yes}}
Please remove the <nowiki>' '</nowiki> in "<nowiki>local disease, dis_ref = ' '</nowiki>". This is how every article using this template shows up as having "Lint Error: Missing End Tag". [[User:Chicdat|🐔dat]] ([[User talk:Chicdat|talk]]) 12:19, 3 February 2022 (UTC)
:Please link to an article with this problem. [[Catalase]] does not have this error, even though it uses this module. Thanks. – [[User:Jonesey95|Jonesey95]] ([[User talk:Jonesey95|talk]]) 15:22, 3 February 2022 (UTC)
::[[PPP2R3A]]. [[User:Chicdat|🐔dat]] ([[User talk:Chicdat|talk]]) 11:00, 4 February 2022 (UTC)
:::I see no issues? What system/browser are you on? [[User:Casualdejekyll|<span style="color:#FF69B4">casualdejekyll</span>]] 12:42, 4 February 2022 (UTC)
::::I'm not a Template Editor, but what concerns me about this request is that the edit you are suggesting does not close a tag??? [[User:Casualdejekyll|<span style="color:#FF69B4">casualdejekyll</span>]] 16:34, 4 February 2022 (UTC)
:::::The problem with [[PPP2R3A]] is that in the "Aliases" field, the value that is fetched from the Wikidata "Also known as" field includes <code><nowiki>subunit B''alpha</nowiki></code>, which by including two straight apostrophe characters, initiates italic formatting incorrectly. I am deactivating this edit request until it can be demonstrated that the module itself is causing problems. – [[User:Jonesey95|Jonesey95]] ([[User talk:Jonesey95|talk]]) 18:42, 4 February 2022 (UTC)
::::::{{to|Jonesey95|Chicdat|s2=🐔dat|Casualdejekyll|s3=casualdejekyll}} the incorrect B''alpha'' in the Aliases section came from the [[:d:Q18030723|"Also known as"]] section on the Wikidata page (uppermost box). I changed the {{code|<nowiki>''</nowiki>}} to a quotation mark ("), and the linter error disappeared. So I don't think the problem is in the infobox modules, it's in the Wikidata alias descriptions. When you come across the "B double prime" shown as a B followed by two single apostrophes, those single apostrophes must be changed either to a quotation mark or the words "double prime". '''''[[User:Paine Ellsworth|<span style="font-size:92%;color:darkblue;font-family:Segoe Script">P.I.&nbsp;Ellsworth</span>]]'''''&numsp;-&nbsp;[[Editor|<span style="color:black">ed.</span>]]&nbsp;[[User talk:Paine Ellsworth|<sup>put'r&nbsp;there</sup>]]&nbsp;<small>21:49, 10 February 2022 (UTC)</small>
:::::::I considered a workaround like that, but I did not think it was valid, given that sources (e.g. [https://pubmed.ncbi.nlm.nih.gov/34322200/ this article], [https://pubmed.ncbi.nlm.nih.gov/17535922/ this article], [https://www.ncbi.nlm.nih.gov/gene?Db=gene&Cmd=ShowDetailView&TermToSearch=5523 Entrez Gene], and more) consistently refer to the gene as "<code>B'<nowiki/>'</code>", not "<code>B"</code>". Changing the two apostrophes to a single quotation mark appears to be changing the name to something that is not accurate. I wonder if the module could be persuaded to wrap the aliases in some sort of nowiki tags. I don't know enough Lua to make it happen. – [[User:Jonesey95|Jonesey95]] ([[User talk:Jonesey95|talk]]) 22:08, 10 February 2022 (UTC)
::::::::Pinging {{U|Uzume}} and {{U|Artoria2e5}}, who have made substantive edits to this module and seem to know what they are doing. I think we want to wrap most of the output of "aliases" in nowiki formatting of some sort (excluding the initial word, which is linked with 'symbol_url', I think). You can see my attempt in the edit history; it didn't work, displaying {{tag|nowiki}} in plain text. – [[User:Jonesey95|Jonesey95]] ([[User talk:Jonesey95|talk]]) 22:23, 10 February 2022 (UTC)
:::::::::I've self-reverted the other two gene pages on Wikidata that I "fixed" in case we come up with a better way to handle this. Sorry to have to reintroduce the Linter errors, but I did fix several other unrelated ones at [[mw:Special:LintErrors]] '''''[[User:Paine Ellsworth|<span style="font-size:92%;color:darkblue;font-family:Segoe Script">P.I.&nbsp;Ellsworth</span>]]'''''&numsp;-&nbsp;[[Editor|<span style="color:black">ed.</span>]]&nbsp;[[User talk:Paine Ellsworth|<sup>put'r&nbsp;there</sup>]]&nbsp;<small>22:48, 10 February 2022 (UTC)</small>
::::::::::If this double prime thing is really a double prime thing, instead of using double quotes or doubled apostrophes, shouldn't you be using a double [[Prime (symbol)|prime symbol]]?  Both single and double prime symbols are available in the charinsert under the edit window here; don't know it they are as easily available at wikidata.
::::::::::—[[User:Trappist the monk|Trappist the monk]] ([[User talk:Trappist the monk|talk]]) 15:45, 17 February 2022 (UTC)
:::::::::::I was hoping that possibility would bear fruit, but if you take a moment to click on the multiple sources that I linked above, you will see that they all use a straight apostrophe character, not a prime character. We have to follow the sources. – [[User:Jonesey95|Jonesey95]] ([[User talk:Jonesey95|talk]]) 15:52, 17 February 2022 (UTC)
::::::::{{re|Jonesey95|Paine Ellsworth|Chicdat}} I think you had the right idea with [[Special:Diff/1071107938/prev|1071107938]], however, that is not how you implement that. To use a parser tag you need to use something more like: <code>[[mw:Extension:Scribunto/Lua reference manual#frame:extensionTag|frame:extensionTag]]</code>. That said, there might be another similar if slightly different way to accomplish things with a similar result: <code>[[mw:Extension:Scribunto/Lua reference manual#mw.text.nowiki|mw.text.nowiki]]</code>. Raw Wikidata should not be trusted to be used verbatim in wikitext output (for these sorts of reasons). I think I have fixed this problem with [[Special:Diff/1071136191/prev|1071136191]]. Thanks for the ping. —[[User:Uzume|Uzume]] ([[User talk:Uzume|talk]]) 02:10, 11 February 2022 (UTC)
:::::::::Looks great. Thanks for knowing the right incantation. – [[User:Jonesey95|Jonesey95]] ([[User talk:Jonesey95|talk]]) 02:30, 11 February 2022 (UTC)
::::::::::{{re|Jonesey95}} There are many issues with how this Scribunto module is written and I have often thought of trying to clean it up but it is a lot of work and affects many articles. If we really need a parser tag like <code>nowiki</code>, that can also be done using the aforementioned function to box its input into an appropriate strip marker that can be added to the output so the parser will yield the boxed value verbatim. —[[User:Uzume|Uzume]] ([[User talk:Uzume|talk]]) 02:42, 11 February 2022 (UTC)