Module talk:Infobox gene

Template-protected edit request on 2 March 2023

Error: [778:86] invalid escape sequence near '\:' مهدی بهرامی مطلق (talk) 16:29, 2 March 2023 (UTC)

Not sure where you saw this error, but I believe \: and \; are not required, please check diff — Martin (MSGJ · talk) 21:51, 3 March 2023 (UTC)

That was a massive and very productive edit! I didn't check all the details but it looked good and fixed the reported problem. Johnuniq (talk) 01:10, 4 March 2023 (UTC)

Had my Lua-challenged mind puzzled! Seems that the error is found in the [edit screen]. Click on the red x-box at the bottom of the edit screen to actually see the error. That does not appear in the sandbox's edit screen. P.I. Ellsworth , ed. ^{put'er there} 07:42, 4 March 2023 (UTC)

 Done * Pppery * _{it has begun...} 22:14, 4 March 2023 (UTC)

Human UniProt search error

I am not sure if this is a problem in Infobox gene or Wikidata, but the rendering of the Human UniProt search link in Mucin-1 is not right. Does anyone have any idea how to fix this? Thanks. Boghog (talk) 06:43, 26 April 2023 (UTC)

 Done The problem was in Wikidata. Only one UniProt ID should be associated with the human protein, not two. Fixed in this edit. Boghog (talk) 06:50, 26 April 2023 (UTC)

Edit request 15 January 2024

Modify Bgee links

The Bgee links changed. To avoid redirections, please include "www" in the URLs as shown below. Thank you very much for your help.

Diff:

−

local bgee_gene_page = "https://bgee.org/gene/"

+

local bgee_gene_page = "https://www.bgee.org/gene/"

−

bgee_title = "[https://bgee.org/ " .. bgee_title .. "]"

+

bgee_title = "[https://www.bgee.org/ " .. bgee_title .. "]"

Tarsmf (talk) 16:21, 15 January 2024 (UTC)

 Done. Tested at FOXS1. – Jonesey95 (talk) 00:27, 16 January 2024 (UTC)

Edit request 22 May 2024

Description of suggested change:

Improve External IDs readability. In the 'Identifiers tab, external IDs are difficult to read because there is no semicolon separating them. The code below adds semicolons between IDs. In addition, it also adds another external ID of the OMA database that is complementary to HomoloGene with richer information about homologous genes. It was tested here: Module_talk:Infobox_gene/testcases. Thank you very much for your help!

Diff: (Line numbers below are just an approximation)

Suggestion to add this trim function (to remove empty spaces) in line 23 or in any place on top of the code:

−

+

local function trim(s) return (s:gsub("^%s*(.-)%s*$", "%1")) end

Line 196

−

p.renderIdentifiers(aliases, hgnc_id, gene_symbol, homologene_id, omim_id, mgi_id, ChEMBL_id, IUPHAR_id, ec_no, entrez_gene)

+

p.renderIdentifiers(aliases, hgnc_id, gene_symbol, homologene_id, omim_id, mgi_id, ChEMBL_id, IUPHAR_id, ec_no, entrez_gene, ensembl)

Line 399

−

function p.renderIdentifiers(aliases, hgnc_id, gene_symbol, homologene_id, omim_id, mgi_id, ChEMBL_id, IUPHAR_id, ec_no, entrez_gene)

+

function p.renderIdentifiers(aliases, hgnc_id, gene_symbol, homologene_id, omim_id, mgi_id, ChEMBL_id, IUPHAR_id, ec_no, entrez_gene, ensembl)

Line 433

−

omim = string.gsub(omim, ", $"," ") --remove comma from end

+

omim = trim(string.gsub(omim, ", $","")) --remove comma from end

Line 446

−

homolo = string.gsub(homolo, ", $"," ") --remove comma from end

+

homolo = trim(string.gsub(homolo, ", $","")) --remove comma from end

−

genecards = genecards.."[https://www.genecards.org/cgi-bin/carddisp.pl?gene="..gene_symbol.." "..gene_symbol.."] "

+

genecards = genecards.."[https://www.genecards.org/cgi-bin/carddisp.pl?gene="..gene_symbol.." "..gene_symbol.."]"

Line 463

−

mgi = string.gsub(mgi, ", $"," ")--remove comma from end

+

mgi = trim(string.gsub(mgi, ", $","")) --remove comma from end

−

ChEMBL = "[[ChEMBL]]"..": ".."[https://www.ebi.ac.uk/chembldb/index.php/target/inspect/CHEMBL"..ChEMBL_id.." "..ChEMBL_id.."] "

+

ChEMBL = "[[ChEMBL]]"..": ".."[https://www.ebi.ac.uk/chembldb/index.php/target/inspect/CHEMBL"..ChEMBL_id.." "..ChEMBL_id.."]"

−

IUPHAR = "[[International_Union_of_Basic_and_Clinical_Pharmacology|IUPHAR]]"..": ".."[http://www.guidetopharmacology.org/GRAC/ObjectDisplayForward?objectId="..IUPHAR_id.." "..IUPHAR_id.."] " --**lclz**

+

IUPHAR = "[[International_Union_of_Basic_and_Clinical_Pharmacology|IUPHAR]]"..": ".."[http://www.guidetopharmacology.org/GRAC/ObjectDisplayForward?objectId="..IUPHAR_id.." "..IUPHAR_id.."]" --**lclz**

Line 478

−

local EC = "[https://www.genome.jp/dbget-bin/www_bget?enzyme+" .. link_ec_no .. " " .. ec_no .. "]"

+

local EC = "[https://www.genome.jp/dbget-bin/www_bget?enzyme+" .. link_ec_no .. " " .. ec_no .. "]" local oma = "[[Orthologous_MAtrix|OMA]]:" oma = oma.."[https://omabrowser.org/oma/vps/"..mw.text.split(ensembl,",")[1].." "..gene_symbol.." - orthologs]" external_id_table = {omim, mgi, homolo, ChEMBL, IUPHAR, genecards, oma} external_id_processed_table = {} for i, v in ipairs(external_id_table) do if (v ~= "") then external_id_processed_table[#external_id_processed_table + 1] = v end end external_ids = tostring(table.concat(external_id_processed_table, "; "))

Line 528

−

:wikitext(omim) :wikitext(mgi) :wikitext(homolo) :wikitext(ChEMBL) :wikitext(IUPHAR) :wikitext(genecards)

+

:wikitext(external_ids)

Tarsmf (talk) 11:39, 24 May 2024 (UTC)

@Tarsmf if I may suggest, you could use table.concat(table, "; ") to add the semi-colons is a more efficient manner — Martin (MSGJ · talk) 21:41, 23 May 2024 (UTC)

Thanks a lot @MSGJ for the suggestion! I have implemented your suggestion in the Module:Infobox gene/sandbox and updated my request above, tested here Module talk:Infobox gene/testcases. However, note that we cannot simply add a ';' for each item of the table since some of them are empty strings depending on the gene wikipedia article. Therefore, we still need to filter out the empty string before implementing the concatenation. Tarsmf (talk) 11:40, 24 May 2024 (UTC)

Looks good,  Done — Martin (MSGJ · talk) 12:02, 24 May 2024 (UTC)