geneontology / go-annotation

This repository hosts the tracker for issues pertaining to GO annotations.
BSD 3-Clause "New" or "Revised" License
35 stars 10 forks source link

Review annotations to 'P-type divalent copper transporter activity' #3589

Closed pgaudet closed 4 months ago

pgaudet commented 3 years ago

Hello, I created the term 'GO:0140581 P-type monovalent copper transporter activity' for https://github.com/geneontology/go-ontology/issues/20699 and it looks like existing annotations to 'GO:0043682 P-type divalent copper transporter activity' are incorrect.

I created a spreadsheet to review the annotations: https://docs.google.com/spreadsheets/d/1EKIvjzUPQwgCTk3F4LJdjK06hEGNYIp_4a_BtO9T0XU/edit#gid=0

There are 4 MGI and 1 CGD annotations to review.

Thanks, Pascale

pgaudet commented 3 years ago

Annotations are here: https://docs.google.com/spreadsheets/d/1EKIvjzUPQwgCTk3F4LJdjK06hEGNYIp_4a_BtO9T0XU/edit#gid=0

Please note that I just created the term a few minutes ago; the term will be available with some delay, depending on release cycles.

hdrabkin commented 3 years ago

Why are they wrong? Are you implying that they should be monovalent Cu? the genes annotated are named 'Cu++ transporting' ; I haven't looked at the refs yet because until I get the replacement term if needed, I won't actually do anything.

pgaudet commented 3 years ago

@hdrabkin this is what I wrote in the comment inthe spreadsheet: "Described in Uniprot as an Cu1 transporter; also change gene product in MGI ? "

hdrabkin commented 3 years ago

This make take a while if this means a nomen event.

pgaudet commented 3 years ago

is 'nomen' = nomenclature ? I dont see how that affects the GO annotations.

hdrabkin commented 3 years ago

The suggestion above was that the gene is mis-named , because the official gene name for the first one is   | Atp7a | ATPase, Cu++ transporting, alpha polypeptide implying divalent copper So, our nomen group needs to be advised if this is really not a divalent copper transporter.

hdrabkin commented 3 years ago

Same for Atp7b

hdrabkin commented 2 years ago

THe papers used for assays in these annotations are using CuCl2 (which is Cu++, NOT Cu+_ We will also check with HGNC, who we co-ordinate with

hdrabkin commented 2 years ago

From nomen: . HGNC have renamed their ATPases with the word copper replacing Cu++. I will be happy to update ours to agree. It should be public in about two weeks. This still does not explain the papers using CuCl2 for their assays. I wonder if actually it can do both.

suzialeksander commented 4 months ago

@hdrabkin did you ever resolve this? looks like the annotations still exist.

deustp01 commented 4 months ago

@LiNiMGI Do you know the answer to this, or can you check in the appropriate place in MGI? (Or have I misunderstood and someone else inherited this problem from Harold?)

LiNiMGI commented 4 months ago

@deustp01 Thanks, i will take a look. Li

LiNiMGI commented 4 months ago

Assays used in both papers for these MGI annotations are using CuCl2. Also a review article PMID:32333855 says Atp7a is a Cu2 transporter, so it might actually can do both. I will keep the MGI annotation and close this ticket for now. Thanks, Li