-
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
Add pathways widget and GO-CAM landing to AmiGO pages #66
Comments
@kltm Bump start this; contact Patrick |
Some test version can be presented at the GOC meeting |
@vanaukenk @ukemi , @pkalita-lbl has made some very nice progress, which is now visible at: https://amigo-staging.geneontology.io . |
Neato! Thanks @kltm and @pkalita-lbl |
That's right, it is using a slightly older version of the widget. The latest version(s) have technical dependencies that make it incompatible with AmiGO. It's possible we could work around it in AmiGO, but it might be less effort to rework those dependencies in the widget for a future version. |
I just searched on one of my favorite genes Pkm Mus musculus (MGI:MGI:97591). It should at least bring back gomodel:5745387b00001516, but returns zero models. |
Yeah, another big caveat is that anything with an MGI ID won't work right now. The new API that backs this needs a fix so that it can resolve |
@ukemi The |
Beautiful! For now I also see the gene-centric models, but at some point we should not display those. A first step might be to use the same filtering criteria that is used other places to assign 'causal model' status. It's not ideal, but a good first step. |
Chris also pointed that out. I've already made the change, but it's not deployed on |
Talking with @pkalita-lbl yesterday, I believe that no tech doc is needed for the remaining parts of this project. |
|
Rescoped; now also see #78 |
Getting this out live will need to wait on the merged GO API hitting production: #14 |
Just to note, when LP left I was told to create a GOCAM browser landing page as a web component and it was in working but got abandoned https://github.com/geneontology/wc-web-gocam Tagging @kltm @pkalita-lbl |
Hi Pascale, David and I can do this, but we will need input from Seth (cc'ed here) and perhaps other GO software people. On two previous manager's calls (8/30 and 9/13), we noted that finishing the pathways widget project was dependent on some of the API refactor work. Seth, let us know if you have time before Wednesday to touch base about this. Next Monday or Tuesday would work for me; tomorrow is mostly booked. Thanks, |
[Bringing in email thread] [@pgaudet] As mentioned, as far as I'm concerned, this project has no active parts that can proceed without the production deployment of the new GO API. There is no more work here for anybody but myself and Sierra; the remaining work is probably on the order of half an hour. While the new GO API--which is required for any further action--is /technically/ deployed, it has some data issues and may need to be withdrawn (geneontology/go-fastapi#77; GO API project currently, but may more to general bug later today) until Sierra can take a look at it. For this reason, we have not proceeded with updating everything to point at the new pages, etc. As there is no more functional action from Kimberly and David here, it may make the communication loop shorter and more clear if I'm put on as PO, as there are only technical considerations remaining (delta the documentation ticket for you and Suzi geneontology/go-site#1929). As an update to the email thread, it looks like we have the GO API fix in that we wanted, so I'll try wrapping up this project in the next day or so. |
Hi @kltm I changed the PO and assignee to reflect your comment above. |
@pgaudet No more active tickets here; closing out. |
Project link
https://github.com/orgs/geneontology/projects/129
Project description
This project is to scoped to
This is a follow-up to Patrick's previous work here: geneontology/amigo#650
We will work with the POs to make the (minimal) changes to AmiGO necessary to support this (this is not meant to be a major push in AmiGO) using current technology and stack.
PI
Chris
Product owner (PO)
Seth
Technical lead (TL)
Seth
Other personnel (OP)
Patrick, Seth (, Tremayne)
Technical specs
N/A
All remaining work is an extension of what we currently have and review is not necessary at this time.
Other comments
Noting previous geneontology/amigo#237
The text was updated successfully, but these errors were encountered: