-
Notifications
You must be signed in to change notification settings - Fork 12
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
Implements #31 - Generators available as extended #34
Conversation
Every in-memory operation is performed with extended points, so the user of the API will always need to convert the Affine Points to extend before performing the operations. To facilitate, we made available by default the generators also in extended form.
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 should include tests that prove that the point is indeed a valid generator.
See tests for the other generator structures.
@@ -24,7 +24,7 @@ | |||
|
|||
#![no_std] | |||
// Catch documentation errors caused by code changes. | |||
#![deny(intra_doc_link_resolution_failure)] | |||
#![deny(broken_intra_doc_links)] |
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.
GH actions doesn't seem to understant that lint. It's fine if it works anyway
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.
They renamed this recently to broken_intra_doc_links
With a updated nightly you should receive a warning/error if using intra_doc_link_resolution_failure
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.
The extended generator was added to eliminate the need to convert to and from affine coordinates in each programme. The tests for this generator are added in this commit.
A lint error occured due to spacing. This is fixed in this commit.
These have just been added |
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.
LGTM
Every in-memory operation is performed with extended points, so the user
of the API will always need to convert the Affine Points to extend
before performing the operations.
To facilitate, we made available by default the generators also in
extended form.