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

Update README.md #123

Merged
merged 2 commits into from
Feb 21, 2023
Merged

Update README.md #123

merged 2 commits into from
Feb 21, 2023

Conversation

mbruns91
Copy link

Revised the text; updated the docker run commands

Revised the text; updated the `docker run` commands
@mbruns91
Copy link
Author

mbruns91 commented Feb 15, 2023

I really don't understand the test failing.
I only came across this kind of error

ImportError: cannot import name 'check_allowlist' from 'conda.core.index' (/opt/conda/lib/python3.9/site-packages/conda/core/index.py)

in the context of circular dependencies. I also can reproduce this in my local docker env. I started to play around with the package versions in the environment.yml for the pyiron/base and pyiron/md images, but haven't yet figured out, where the problem originates. And I'm not even sure that I'll solve it this way. Any suggestions @jan-janssen ?

Fix the version of libmamba (#124)
Copy link
Member

@jan-janssen jan-janssen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me

@jan-janssen
Copy link
Member

The trick is basically comparing the last working build with the current build. As the last build was not that long ago I could look both up here on the GitHub actions. Alternatively there is always the option to pull an old image and then check where it might be incompatible. Finally, another hint was the error message being related to conda so then as I did not see any changes regarding conda, mamba was the next related change.

@mbruns91 mbruns91 merged commit 5a7e3a7 into main Feb 21, 2023
@delete-merged-branch delete-merged-branch bot deleted the mbruns91-update_readme branch February 21, 2023 06:28
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