-
Notifications
You must be signed in to change notification settings - Fork 139
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
[Tests] [Python] Follow-up for 'pycudaq-mlir' tests #1532
Conversation
CUDA Quantum Docs Bot: A preview of the documentation can be found here. |
7f1f057
to
a8c92cb
Compare
CUDA Quantum Docs Bot: A preview of the documentation can be found here. |
Converting to draft till the CI stage for building assets is fixed... |
docker/build/assets.Dockerfile
Outdated
--param nvqpp_site_config=build/targettests/lit.site.cfg.py | ||
--param nvqpp_site_config=build/targettests/lit.site.cfg.py && \ | ||
"$LLVM_INSTALL_PREFIX/bin/llvm-lit" -v build/python/tests/mlir \ | ||
--param nvqpp_site_config=build/python/tests/mlir/lit.site.cfg.py |
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.
Python is built separately in its own stage. You will need to copy over the build folder from that stage. Scikit is used to build the python support; the pyproject.toml defines the build and disables building tests. I haven't checked where they would end up in if they are enabled during the build. Give it a go and let me know if you run into trouble.
CUDA Quantum Docs Bot: A preview of the documentation can be found here. |
CUDA Quantum Docs Bot: A preview of the documentation can be found here. |
5285ea5
to
b75f919
Compare
CUDA Quantum Docs Bot: A preview of the documentation can be found here. |
CUDA Quantum Docs Bot: A preview of the documentation can be found here. |
22f5ba9
to
93c1bc2
Compare
CUDA Quantum Docs Bot: A preview of the documentation can be found here. |
CUDA Quantum Docs Bot: A preview of the documentation can be found here. |
CUDA Quantum Docs Bot: A preview of the documentation can be found here. |
CUDA Quantum Docs Bot: A preview of the documentation can be found here. |
CUDA Quantum Docs Bot: A preview of the documentation can be found here. |
CUDA Quantum Docs Bot: A preview of the documentation can be found here. |
CUDA Quantum Docs Bot: A preview of the documentation can be found here. |
CUDA Quantum Docs Bot: A preview of the documentation can be found here. |
CUDA Quantum Docs Bot: A preview of the documentation can be found here. |
4b4771d
to
3743811
Compare
CUDA Quantum Docs Bot: A preview of the documentation can be found here. |
3743811
to
f29de4c
Compare
CUDA Quantum Docs Bot: A preview of the documentation can be found here. |
CUDA Quantum Docs Bot: A preview of the documentation can be found here. |
source in the 'assets' dockerfile. * Trying to always build 'python/tests/mlir'
(should be installed beforehand)
Change the order of running `pycudaq-mlir` tests
-- Build wheel with tests enabled -- Run all tests in single stage
(dependencies for enabling tests).
e6c65f4
to
6f7bf3a
Compare
CUDA Quantum Docs Bot: A preview of the documentation can be found here. |
Closing this since it needs further work in build configuration. Setting this flag in the toml file - |
@khalatepradnya I fixed the issues with building the tests for the wheels in #1603. There is an update needed in the pyproject.toml file to add numpy and pytest as requirements for the build-system, and I updated the CMakefile.txt to only add the top-level test subdirectory if the CUDAQ_DISABLE_CPP_FRONTEND is false. Please double check if that picks up all necessary tests, or if I missed some (I see 50 tests running). |
Thank you, I will check. |
Run the Python MLIR tests enabled in PR#1405 while building from source in the 'assets' dockerfile.