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

Feature/setup pseudos #25

Merged
merged 3 commits into from
Nov 27, 2020
Merged

Feature/setup pseudos #25

merged 3 commits into from
Nov 27, 2020

Conversation

csadorf
Copy link
Member

@csadorf csadorf commented Nov 27, 2020

Optional step to install pseudos when necessary.

@csadorf csadorf marked this pull request as draft November 27, 2020 16:18
@csadorf csadorf removed the request for review from yakutovicha November 27, 2020 16:18
@csadorf
Copy link
Member Author

csadorf commented Nov 27, 2020

Discovered an issue, will re-request review after it is resolved.

@csadorf csadorf marked this pull request as ready for review November 27, 2020 17:31
Copy link
Member

@yakutovicha yakutovicha left a comment

Choose a reason for hiding this comment

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

thanks @csadorf!

@csadorf csadorf merged commit 5fc3277 into develop Nov 27, 2020
@csadorf csadorf deleted the feature/setup-pseudos branch November 27, 2020 18:19
csadorf added a commit to csadorf/aiidalab-qe that referenced this pull request Jan 21, 2021
* Add optional installation step for required pseudo potential families.

* Show warning icon in pseudopotentials config tab.

When required pseudos are not installed.
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.

None yet

2 participants