-
Notifications
You must be signed in to change notification settings - Fork 33
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
Create a Subtract bloq #924
Conversation
I don't think the relative -> absolute is the correct fix for this particular issue |
Can you take a look at #933, revert the imports change, and resolve conflicts? |
@mpharrigan done, ptal |
@@ -409,7 +409,7 @@ class SimpleAddConstant(Bloq): | |||
[Improved quantum circuits for elliptic curve discrete logarithms](https://arxiv.org/abs/2001.09580) Fig 2a | |||
""" | |||
|
|||
bitsize: int | |||
bitsize: Union[int, sympy.Expr] |
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.
We have a SymbolicInt in qualtran.symbolics.types now.
Args: | ||
a_dtype: Quantum datatype used to represent the integer a. | ||
b_dtype: Quantum datatype used to represent the integer b. Must be large | ||
enough to hold the result in the output register of a + b, or else it simply |
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.
nit: a - b
related to #235 since to do comparison in N Toffolis we need to be able to do subtraction