-
Notifications
You must be signed in to change notification settings - Fork 132
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
Switch to mamba by default. #2137
Conversation
Job Mingw Test on 5ef2527 : invalidated by @joshua-cogliati-inl Failed tests/framework/ROM/MSR/parallel |
Job Test qsubs sawtooth on 5ef2527 : invalidated by @joshua-cogliati-inl Failed tests/cluster_tests/InternalParallel/test_hybrid_model_code |
@joshua-cogliati-inl Can you also fix the issue when multiple mambas present in conda environment? As you suggested: |
@wangcj05 I pushed that change, can you test it? Thanks |
Job Test qsubs sawtooth on bbea84b : invalidated by @joshua-cogliati-inl failed in set python environment |
@joshua-cogliati-inl It is complains about python, Here is what I got:
|
…nment Basically, the mamba environment might not have a working python, so figure out dependencies install mamba install rest of dependencies.
Job Mingw Test on 0322a6c : invalidated by @joshua-cogliati-inl failed in fetch |
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.
This PR is good.
@joshua-cogliati-inl Can we make some test machine(s) still use conda to install libraries? I think we should have a way to test it.
I will try and get that setup tomorrow. |
PR changes are good, checklist is satisfied. |
Pull Request Description
What issue does this change request address?
See #1806
What are the significant changes in functionality due to this change request?
Trying to see what happens if mamba is used by default.
For Change Control Board: Change Request Review
The following review must be completed by an authorized member of the Change Control Board.
<internalParallel>
to True.raven/tests/framework/user_guide
andraven/docs/workshop
) have been changed, the associated documentation must be reviewed and assured the text matches the example.