-
Notifications
You must be signed in to change notification settings - Fork 184
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
Optimized support for applying exponentials of Paulis #660
Conversation
Command Bot: Processing... |
1 similar comment
Command Bot: Processing... |
Command Bot: Processing... |
I couldn't reproduce the segfault in https://github.com/NVIDIA/cuda-quantum/actions/runs/6207439851/job/16853577621 locally:
All the expected tests are passing
Detail run of the failing
|
Command Bot: Processing... |
7fb8721
to
0c49cd4
Compare
This looks great and it'd be really useful in several ongoing projects with partners. Is there any way we can bump this up? @bettinaheim |
0c49cd4
to
0dc0b32
Compare
Command Bot: Processing... |
1 similar comment
Command Bot: Processing... |
5e0738b
to
26f62f6
Compare
Command Bot: Processing... |
2 similar comments
Command Bot: Processing... |
Command Bot: Processing... |
documenting this, for some reason on CI, we get a function call from Thread 1 "python3.11" received signal SIGSEGV, Segmentation fault.
0x0000000000000000 in ?? ()
#0 0x0000000000000000 in ?? ()
#1 0x00007f94325425eb in std::_Function_handler<void (bool), nvqir::CircuitSimulator::applyExpPauli(double, std::vector<unsigned long, std::allocator<unsigned long> > const&, cudaq::spin_op const&)::{lambda(cudaq::pauli, unsigned long)#1}::operator()(cudaq::pauli, unsigned long) const::{lambda(bool)#2}>::_M_invoke(std::_Any_data const&, bool&&) ()
from /root/.local/lib/python3.11/site-packages/cudaq/../lib/libnvqir-custatevec-fp32.so
#2 0x00007f94324c2d4e in ?? ()
from /root/.local/lib/python3.11/site-packages/cudaq/../lib/libnvqir-qpp.so
#3 0x00007f945faffbef in __quantum__qis__exp_pauli ()
from /root/.local/lib/python3.11/site-packages/cudaq/../lib/libnvqir.so
#4 0x00007f946105f0b4 in __nvqpp__mlirgen____nvqppBuilderKernel_108506462586
()
#5 0x00007f946105f1b1 in __nvqppBuilderKernel_108506462586.thunk ()
#6 0x00007f945fce3e33 in ?? ()
from /root/.local/lib/python3.11/site-packages/cudaq/../lib/libcudaq-platform-default.so
#7 0x00007f945fb532d5 in altLaunchKernel () |
Command Bot: Processing... |
1 similar comment
Command Bot: Processing... |
62b202a
to
54bb8af
Compare
Command Bot: Processing... |
54bb8af
to
1039932
Compare
Command Bot: Processing... |
Command Bot: Processing... |
CUDA Quantum Docs Bot: A preview of the documentation can be found here. |
2909ffa
to
846c7ac
Compare
Signed-off-by: Alex McCaskey <amccaskey@nvidia.com>
Convert the headers to use ASCIIZ string literals for the final argument to exp_pauli(). (Simplifies the AST presented to the bridge and falls back to using code that handles string literals.) Update LowerToQIR for the new types, fix bugs. Fix up tests.
Signed-off-by: Alex McCaskey <amccaskey@nvidia.com>
Signed-off-by: Alex McCaskey <amccaskey@nvidia.com>
Signed-off-by: Alex McCaskey <amccaskey@nvidia.com>
Signed-off-by: Alex McCaskey <amccaskey@nvidia.com>
846c7ac
to
95f2641
Compare
Command Bot: Processing... |
CUDA Quantum Docs Bot: A preview of the documentation can be found here. |
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.
LGTM 👍
Co-authored-by: Thien Nguyen <58006629+1tnguyen@users.noreply.github.com>
Command Bot: Processing... |
CUDA Quantum Docs Bot: A preview of the documentation can be found here. |
Enable the following in the runtime and the bridge:
Also:
Add a
cc.string
type, acc.string_literal
op, and updates to lower to all this LLVM.