Skip to content

Conversation

@tcoratger
Copy link
Collaborator

🗒️ Description

@GrapeBaBa I think we had an ordering problem when compared to Plonky3, please check this, this should solve your problem without any leansig modification.

🔗 Related Issues or PRs

Should close #243

✅ Checklist

  • Ran tox checks to avoid unnecessary CI fails:
    uvx tox
  • Considered adding appropriate tests for the changes.
  • Considered updating the online docs in the ./docs/ directory.

@unnawut
Copy link
Collaborator

unnawut commented Jan 2, 2026

@tcoratger This requires key re-generation right? So I can do so for https://github.com/leanEthereum/leansig-test-keys for prod keys as well

@tcoratger
Copy link
Collaborator Author

@tcoratger This requires key re-generation right? So I can do so for https://github.com/leanEthereum/leansig-test-keys for prod keys as well

Yes correct, please @GrapeBaBa confirm and then we can regenerate the keys

@GrapeBaBa
Copy link
Contributor

Works fine

@tcoratger
Copy link
Collaborator Author

Thanks, will merge, @unnawut then I guess we can regenerate the production keys on the other repo (only if they are generated via the Python code, no need if they are generated via Rust).

@tcoratger tcoratger merged commit 87e5677 into leanEthereum:main Jan 3, 2026
10 checks passed
@unnawut
Copy link
Collaborator

unnawut commented Jan 6, 2026

Btw. I've been running the python keygen in the background since the weekend, expecting to have the keys to update https://github.com/leanEthereum/leansig-test-keys around end of today if not early tomorrow.

@unnawut
Copy link
Collaborator

unnawut commented Jan 7, 2026

Hey all. The new prod keys and test vectors have been generated.

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.

poseidon2 not consistent between python and rust

3 participants