Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Make projections not required #61

Merged
merged 3 commits into from
Dec 18, 2019

Conversation

greschd
Copy link
Member

@greschd greschd commented Dec 3, 2019

Fixes #15.

Reverts back to printing an empty projections block if the input is not given (which will trigger the Wannier90 default), instead of explicitly requiring the input.

Testing is implemented with the features from #57, so this PR should be merged after that.

@greschd greschd requested a review from giovannipizzi December 3, 2019 11:10
@greschd greschd changed the title [BLOCKED] Make projections not required Make projections not required Dec 9, 2019
@greschd greschd requested a review from normarivano December 16, 2019 10:24
@normarivano normarivano merged commit 5f5e5c4 into aiidateam:develop Dec 18, 2019
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

random projections as default
2 participants