-
Notifications
You must be signed in to change notification settings - Fork 14
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
Neurodamus-models to depend on neurodamus-py (bringing core) #1406
Conversation
Keeping old function version_from_model_core_dep to enable building latest stable deployed version depending still on neurodamus-core. Changed recipes to call the function once per version for readability
14a22c2
to
544633f
Compare
1455264
to
d7c20e3
Compare
var/spack/repos/builtin/packages/neurodamus-neocortex/package.py
Outdated
Show resolved
Hide resolved
var/spack/repos/builtin/packages/neurodamus-hippocampus/package.py
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Conflicts: bluebrain/repo-bluebrain/packages/neurodamus-hippocampus/package.py bluebrain/repo-bluebrain/packages/neurodamus-mousify/package.py bluebrain/repo-bluebrain/packages/neurodamus-neocortex/package.py bluebrain/repo-bluebrain/packages/neurodamus-thalamus/package.py deploy/environments/libraries.yaml var/spack/repos/builtin/packages/py-charm4py/package.py
70ac82c
to
015a737
Compare
To test your PR, use the following on BlueBrain5: unset MODULEPATH
. /gpfs/bbp.cscs.ch/ssd/apps/bsd/pulls/1406/config/modules.sh
module load unstable Please test the following updated modules:
|
fd65cc7
to
160c927
Compare
To test your PR, use the following on BlueBrain5: unset MODULEPATH
. /gpfs/bbp.cscs.ch/ssd/apps/bsd/pulls/1406/config/modules.sh
module load unstable Please test the following updated modules:
|
To test your PR, use the following on BlueBrain5: unset MODULEPATH
. /gpfs/bbp.cscs.ch/ssd/apps/bsd/pulls/1406/config/modules.sh
module load unstable Please test the following updated modules:
|
To test your PR, use the following on BlueBrain5: unset MODULEPATH
. /gpfs/bbp.cscs.ch/ssd/apps/bsd/pulls/1406/config/modules.sh
module load unstable Please test the following updated modules:
|
To test your PR, use the following on BlueBrain5: unset MODULEPATH
. /gpfs/bbp.cscs.ch/ssd/apps/bsd/pulls/1406/config/modules.sh
module load unstable Please test the following updated modules:
|
To test your PR, use the following on BlueBrain5: unset MODULEPATH
. /gpfs/bbp.cscs.ch/ssd/apps/bsd/pulls/1406/config/modules.sh
module load unstable Please test the following updated modules:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks reasonable. I'd prefer if the structure were simpler still, but we can leave that for future work.
To test your PR, use the following on BlueBrain5: unset MODULEPATH
. /gpfs/bbp.cscs.ch/ssd/apps/bsd/pulls/1406/config/modules.sh
module load unstable Please test the following updated modules:
|
Summary
As part of the repo reorganization (2021), neurodamus-core becomes officially an addendum to neurodamus-py
This PR changes the way neurodamus is built, and ultimately loaded, by setting neurodamus-py as a direct dependency of the models.
Context
Neurodamus-core has became a submodule of neurodamus-py to support the long term goal of dropping it if favor of pure Python. Consequently it can/must no longer be considered a main package.
We therefore make neurodamus-models to depend directly on neurodamus-py, since it brings it all, which streamlines the process of building, deploying and even loading.
The end user will be able to simply load e.g.
neurodamus-neocortex
and have a fully functional environment with guaranteed cross-compatibility of the neocortex model, core and (Edit: Soon) neurodamus-py.Edit: We disabled auto-loading neurodamus-py since it will bring the intel stack, which is not optimal because we may need to use other stacks (bglig-py...). These issues should be addresses in a dedicated ticket.
Extra
Bumped neurodamus-py to 2.10.0, bringing