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

157 review eq operators #160

Merged
merged 1 commit into from
May 22, 2024
Merged

157 review eq operators #160

merged 1 commit into from
May 22, 2024

Conversation

vo-nil
Copy link
Contributor

@vo-nil vo-nil commented May 9, 2024

No description provided.

@vo-nil vo-nil requested a review from martun May 9, 2024 21:50
Copy link

github-actions bot commented May 9, 2024

Test Results

     18 files       18 suites   14m 10s ⏱️
1 580 tests 1 580 ✔️ 0 💤 0
3 160 runs  3 160 ✔️ 0 💤 0

Results for commit e33676e.

♻️ This comment has been updated with latest results.

@vo-nil vo-nil marked this pull request as ready for review May 13, 2024 09:06
@vo-nil vo-nil force-pushed the 157-review-eq-operators branch 2 times, most recently from b985367 to 9e32359 Compare May 13, 2024 20:58
@vo-nil vo-nil marked this pull request as draft May 14, 2024 13:53
@vo-nil vo-nil force-pushed the 157-review-eq-operators branch 2 times, most recently from e5042af to 246a62b Compare May 14, 2024 20:34
@vo-nil vo-nil marked this pull request as ready for review May 14, 2024 20:34
@vo-nil vo-nil marked this pull request as draft May 15, 2024 06:04
@vo-nil vo-nil force-pushed the 157-review-eq-operators branch 3 times, most recently from 96ae780 to e33676e Compare May 17, 2024 21:20
@vo-nil vo-nil marked this pull request as ready for review May 17, 2024 21:21
Copy link

changeset-bot bot commented May 20, 2024

⚠️ No Changeset found

Latest commit: f048839

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

Copy link

github-actions bot commented May 20, 2024

Linux Test Results

1 563 tests  ±0   1 563 ✅ ±0   5m 47s ⏱️ -1s
    7 suites ±0       0 💤 ±0 
    7 files   ±0       0 ❌ ±0 

Results for commit 2aa6414. ± Comparison against base commit 4661461.

♻️ This comment has been updated with latest results.

Added eq-operators for fields #157

Removed division operator

data[2] typo #157

Work on curve operations #157

Arithmetic errors fixed #157

Removed comments #157

Removed divisions from hash_to_curve #157

work in progress #157

Reworked dobule and add functions to inplace variants #157

Update for short weierstrass curves #157

Renamed functions to emphasise inplace operation #157

Added type traits and different implementations for curves that support mixed_add #157

twisted extended #157

Replace double with double_inplace for multiexp and wnaf #157

renamed typetrait, removed doubling and mixed_add from static tests #157

disable curves_static test, update on fields #157

Clear version conflict markers #157

work in progress

Good data for babyjubjub montgomery affine #157

Good data for jubjub montgomery affine #157

Fixed montgomery affine functions #157

remove space #157

Reworked add/dbl/madd to return void #157

void mixed_add for element_g2 #157

Reverted EC_INF define, updated addition and doubling function for g2 #157

Revert "Reverted EC_INF define, updated addition and doubling function for g2 #157"

This reverts commit 46824d3.

reverted g2 operations #157

rewrite for clarity #157

updates for blueprint compatibility #157

reworked operator+ #157
@martun martun self-requested a review May 22, 2024 06:03
@vo-nil vo-nil self-assigned this May 22, 2024
@vo-nil vo-nil merged commit a0e078f into master May 22, 2024
3 checks passed
@vo-nil vo-nil deleted the 157-review-eq-operators branch May 22, 2024 06:25
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.

None yet

2 participants