-
Notifications
You must be signed in to change notification settings - Fork 37
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
👌 IMPROVE: Use the aiida-pseudo
package for installing pseudos
#215
Comments
Not at present. This relates to marvel-nccr/ansible-role-aiida#35 |
It currently is working, see master branch: https://github.com/aiidateam/aiida-pseudo/commits/master |
aiida-pseudo
package for loading a new pseudo family?aiida-pseudo
package for loading a new pseudo family in next tutorial
I believe the answer is no then for the coming tutorial, so I've changed the name of this issue as a reminder for the next tutorial |
Would people agree to replace these 4 commands with a simple import?
Perhaps this section 2.2.4 can also be deleted entirely and the pseudopotential import moved to a place where it is needed (not sure people need to see this before starting to run their first calculation). |
That will import the pseudos, but that won't work with the rest of the tutorial. The tutorial relies on a group of type |
Ok, thanks for the clarification. |
Indeed, very important to check this. @ramirezfranciscof I think this will be done first in the "Running calculations" section? |
aiida-pseudo
package for loading a new pseudo family in next tutorialaiida-pseudo
package for installing pseudos
pseudo setup via |
This would avoid to run these lines (see #214):
@sphuber is the package ready? What would be the equivalent commands? @chrisjsewell is it already in the quantum mobile?
The text was updated successfully, but these errors were encountered: