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.

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

Introduce checks about AllocatedPoint #258

Merged
merged 4 commits into from
Nov 14, 2023
Merged

Introduce checks about AllocatedPoint #258

merged 4 commits into from
Nov 14, 2023

Conversation

srinathsetty
Copy link
Collaborator

@srinathsetty srinathsetty commented Nov 13, 2023

This PR introduces checks that non-deterministically provided curve points either reside on the curve or they are a point at infinity.

h/t Daniel Dore

@srinathsetty srinathsetty merged commit f602a48 into main Nov 14, 2023
6 checks passed
@srinathsetty srinathsetty deleted the on_curve branch November 14, 2023 01:23
huitseeker pushed a commit to huitseeker/Nova that referenced this pull request Nov 14, 2023
* introduce checks on AllocatedPoint

* update version

* fix test_pp_digest

* optimize checks
huitseeker added a commit to huitseeker/Nova that referenced this pull request Nov 14, 2023
* Introduce checks about AllocatedPoint (microsoft#258)

* introduce checks on AllocatedPoint

* update version

* fix test_pp_digest

* optimize checks

* chore: adjust supernova tests

---------

Co-authored-by: Srinath Setty <[email protected]>
huitseeker added a commit to huitseeker/Nova that referenced this pull request Jan 24, 2024
…crosoft#258)

- Removed OpenCL support from GPU testing and respective CI/CD processes
- Revised `gpu-ci.yml` to reflect changes in the GPU testing environment
- Eliminated unnecessary dependencies from main dependencies of `Cargo.toml`
- Moved dependencies `tap`, `tracing-texray`, and `tracing-subscriber` to development dependencies
- Cleaned up formatting in `dev-dependencies` section of `Cargo.toml`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant