geneontology / minerva

BSD 3-Clause "New" or "Revised" License
6 stars 8 forks source link

optionally preserve evidence in model copy #513

Closed balhoff closed 1 year ago

balhoff commented 1 year ago

See requirements in #512.

balhoff commented 1 year ago

@vanaukenk @kltm I've got a PR for this now at #514. The PR is already merged into the dev branch, so you can update the code on minerva-dev for testing.

balhoff commented 1 year ago

@tmushayahama when calling the model copy operation, provide an argument preserve-evidence with a boolean value of true or false, in the same place you provide the model-id argument.

kltm commented 1 year ago

This should now be "live" on noctua-dev.

vanaukenk commented 1 year ago

Testing by @ukemi @vanaukenk @pmasson55

All looks okay to move to production on the 2023-05-25 maintenance outage

balhoff commented 1 year ago

@vanaukenk thanks, I've gone ahead and merged the PR to the master branch.