-
Notifications
You must be signed in to change notification settings - Fork 14
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
Add bc for computing resources required by qeapp image #497
Conversation
Hi @danielhollas, why the test is going to upload to registry? Should I open PR from origin repo? |
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.
I don't know why the test failed. Maybe try restarting it?
I don't understand the question, your PR is opened from origin, so it should be fine to push to ghcr.io
@@ -10,7 +10,8 @@ USER root | |||
# https://pymatgen.org/installation.html#installation-tips-for-optional-libraries | |||
# rsync: needed to support the new AiiDA backup command | |||
# povray: rendering engine used in aiidalab-widgets-base | |||
ENV EXTRA_APT_PACKAGES "curl povray rsync build-essential" | |||
# bc: needed to compute the resources for computer setup |
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.
Is this really needed? We could always use a Python one liner.
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.
I don't want to use python, bc is a tiny thing and all computation just happened in shell script.
I mean not from my forked repo but from |
Ah, sorry, I am on a phone app and it did not show me that this PR is from a fork. Yes, for this repo you always have to open the PRs from the origin. |
Replaced by #499 |
No description provided.