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

Minor modifs in environment and .gitignore #22

Merged
merged 2 commits into from
Feb 18, 2022
Merged

Minor modifs in environment and .gitignore #22

merged 2 commits into from
Feb 18, 2022

Conversation

Bultako
Copy link
Member

@Bultako Bultako commented Feb 18, 2022

Adding PYTHONNOUSERSITE=1 prevents the user site packages to take precedence over the packages in the environment.

@adonath
Copy link
Member

adonath commented Feb 18, 2022

Thanks @Bultako, weirdly enough this change leads to a test fail here https://github.com/gammapy/gammapy-v1.0-paper/runs/5246803041?check_suite_focus=true#step:3:1857 Not sure what is happening, maybe showyourwork somewhere relies on the user site packages and we have to add the dependency to the conda env instead?

@adonath
Copy link
Member

adonath commented Feb 18, 2022

Ok, it seems unrelated aws/aws-sam-cli#3661

@adonath adonath merged commit 472b933 into main Feb 18, 2022
@Bultako Bultako deleted the env branch February 21, 2022 14:23
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

Successfully merging this pull request may close these issues.

2 participants