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.

Sign up for GitHub

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

Jump to bottom

Added marshalling for all supported curves #48

Merged
merged 2 commits into from
Oct 4, 2024
Merged

Added marshalling for all supported curves #48

merged 2 commits into from
Oct 4, 2024

Conversation

vo-nil
Copy link
Contributor

@vo-nil vo-nil commented Sep 30, 2024

closes #47

@vo-nil vo-nil self-assigned this Sep 30, 2024
@vo-nil vo-nil marked this pull request as draft September 30, 2024 08:39
@vo-nil vo-nil force-pushed the 47-implement-marshalling-for-all-curves branch from 034cd9b to 4349217 Compare October 3, 2024 07:55
@vo-nil
Added marshalling for all supported curves
b669fdc 
closes #47

Add missing header to test #47

Add missing header to test #47

Fixed placeholder_proof test #47

added missing headers

Updated marshalling for ed25519 curve #47

Add missing header #47

Add missing headers
@vo-nil vo-nil force-pushed the 47-implement-marshalling-for-all-curves branch from e1dba47 to b669fdc Compare October 3, 2024 15:44
@github-actions GitHub Actions
Copy link

github-actions bot commented Oct 3, 2024 •
edited
Loading

Clang Test Results

  153 files  ±0    153 suites  ±0   10m 45s ⏱️ + 3m 48s
3 207 tests +9  3 201 ✅ +9  6 💤 ±0  0 ❌ ±0 
3 274 runs  +9  3 268 ✅ +9  6 💤 ±0  0 ❌ ±0 

Results for commit e1238fa. ± Comparison against base commit f5a0011.

♻️ This comment has been updated with latest results.

@github-actions GitHub Actions
Copy link

github-actions bot commented Oct 3, 2024 •
edited
Loading

Gcc Test Results

  153 files  ±0    153 suites  ±0   11m 55s ⏱️ + 3m 57s
3 207 tests +9  3 201 ✅ +9  6 💤 ±0  0 ❌ ±0 
3 274 runs  +9  3 268 ✅ +9  6 💤 ±0  0 ❌ ±0 

Results for commit e1238fa. ± Comparison against base commit f5a0011.

♻️ This comment has been updated with latest results.

@vo-nil vo-nil marked this pull request as ready for review October 3, 2024 16:10
@vo-nil vo-nil requested a review from Iluvmagick October 4, 2024 07:26
@vo-nil vo-nil merged commit 4ad42ed into master Oct 4, 2024
8 checks passed
@vo-nil vo-nil deleted the 47-implement-marshalling-for-all-curves branch October 4, 2024 09:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ETatuzova ETatuzova ETatuzova approved these changes

@Iluvmagick Iluvmagick Iluvmagick approved these changes

@martun martun Awaiting requested review from martun

Assignees

@vo-nil vo-nil

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Implement marshalling of curve elements for all supported curves
3 participants
@vo-nil @Iluvmagick @ETatuzova