-
Notifications
You must be signed in to change notification settings - Fork 8
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Global date change when using the graph editor (x-hint-layout, etc.) #533
Comments
I should also point out that this doesn't always happen. Yesterday when we were looking at a more 'conventional' GO-CAM model and performed a similar task, not all the dates displayed in the form changed. I'm not sure how to repeat that behavior. |
As a simple quick cross-check, it might be worth checking the behavior in the graph editor, as that it closest the underlying communication with minerva. |
@kltm, this behavior was seen when we were using the graph editor. In some cases when things aren't available for editing in the form (CC annotations), curators are forced to use the graph. |
Hm. Very odd that it can't be done systematically. It does suggest the issue is with minerva however, as there is likely little code overlap with the different front ends that could this. |
If you look at the GPAD export, the dates are still This CC annotation is in the GPAD: MGI MGI:1096381 located_in GO:0016604 PMID:18644859 ECO:0000314 20230713 MGI contributor=https://orcid.org/0000-0001-7476-6306|noctua-model-id=gomodel:6494e2e900001751|model-state=development But the form editor shows it with a |
Maybe @tmushayahama could comment on how the form editor determines the date. Because unfortunately it's a separate implementation from the Minerva GPAD export. |
from my research I am finding some things on owl file are getting new dates even if not related to that edge I am using this algorithm and taking from date annotation Procedure for each evidence inside an activity get its date |
@balhoff After examination, it looks like the underlying issue is that changes to the x-hint and y-hint are triggering updates to contributor, group, and date. Ideally (and we thought this was true), changes to these specific annotations should cause no changes to other metadata. |
Noting here a general testing strategy that we used to de-bug this issue and that we should use as a fundamental testing strategy more often:
|
@LiNiMGI pointed out on the workbenches call that when she is required to move to the graph editor from the form. If she changes a single item in the graph and moves back to the form, all of the dates displayed on the form change.
@balhoff , this might actually be a minerva issue. Please move this ticket as appropriate.
I have set up test models to illustrate this:
sequence alignment evidence used in manual assertion
contributor: https://orcid.org/0000-0001-7476-6306
date: 2023-07-13
providedBy: (http://informatics.jax.org/)
source: Biological clock in total darkness: the Clock/MOP3 circadian system of the blind subterranean mole rat
with: EMBL:AJ318060
Before:
http://model.geneontology.org/6494e2e900001751/6494e2e900001761
http://model.geneontology.org/6494e2e900001751/6494e2e900001761 rdf:type owl:NamedIndividual ,
http://identifiers.org/mgi/MGI:1096381 ;
http://purl.obolibrary.org/obo/RO_0001025 http://model.geneontology.org/6494e2e900001751/6494e2e900001816 ;
http://purl.org/dc/elements/1.1/contributor "https://orcid.org/0000-0001-7476-6306" ;
http://purl.org/dc/elements/1.1/date "2023-07-13"^^xsd:string ;
http://purl.org/pav/providedBy "http://informatics.jax.org" .
http://model.geneontology.org/6494e2e900001751/6494e2e900001762
http://model.geneontology.org/6494e2e900001751/6494e2e900001762 rdf:type owl:NamedIndividual ,
http://purl.obolibrary.org/obo/GO_0006606 ;
http://purl.org/dc/elements/1.1/contributor "https://orcid.org/0000-0001-7476-6306" ;
http://purl.org/dc/elements/1.1/date "2023-07-13"^^xsd:string ;
http://purl.org/pav/providedBy "http://informatics.jax.org" .
http://model.geneontology.org/6494e2e900001751/6494e2e900001814
http://model.geneontology.org/6494e2e900001751/6494e2e900001814 rdf:type owl:NamedIndividual ,
http://purl.obolibrary.org/obo/GO_0008134 ;
http://purl.obolibrary.org/obo/RO_0002333 http://model.geneontology.org/6494e2e900001751/6494e2e900001821 ;
http://purl.org/dc/elements/1.1/contributor "https://orcid.org/0000-0001-7476-6306" ;
http://purl.org/dc/elements/1.1/date "2023-07-13"^^xsd:string ;
http://purl.org/pav/providedBy "http://informatics.jax.org" .
After: There are 24 places where the date has been changed to July 14th, for example:
http://model.geneontology.org/6494e2e900001751/6494e2e900001761
http://model.geneontology.org/6494e2e900001751/6494e2e900001761 rdf:type owl:NamedIndividual ,
http://identifiers.org/mgi/MGI:1096381 ;
http://purl.obolibrary.org/obo/RO_0001025 http://model.geneontology.org/6494e2e900001751/6494e2e900001816 ;
http://geneontology.org/lego/hint/layout/x "1453.125"^^xsd:string ;
http://geneontology.org/lego/hint/layout/y "381.25"^^xsd:string ;
http://purl.org/dc/elements/1.1/contributor "https://orcid.org/0000-0001-7476-6306" ;
http://purl.org/dc/elements/1.1/date "2023-07-14"^^xsd:string ;
http://purl.org/pav/providedBy "http://informatics.jax.org" .
http://model.geneontology.org/6494e2e900001751/6494e2e900001762
http://model.geneontology.org/6494e2e900001751/6494e2e900001762 rdf:type owl:NamedIndividual ,
http://purl.obolibrary.org/obo/GO_0006606 ;
http://geneontology.org/lego/hint/layout/x "5193.75"^^xsd:string ;
http://geneontology.org/lego/hint/layout/y "75"^^xsd:string ;
http://purl.org/dc/elements/1.1/contributor "https://orcid.org/0000-0001-7476-6306" ;
http://purl.org/dc/elements/1.1/date "2023-07-14"^^xsd:string ;
http://purl.org/pav/providedBy "http://informatics.jax.org" .
http://model.geneontology.org/6494e2e900001751/6494e2e900001814
http://model.geneontology.org/6494e2e900001751/6494e2e900001814 rdf:type owl:NamedIndividual ,
http://purl.obolibrary.org/obo/GO_0003712 ;
http://purl.obolibrary.org/obo/RO_0002333 http://model.geneontology.org/6494e2e900001751/6494e2e900001821 ;
http://geneontology.org/lego/hint/layout/x "78.75"^^xsd:string ;
http://geneontology.org/lego/hint/layout/y "296"^^xsd:string ;
http://purl.org/dc/elements/1.1/contributor "https://orcid.org/0000-0001-7476-6306" ;
http://purl.org/dc/elements/1.1/date "2023-07-14"^^xsd:string ;
http://purl.org/pav/providedBy "http://informatics.jax.org" .
The text was updated successfully, but these errors were encountered: