Skip to content
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

Better encapsulate configuration and paths to analysis components #3

Open
peabody124 opened this issue Mar 24, 2021 · 1 comment
Open

Comments

@peabody124
Copy link
Owner

peabody124 commented Mar 24, 2021

Currently set paths to analysis tools in:

https://github.com/peabody124/PosePipeline/blob/main/pose_pipeline/env.py#L26

And path to data storage in:

https://github.com/peabody124/PosePipeline/blob/main/pose_pipeline/pipeline.py#L19

The later should be set with the other DataJoint configuration settings, rather than the pipeline. The former is a bit trickier, and perhaps should even mimic the DataJoint approach for storage a package-specific configuration file on disk. This should also work well with a Docker version of the pipeline (e.g. #1).

@peabody124
Copy link
Owner Author

8724b0e addresses the storage configuration issue. Still need to handle the environment variables better.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant