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

Stop normalising to_halfturns to [0,2\pi) #136

Closed
doug-q opened this issue Oct 23, 2024 · 0 comments · Fixed by #137
Closed

Stop normalising to_halfturns to [0,2\pi) #136

doug-q opened this issue Oct 23, 2024 · 0 comments · Fixed by #137
Assignees
@ss2165

Comments

@doug-q
Copy link
Collaborator

doug-q commented Oct 23, 2024

No description provided.

@ss2165 ss2165 self-assigned this Oct 23, 2024
ss2165 added a commit that referenced this issue Oct 23, 2024
@ss2165
fix: don't normalise half turns
247b096 
Closes #136

drive-by: cargo update (hugr 0.13.2)

I have tested the test inhttps://github.com/quantinuum-dev/guppy-integration/pull/19 now passes and the example has expected behaviour with crz oracle.
@ss2165 ss2165 mentioned this issue Oct 23, 2024
Merged
github-merge-queue bot pushed a commit that referenced this issue Oct 23, 2024
@ss2165
fix: don't normalise half turns (#137)
c73f36b 
Closes #136

drive-by: cargo update (hugr 0.13.2)

I have tested the test in
quantinuum-dev/guppy-integration#19 now passes and the
example has expected behaviour with crz oracle.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ss2165 ss2165

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: don't normalise half turns CQCL/hugr-llvm
2 participants
@ss2165 @doug-q