-
Notifications
You must be signed in to change notification settings - Fork 135
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
Nexus: support supercell twists in PySCF workflows #5073
base: develop
Are you sure you want to change the base?
Conversation
Ready to proceed. |
could you suggest a reviewer? |
Yes, already flagged @anbenali for this. |
Oh thanks. I missed it. |
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.
Noting that I am currently running the examples locally. These appear to be running OK. PySCF convergence tolerance is high so they take some minutes.
ntest_nexus_pyscf_input failed on me, but I have yet to investigate further
1452: Test name : pyscf_input
1452: Test sublabel : test_write
1452: Test exception: "AssertionError: "
1452: Test backtrace:
1452: File "/home/pk7/projects/qmc/git_QMCPACK_prckent/qmcpack/nexus/bin/nxs-test", line 478, in run
1452: self.operation()
1452: File "/home/pk7/projects/qmc/git_QMCPACK_prckent/qmcpack/nexus/bin/nxs-test", line 1210, in pyscf_input
1452: nunit('write')
1452: File "/home/pk7/projects/qmc/git_QMCPACK_prckent/qmcpack/nexus/bin/nxs-test", line 349, in nunit
1452: run_external_unit_test(test_name,unit_test)
1452: File "/home/pk7/projects/qmc/git_QMCPACK_prckent/qmcpack/nexus/bin/nxs-test", line 388, in run_external_unit_test
1452: unit_test()
1452: File "/home/pk7/projects/qmc/git_QMCPACK_prckent/qmcpack/nexus/tests/unit/test_pyscf_input.py", line 577, in test_write
1452: assert(text_eq(text,ref_text))
1452: ^^^^^^^^^^^^^^^^^^^^^^
1452:
1452: Test status: fail
Hopefully unrelated, but note that my installation of graphviz can't produce PNGs, so this trips up a few tests all the time.
Proposed changes
This PR adds support for arbitrary supercell twists/twist grids in workflows involving PySCF and QMCPACK.
This PR is now ready for review.
What type(s) of changes does this code introduce?
Does this introduce a breaking change?
What systems has this change been tested on?
Laptop, Improv at ALCF
Checklist
Path out of WIP