Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[DRAFT] Spin Op Changes with Ben's Updates #2746

Draft
wants to merge 114 commits into
base: main
Choose a base branch
from
Draft

[DRAFT] Spin Op Changes with Ben's Updates #2746

wants to merge 114 commits into from

Conversation

bmhowe23
Copy link
Collaborator

Doing CI testing on my changes to Bettina's branch.

bettinaheim and others added 30 commits March 5, 2025 16:55
@bettinaheim
[partial commit] committing the trivial changes - depends on changes …
4830bb4 
…I haven't committed yet

Signed-off-by: Bettina Heim <[email protected]>
@bettinaheim
[partial commit] more changes that should be ok to commit - depends o…
b834f98 
…n changes I haven't committed yet

Signed-off-by: Bettina Heim <[email protected]>
@bettinaheim
[partial commit] partial progress - depends on changes I haven't comm…
36097ff 
…itted yet

Signed-off-by: Bettina Heim <[email protected]>
@bettinaheim
row majore for matrix class
94da283 
Signed-off-by: Bettina Heim <[email protected]>
@bettinaheim
cmake tests pass - needs clean up
3854090 
Signed-off-by: Bettina Heim <[email protected]>
@bettinaheim
this should cover all C++ llvm lit tests
61a9228 
Signed-off-by: Bettina Heim <[email protected]>
@bettinaheim
some cleanup
48a7ec8 
Signed-off-by: Bettina Heim <[email protected]>
@bettinaheim
addressing some fixmes
f6e3630 
Signed-off-by: Bettina Heim <[email protected]>
@bettinaheim
cleaning up some comments
322b8dd 
Signed-off-by: Bettina Heim <[email protected]>
@bettinaheim
going with get_term_id for term expectation retrieval - just seems fa…
fd7d1a2 
…r cleaner and more predictable

Signed-off-by: Bettina Heim <[email protected]>
@bettinaheim
... and a couple more comments
a0c84af 
Signed-off-by: Bettina Heim <[email protected]>
@bettinaheim
throw if we don't find an expectation value
604ffa7 
Signed-off-by: Bettina Heim <[email protected]>
@bettinaheim
making product iterable
21265e4 
Signed-off-by: Bettina Heim <[email protected]>
@bettinaheim
cleaning up ordering conventions
db7e756 
Signed-off-by: Bettina Heim <[email protected]>
@bettinaheim
minor clean up
3f87949 
Signed-off-by: Bettina Heim <[email protected]>
@bettinaheim
iterator over sum
735f595 
Signed-off-by: Bettina Heim <[email protected]>
@bettinaheim
sorting out operator names
00036ba 
Signed-off-by: Bettina Heim <[email protected]>
@bettinaheim
forgot to save a file
b23343a 
Signed-off-by: Bettina Heim <[email protected]>
@bettinaheim
removing file that should not be committed
f2aa283 
Signed-off-by: Bettina Heim <[email protected]>
@bettinaheim
cleaning up the mess in the operators.h, operator_sum.cpp and product…
dda0eb8 
…_operator.cpp

Signed-off-by: Bettina Heim <[email protected]>
@bettinaheim
some renaming
e55849f 
Signed-off-by: Bettina Heim <[email protected]>
@bettinaheim
more class renaming
6afc0e1 
Signed-off-by: Bettina Heim <[email protected]>
@bettinaheim
reenabling most of the old spin ops tests
b2692aa 
Signed-off-by: Bettina Heim <[email protected]>
@bettinaheim
reenabling most of the old spin ops tests
f672373 
Signed-off-by: Bettina Heim <[email protected]>
@bmhowe23
Early cmake changes necessary (?) for libraries
ba301f2 
Signed-off-by: Ben Howe <[email protected]>
@bettinaheim
Merge pull request NVIDIA#26 from bmhowe23/spin_ops
9b15ccd 
Early cmake changes necessary (?) for libraries
@cuda-quantum-bot
Cleaning up docs preview for PR NVIDIA#26.
aafa774
@bettinaheim
some tools for better backward compatibility
81da29b 
Signed-off-by: Bettina Heim <[email protected]>
@bettinaheim
some progress on the python bindings
1614d8c 
Signed-off-by: Bettina Heim <[email protected]>
@bettinaheim
isolated commit to have the context own its spinop
2a10655 
Signed-off-by: Bettina Heim <[email protected]>
@copy-pr-bot copy-pr-bot
Copy link

copy-pr-bot bot commented Mar 20, 2025

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@bmhowe23
Copy link
Collaborator Author

bmhowe23 commented Mar 20, 2025 •
edited by github-actions bot
Loading

/ok to test

Command Bot: Processing...

github-actions bot pushed a commit that referenced this pull request Mar 20, 2025
@cuda-quantum-bot
Docs preview for PR #2746.
e2efce7
@github-actions GitHub Actions
Copy link

CUDA Quantum Docs Bot: A preview of the documentation can be found here.

bettinaheim and others added 20 commits March 20, 2025 11:02
@bettinaheim
moving to size_t across the board for operators
875cff0 
Signed-off-by: Bettina Heim <[email protected]>
@bettinaheim
switching dimensions to int64_t to match cudensitymat
6ae3e85 
Signed-off-by: Bettina Heim <[email protected]>
@bettinaheim
formatting
093ecf6 
Signed-off-by: Bettina Heim <[email protected]>
@bettinaheim
adjusting data type in test
a264782 
Signed-off-by: Bettina Heim <[email protected]>
@bettinaheim
Merge branch 'main' into spin_ops
04d3515
@bettinaheim
hopefully makes doxygen happy
4a2ace4 
Signed-off-by: Bettina Heim <[email protected]>
@bettinaheim
Merge branch 'spin_ops' of https://github.com/bettinaheim/cuda-quantum …
6d6baad 
…into spin_ops
@bettinaheim
formatting
ce43284 
Signed-off-by: Bettina Heim <[email protected]>
@bmhowe23
Merge remote-tracking branch 'bettinaheim/spin_ops' into spin_ops-bmh
c632a00
@bettinaheim
review comments
23aed9b 
Signed-off-by: Bettina Heim <[email protected]>
@bettinaheim
eliminate user_facing_order
76459e8 
Signed-off-by: Bettina Heim <[email protected]>
@bettinaheim
adding min_degree and max_degree
e897a2c 
Signed-off-by: Bettina Heim <[email protected]>
@bettinaheim
convenience namespaces
a45953f 
Signed-off-by: Bettina Heim <[email protected]>
@bettinaheim
Merge branch 'main' into spin_ops
cc25522
@bmhowe23
Merge remote-tracking branch 'bettinaheim/spin_ops' into spin_ops-bmh
272ccac
@bettinaheim
some ergonomic changes
7dfe497 
Signed-off-by: Bettina Heim <[email protected]>
@bettinaheim
two more deprecation warnings
0f7c461 
Signed-off-by: Bettina Heim <[email protected]>
@bmhowe23
Merge remote-tracking branch 'bettinaheim/spin_ops' into spin_ops-bmh
6f4f3ea
@bettinaheim
adding trim helper
a412093 
Signed-off-by: Bettina Heim <[email protected]>
@bmhowe23
Merge remote-tracking branch 'bettinaheim/spin_ops' into spin_ops-bmh
7386675
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@bmhowe23 @bettinaheim @cuda-quantum-bot @annagrin