-
Notifications
You must be signed in to change notification settings - Fork 5
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
Should probably just include pip dependencies in the conda creation #39
Comments
JimCircadian
added a commit
to JimCircadian/icenet-pipeline
that referenced
this issue
Mar 28, 2024
…asic pip dependencies
Thanks for checking @JimCircadian, it doesn't affect my changes 👋🏼 . |
JimCircadian
added a commit
to JimCircadian/icenet-pipeline
that referenced
this issue
Apr 12, 2024
…asic pip dependencies
bnubald
changed the title
Should probably just include pip dependecies in the conda creation
Should probably just include pip dependencies in the conda creation
Aug 21, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Keep on forgetting to manually install model-ensembler, which isn't an explicit dependency of icenet so needs to be included in the conda environment install. Why not do this to pin icenet too, provided it doesn't interfere with the solver (which is painful enough at the best of times).
Does this link to any changes you've been making recently @bnubald? Going to see if it works well as I'm recreating environments often at the moment across HPCs 😉
The text was updated successfully, but these errors were encountered: