cbrain icon indicating copy to clipboard operation
cbrain copied to clipboard
verified publisher icon aces

Boutiques parameter values Fixer module

Open MontrealSergiy opened this issue 3 years ago • 2 comments

addresses issue #1188 It did not fit in 20 lines due to some basic handling of constraints/dependencies.

If must I can do stripped down version, yet I feel there could be constraints/dependencies among preset params even in case of the resource control, for example options to use min memory possibly and memory below threshold could be mutually exclusive.

test file with dependencies could be

https://github.com/MontrealSergiy/cbrain-plugins-test/blob/main/boutiques_descriptors/dependencyhell.json

MontrealSergiy avatar Aug 01 '22 15:08 MontrealSergiy

You said, it fix the issue 1254:

addresses issue https://github.com/aces/cbrain/pull/1254 ,

But you link to the pull request can you point to the issue.

natacha-beck avatar Aug 05 '22 11:08 natacha-beck

But you link to the pull request can you point to the issue.

done

MontrealSergiy avatar Aug 09 '22 14:08 MontrealSergiy

not sure is it ok to rebase on master (most recent) or dev

MontrealSergiy avatar Jan 19 '24 23:01 MontrealSergiy

The new version seems OK, I will test it and if it works I will merge it

prioux avatar Jan 25 '24 17:01 prioux