search

monarch-initiative / mondo-ingest

Coordinating the mondo-ingest with external sources
https://monarch-initiative.github.io/mondo-ingest/
6 stars 3 forks source link

DO NOT MERGE: Synonym sync: acronym case exception - build #677

Closed joeflack4 closed 1 week ago

joeflack4 commented 1 month ago

Build for:

Maybe this build isn't entirely necessary, given the simple changes of #671. But we are about done with this round of synonym sync and about to merge it, and it needs a build first. But this feature branch is still open, so I ought to build here instead. We do however have some synonym-related OMIM changes (https://github.com/monarch-initiative/omim/pull/132) still yet to be merged. Not sure if we need to run a build on #93 when those changes are merged.

twhetzel commented 2 weeks ago

@joeflack4 did you check back to see why this run took ~7 hr?

joeflack4 commented 2 weeks ago

@twhetzel Until now, nothing beyond this initial comment.

I did just run the synonym sync Python part for Orphanet just now, since that is the part I noticed that for some reason appeared to be stuck a long time, and it completed within 11 seconds.

I would mark this as a strange anomaly. The only other reason I can think of why it might have taken a long time is possibly due to multiple updates to mondo master as this was running. Each time that happens, it can add another ~20-30 minutes to the build time, as it will generate another mondo.owl and mondo.sssom.tsv.