You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository has been archived by the owner on Apr 25, 2023. It is now read-only.
Our current use mode is to call specific wrapper functions (run_preproc, run_segmentation, etc..) and pass file paths to data locations. There are a few problems with how we've implemented this that i'd like to document here, and address soon:
The wrapper functions have hardcoded paths to data.
The wrapper functions have multiple operations that are never run simultaneously and that are selected by commenting/uncommenting code. (example: run_patch.py Worker can both "extract_patches" and "build_trajectories")
Some critical batches of code are standalone scripts or reference files that are generated outside of this codebase (example: NNSegmentation.run.py is a script that calls this file -> Annotations_8Sites.pkl, which is not written anywhere within the codebase)
potential solutions:
wrapper functions could have main with CLI parsing.
wrapper functions need to be rewritten to accept kwargs for the specific type of run. This works well with CLI
The two files that do this are both for NN training -- the Unet and the VQ-VAE.
Additionally, I think it could be generally useful to have all functions that write critical files also return that data. This will make a full script "pipeline" feasible.
The text was updated successfully, but these errors were encountered:
Sign up for freeto subscribe to this conversation on GitHub.
Already have an account?
Sign in.
@miaecle
Please see recent changes to readme.
Our current use mode is to call specific wrapper functions (run_preproc, run_segmentation, etc..) and pass file paths to data locations. There are a few problems with how we've implemented this that i'd like to document here, and address soon:
potential solutions:
Additionally, I think it could be generally useful to have all functions that write critical files also return that data. This will make a full script "pipeline" feasible.
The text was updated successfully, but these errors were encountered: