-
Notifications
You must be signed in to change notification settings - Fork 15
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ENH: Add method to overwrite / redo some analysis steps #311
Comments
I like the idea of setting up a toggle parameter to greenlight potential overwrites, as it can be very annoying to launch a run only to realize a single change (for me, usually related to HPI processing) will cause an error. I actually do something similar in my own code:
where delete_sssfiles() is a simple program I stashed away in my scoring script. But this just deletes the files outright, forcing the recomputation. |
I think the annotation is completely dependent on the cHPI fitting, so maybe we can get away with just adding a In both cases, |
It may be worth implementing your above idea of recomputing (if the toggle is |
It would be more work, but perhaps such a meta file could be comprehensive across all of the important MNEFun parameters, as well as archivable and human-readable, effectively providing a diary of MNEFun processing for that experiment directory. (I picture a command line tool that could give a nicely formatted readout of the last run or a prior run.) Just a pipeline dream? |
Yes in principle this would work with something like |
At least estimation of:
_maxbad.txt
.pos
-annot.h5
-counts.h5
These steps are slow so currently they are recomputed only when they are missing. This means the way to say "recompute these" is to delete files from disk (not great). We could add an
overwrite
/recompute
parameter to control which of these to recompute. Or maybe make it so the inputs to the computation function are cached properly (joblib
?) so that recomputation is automatic when the relevant parameters change.The text was updated successfully, but these errors were encountered: