Closed kltm closed 1 year ago
PR to come later today: apply https://github.com/geneontology/reactome-go-cams/pull/9 to noctua-models/master
(drop-n-reload of Reactome models). @kltm Would this be OK for Thursday?
Thanks @dustine32 !!!!
Reactome models PR is in! https://github.com/geneontology/noctua-models/pull/247
@ukemi No prob, and thank you for nudging me - I meant to get this in place last week!
@dustine32 I have no problem with this and am adding it to the list, but will leave overriding either way to @vanaukenk (who may be out, so we'll treat merging as default @suzialeksander ).
Confirming that @vanaukenk has left me to do the reviewing. As long as it's a basic "do the new Reactome models exist and do they look reasonably complete" I can definitely review this as part of the regular outage tasks.
@suzialeksander. The models are currently loaded into dev if you need a spot-check comparison. They all have Reactome in the title. For some reason there are still issues searching by group. Hopefully we can track that down by the next load.
@ukemi I think I just figured out the group thing and explained it further in https://github.com/geneontology/noctua-models/pull/247#issuecomment-1307747993.
@suzialeksander Diff ready: http://skyhook.berkeleybop.org/noctua-models-migrations/reports/ontology-replaced-by.diff
Merge in https://github.com/geneontology/minerva/pull/505Tests not passingmaster
replaced_by
term update on blazegraph (@vanaukenk)Update NEOcanceled due to apparent timeout failure; machine too busy? Not too bad as we updated out of sync last week