-
Notifications
You must be signed in to change notification settings - Fork 0
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
Links to PRO entities #81
Comments
Weird we should be using PURLs - this is what's in the dbxref file: @kltm is this what is implemented? Are the PURLs not resolved correctly? (this make it a PRO issue, not GO). |
It would be good to see the links in context--I'm not sure where this is occurring. |
The xref file specifies PR_000060258 |
@ValWood Can you give a link to where in Noctua you are seeing this? |
In my model |
Links: Replicate:
The expectation would be https://purl.obolibrary.org/obo/PR_000060258 (go-site metadata), or at least "correct-ish" https://proconsortium.org/cgi-bin/entry_pro?id=PR_000050512 . |
Going to https://amigo.geneontology.org/amigo/gene_product/PomBase:SPBC29A10.14 , then adding direct term filter "topological DNA co-entrapment activity", we get an annotation with isoform PR:000050512, that links to https://purl.obolibrary.org/obo/PR_000050512 . This would seem to indicate that the metadata, at least some places, is healthy. |
Noting that "noctua-visual-pathway-editor/src/@noctua.curie/data/go-context.ts" seems correct.
...unfortunately, I'm having trouble building the local packages for the moment (I may need to ask @pkalita-lbl for help here). It may be worthwhile first to clarify with @tmushayahama where this metadata is coming from: an npm package, go-site, or the contexts file? |
Add draft documentation for how to do npm package releases from the docker images; for https://github.com/geneontology/noctua/issues/879 . Thanks to @pkalita-lbl
I think we need @tmushayahama 's input here before acting. @tmushayahama , can you pinpoint where that information is coming from, in case packages need a refresh or if it's wired in some other way? |
Noting that the incorrect link is also happening for PRO entries on the Noctua form editor. |
ping @tmushayahama , can you pinpoint where that information is coming from, in case packages need a refresh or if it's wired in some other way? |
Links go to
https://proconsortium.org/cgi-bin/pro/entry_pro?id=PR:000050512
should go to
https://proconsortium.org/cgi-bin/entry_pro?id=PR_000050512
The text was updated successfully, but these errors were encountered: