Repository navigation
Rework and generalize save_script_parameters - #156
rohangirishrao wants to merge 7 commits into
Conversation
|
Updated imcf-fiji-mocks for the new version with |
This is now done with the 0.15.0 release, plus the required changes have been integrated into @rohangirishrao Could you please rebase your PR-branch onto current |
The parameters file is now timestamped and has the name of the script as a prefix. An additional overwrite option is also present, so that the same file can be overwritten while testing.
Destination is now optimal, so the test has to reflect this
ae6576b to
1f53a66
Compare
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## devel #156 +/- ##
====================================
Coverage 33% 33%
====================================
Files 25 25
Lines 1806 1819 +13
====================================
+ Hits 591 604 +13
Misses 1215 1215 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
Follow-up to issue #140 and issue #145
✨ Changed
destinationnow defaults to" ", and saving is skipped and logged when it's empty<script_name>_run-<YYYY-MM-DD--HH-MM>_params.txtoverwrite(default:False): whenTrue, it writes to a fixed<script_name>_params.txtinstead, and does not create new files on each run.💥 Breaking
save_file_namehas been removed, so calls that pass that will break!