-
Notifications
You must be signed in to change notification settings - Fork 13
Add python interface tests #66
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
Conversation
7f61244
to
86da2ea
Compare
Awesome! Would you mind adding the example_qft.py as a test case? |
03abd97
to
56d35cc
Compare
55952ce
to
dd835e9
Compare
44739cd
to
37895ac
Compare
Hey @steffi7574, I added all the examples as test cases now! Some have minor differences such as I also added I added a I also made a small change to |
Add python interface test based on the examples. The tests compare the energy, population, and pulses at 10 samples of the output values. Some examples were changed slightly to make sure the random numbers were reproducible and they weren't too slow.
Note: If you add more similar pytests just ensure the test function has this signature:
the testname should start with
test_
for pytest to recognize it and thempi_exec
is so a pytest fixture will get used and if you passmpi_exec=mpi_exec
to quandary then the tests can run on the CI with the correct run command (srun or flux run instead of mpirun).