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

image update #106

Closed
wants to merge 10 commits into from
Closed

image update #106

wants to merge 10 commits into from

Conversation

mbruns91
Copy link
Contributor

@mbruns91 mbruns91 commented Feb 2, 2023

updated basemimage in ROOT_CONTAINER in docker-stacks/base and environment.yml for all images.

@@ -2,5 +2,7 @@ channels:
- conda-forge
dependencies:
- ipywidgets
- pyiron_base =0.5.7
- seaborn =0.11.2
- pyiron_base =0.5.31
Copy link
Member

Choose a reason for hiding this comment

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

Can we already use pyiron_base=0.5.32 that should fix the SQLalchemy issues? Or we temporarily enforce sqlalchemy<2.0.0. pyiron/pyiron_atomistics#949

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yeah, sure. I edited this file before the sqlalchemy issue arised and forgot to check again... My bad, sorry.

Copy link
Member

Choose a reason for hiding this comment

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

If you also need pyiron_atomistics then please add sqlalchemy<2.0.0. pyiron_base is currently backwards compatible conda-forge/pyiron_base-feedstock#147

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I'll do that 👍

Comment on lines +9 to +10
- damask
- openssh
Copy link
Member

Choose a reason for hiding this comment

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

The pyiron_base environment should be minimal. So it should not contain specific tools like damask

@mbruns91 mbruns91 closed this Feb 6, 2023
@jan-janssen
Copy link
Member

@mbruns91 Can you test the current image with the new DAMASK version? Just to make sure everything is prepared for Wednesday.

@mbruns91
Copy link
Contributor Author

mbruns91 commented Feb 6, 2023

We just tested it on a similar server as will be used on Wednesday, everything seemed to be working fine 👍

@jan-janssen jan-janssen deleted the update_images branch February 22, 2023 03:39
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