Explicitly pin neuron version when mixing gcc and intel compilers #1791
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* In #1777 we have made coreneuron optional for upcoming neuron > 8 releases: ``` depends_on("coreneuron+legacy-unit~caliper", when="@:8.99+coreneuron+legacy-unit~caliper") ... ``` * Because of this change, spack is now preferring develop version of neuron when we mix gcc with intel: ``` $ spack spec -I py-bluepyopt%gcc ^neuron%intel Input spec -------------------------------- - py-bluepyopt%gcc - ^neuron%intel Concretized -------------------------------- - [email protected]%[email protected]+neuron patches=dc37639246b3a3320c840f1c4f635940b8510a896a667d4707c3d355d9c6fb81 arch=linux-rhel7-skylake - ^neuron@develop%[email protected]+binary~caliper~codegenopt+coreneuron~debug~gpu~interviews~ipo+legacy-fr+legacy-unit+memacs+mod-compatibility+mpi~nmodl~openmp+python+report+rx3d+shared~sympy~sympyopt~tests~unified build_type=RelWithDebInfo model_tests=None patches=708cb04826b394a858069d93e8c08e1e81e914c23e1ef3da0486e8233834ff6c sanitizers=None arch=linux-rhel7-skylake ``` but with single compiler, it uses last release correctly ``` $ spack spec -I py-bluepyopt ^neuron%intel Input spec -------------------------------- - py-bluepyopt - ^neuron%intel Concretized -------------------------------- - [email protected]%[email protected]+neuron patches=dc37639246b3a3320c840f1c4f635940b8510a896a667d4707c3d355d9c6fb81 arch=linux-rhel7-skylake - ^[email protected]%[email protected]+binary~caliper~codegenopt+coreneuron~debug~gpu~interviews~ipo+legacy-fr+legacy-unit+memacs+mod-compatibility+mpi~nmodl~openmp+python+report+rx3d+shared~sympy~sympyopt~tests~unified build_type=RelWithDebInfo model_tests=None patches=708cb04826b394a858069d93e8c08e1e81e914c23e1ef3da0486e8233834ff6c sanitizers=None arch=linux-rhel7-skylake - ^[email protected]%[email protected] arch=linux-rhel7-skylake ``` * I tried various spec changes in neuron and coreneuron but don't see any satisfactory change to force spack to use last release than develop. So as part of this commit, pin the last release whenever we are mixing compilers. ANYWAY, this will be removed with #1781.
|
Thank you for your pull request! Should you want to clear the PR build directory after failures, please use this pipeline. Before running the cleanup pipeline, please ensure that any PR building pipelines have been cancelled or finished running. |
|
To test your PR, use the following on BlueBrain5: unset MODULEPATH
. /gpfs/bbp.cscs.ch/ssd/apps/bsd/pulls/1791/config/modules.sh
module load unstablePlease test the following updated modules:
|
As an example of this, a simplified coreneuron recipe without any dependency (commented out all) gives: and then I added back just |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
but with single compiler, it uses last release correctly
ninjaalso changes spack preference. (related to python dependency)neuron@develop. I will introduce change in neuron recipe to test the same.