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

split dependencies into optional groups #66

Open
tmchartrand opened this issue Dec 5, 2024 · 2 comments
Open

split dependencies into optional groups #66

tmchartrand opened this issue Dec 5, 2024 · 2 comments

Comments

@tmchartrand
Copy link

I've been working on some code that uses basic functions of this library to parse the ABC atlas manifests etc - essentially only abc_atlas_access.abc_atlas_cache.manifest. I've been installing with --no-deps to avoid bringing in all the dependencies, but now I'm realizing that as I prepare to release this code there's no way for me to specify the abc_atlas_access dependency while overriding its dependencies in turn (short of depending on a fork instead).

I'd suggest at the very least splitting out the dependencies for code in src/ from those for notebooks (simpleITK for instance). My preferred solution would also split out anndata and cloud dependencies that are only needed for some of the core functionality, but I realize that could end up confusing some users.

Thanks, happy to discuss in person, just let me know!

@morriscb
Copy link
Collaborator

Hey @tmchartrand,

Thanks for the suggestion. I've been wanting to reorganize the project file a bit and this is a good enough reason to do so. I'm working on a release that is going out with the aging mouse dataset so I'll see if I can't separate dependencies out for the notebooks and testing.

@morriscb
Copy link
Collaborator

#67

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

2 participants