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

fix: Deactivate G2 precompiles #62

Merged
merged 2 commits into from
Jun 21, 2024
Merged

fix: Deactivate G2 precompiles #62

merged 2 commits into from
Jun 21, 2024

Conversation

huitseeker
Copy link
Contributor

minimal de-activation of G2 precompiles to ease performance testing

@huitseeker huitseeker requested review from wwared and storojs72 June 20, 2024 08:29
@huitseeker huitseeker changed the base branch from dev to plonk June 20, 2024 08:29
@huitseeker huitseeker changed the title Deactivate G2 precompiles fix: Deactivate G2 precompiles Jun 20, 2024
@huitseeker huitseeker marked this pull request as ready for review June 20, 2024 08:30
@huitseeker huitseeker force-pushed the deactivate_g2 branch 4 times, most recently from cf86a15 to e1d8cc7 Compare June 21, 2024 00:45
Copy link
Contributor

@wwared wwared left a comment

Choose a reason for hiding this comment

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

The impact is very small, hard to measure since it's on the order of regular measurement noise (around ~2-3 seconds)

I'm ok merging this for now though. I believe this also helps fix the stack overflows/SIGSEGVs in #65

@storojs72: This change requires a re-build of the plonk artifacts (I think)

@wwared wwared merged commit 160a24e into plonk Jun 21, 2024
5 of 6 checks passed
@wwared wwared deleted the deactivate_g2 branch June 21, 2024 03:03
huitseeker added a commit that referenced this pull request Jun 24, 2024
* chore: de-activate G2 precompiles

* fix: remove duplicate insertion
wwared pushed a commit that referenced this pull request Jun 24, 2024
* chore: de-activate G2 precompiles

* fix: remove duplicate insertion
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.

2 participants