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

Rekey - alt_bn128 and poseidon_syscall #319

Merged
merged 1 commit into from
Mar 22, 2024

Conversation

Lichtso
Copy link

@Lichtso Lichtso commented Mar 19, 2024

Problem

#294 requires new feature gate keys.

Summary of Changes

  • Rekeys enable_alt_bn128_compression_syscall and enable_poseidon_syscall.
  • Adds simplify_alt_bn128_syscall_error_codes.

Copy link

@0x0ece 0x0ece left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We rediscussed this. No rekey, just 1 new feature gate.

@Lichtso Lichtso force-pushed the feature/rekey_alt_bn128_and_poseidon branch from de09fd7 to 81f5ecb Compare March 21, 2024 08:59
@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 81.9%. Comparing base (e8526f6) to head (81f5ecb).
Report is 18 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #319   +/-   ##
=======================================
  Coverage    81.9%    81.9%           
=======================================
  Files         837      837           
  Lines      226874   226875    +1     
=======================================
+ Hits       185868   185875    +7     
+ Misses      41006    41000    -6     

@Lichtso Lichtso merged commit 8f830c4 into master Mar 22, 2024
47 checks passed
@Lichtso Lichtso deleted the feature/rekey_alt_bn128_and_poseidon branch March 22, 2024 06:58
Copy link

mergify bot commented Apr 19, 2024

Backports to the beta branch are to be avoided unless absolutely necessary for fixing bugs, security issues, and perf regressions. Changes intended for backport should be structured such that a minimum effective diff can be committed separately from any refactoring, plumbing, cleanup, etc that are not strictly necessary to achieve the goal. Any of the latter should go only into master and ride the normal stabilization schedule. Exceptions include CI/metrics changes, CLI improvements and documentation updates on a case by case basis.

mergify bot pushed a commit that referenced this pull request Apr 19, 2024
Adds simplify_alt_bn128_syscall_error_codes.

(cherry picked from commit 8f830c4)
Lichtso added a commit that referenced this pull request Apr 23, 2024
Rekey - alt_bn128 and poseidon_syscall (#319)

Adds simplify_alt_bn128_syscall_error_codes.

(cherry picked from commit 8f830c4)

Co-authored-by: Alexander Meißner <AlexanderMeissner@gmx.net>
anwayde pushed a commit to firedancer-io/agave that referenced this pull request Jul 23, 2024
…) (anza-xyz#899)

Rekey - alt_bn128 and poseidon_syscall (anza-xyz#319)

Adds simplify_alt_bn128_syscall_error_codes.

(cherry picked from commit 8f830c4)

Co-authored-by: Alexander Meißner <AlexanderMeissner@gmx.net>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants