search

loinc / comp-loinc

Computational LOINC in OWL.
MIT License
13 stars 4 forks source link

Alt output dir if `--fast-run` #76

Closed joeflack4 closed 1 month ago

joeflack4 commented 2 months ago

Overview

It looks like the output dir is the same whether --fast-run is on or off. This can result in accidentally overwriting recent, still valid build files with these smaller ones.

ShahimEssaid commented 2 months ago

Yes, appending a "-fast" director suffix would be a good idea. Also, the fast option and its use is very hacky at this point to work around the very slow parts. We can expand its use along with better logging to give faster feedback. It's also limited to the creation of the entities and should probably stay that way. It won't be useful to add this to the graph loading since needed information will be missing with an incomplete load. But, that part doesn't take too long anyway.

Sent from mobile

On Tue, Sep 17, 2024, 2:20 PM Joe Flack @.***> wrote:

Overview

It looks like the output dir is the same whether --fast-run is on or off. This can result in accidentally overwriting recent, still valid build files with these smaller ones.

— Reply to this email directly, view it on GitHub https://github.com/loinc/comp-loinc/issues/76, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABELXULCD6SW7YCUHR5WAVLZXCMJFAVCNFSM6AAAAABOMKLC6SVHI2DSMVQWIX3LMV43ASLTON2WKOZSGUZTEMJYGA2DOMY . You are receiving this because you are subscribed to this thread.Message ID: @.***>