-
Notifications
You must be signed in to change notification settings - Fork 53
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
Add ECAdd() Bloq #1425
Merged
Merged
Add ECAdd() Bloq #1425
Changes from 27 commits
Commits
Show all changes
28 commits
Select commit
Hold shift + click to select a range
785c7e2
Initial commit of ec add waiting on equals to be merged.
fpapa250 a4d6317
Merge branch 'main' into ec-add-decomp
fpapa250 3d1077c
Working on tests for ECAdd
fpapa250 cafb1b7
ECAdd implementation and tests
fpapa250 2fad7cf
remove modmul typo
fpapa250 65833a8
Merge branch 'main' into ec-add-decomp
fpapa250 70eec40
Merge branch 'main' into ec-add-decomp
fpapa250 ac78cf9
Merge branch 'ec-add-decomp' of github.com:fpapa250/Qualtran into ec-…
fpapa250 2f4b91b
Fix mypy errors
fpapa250 2b7c856
Better bugfix for ModAdd
fpapa250 f08bcec
Change mod inv classical impl to use monttgomery inv
fpapa250 8f3a8b8
Merge branch 'main' into ec-add-decomp
NoureldinYosri 31244a5
Fix pytest error
fpapa250 a64e739
Merge branch 'ec-add-decomp' of github.com:fpapa250/Qualtran into ec-…
fpapa250 d209e4e
Merge branch 'main' into ec-add-decomp
NoureldinYosri 41d7e03
Fix some comments
fpapa250 1e0e118
ECAdd lots of testing
fpapa250 06f7412
Merge branch 'main' into ec-add-decomp
fpapa250 94cd202
Add comments about bugs to be fixed
fpapa250 9227c0d
Reduce complexity by keeping intermediate values mod p
fpapa250 d9918fb
Merge branch 'main' into ec-add-decomp
mpharrigan 9562d22
Merge branch 'main' into ec-add-decomp
mpharrigan 5828eb7
Stash qmontgomery tests
fpapa250 d8ba6d3
Address comments
fpapa250 d09e500
Fix montgomery prod/inv calculations + pylint/mypy
fpapa250 60961aa
Merge branch 'main' into ec-add-decomp
mpharrigan 6f0f94b
Merge branch 'main' into ec-add-decomp
mpharrigan cf2610a
Merge branch 'main' into ec-add-decomp
fpapa250 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
I don't think this should be a shim
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.
I agree it's easy to implement, but I think it is out of scope for this PR - it is a component of ModInv not ECAdd. I just changed it in this PR to make the Toffoli count test more accurate.