dattalab / moseq2-extract

Here it comes...
Other
3 stars 4 forks source link

fix: flags gets written to config.yaml and users can't easily change it #137

Closed versey-sherry closed 3 years ago

versey-sherry commented 3 years ago

Issue being fixed or feature implemented

The new CLI writes all the parameters to config.yaml to keep track of all the parameters run. However, if the users have specified a flag, and the flag will be set to True in the config.yaml and users may not be aware of that when using the config.yaml later.

What was done?

Instead of writing all the parameters used in the step to config.yaml, only non-flag parameters get written to the config.yaml. Flags specified in config.yaml won't get updated.

How Has This Been Tested?

CI and locally

Breaking Changes

NA

Checklist: