-
Notifications
You must be signed in to change notification settings - Fork 1
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
Added functions for mpi support in qutensornet #53
base: main
Are you sure you want to change the base?
Added functions for mpi support in qutensornet #53
Conversation
4023539
to
e6be227
Compare
for more information, see https://pre-commit.ci
Tests are missing |
Added pytest. Need to install pytest-mpi. |
The rather ideal situation in which it could be convenient to just vendor the part we're interested in. What do you exactly need from |
To test mpi implementation. I thought you were referring that pytests are missing in your previous comment. |
Indeed
I was looking for a much more specific answer. How are you planning to use the features of |
I was looking for how pytests can be implemented while using mpi. @liweintu suggested this pytest-mpi which I am using now. It is in test_quimb_backend.py to test the function which parallelize the circuit evaluation. As commented in the file, pytest for mpi implementation can be checked by command |
This is a basic Pytest feature, no need of plugins to do it. You can define yourself the marker in the Pytest configs (not even needed to use executable code, just write in The official documentation about markers is here: |
Thanks for the suggestion. I have updated pyproject.toml to use for 2 cpus (just used minimum requirement) and tests are passing using the marker feature. Updated the files in the repo. |
No description provided.