Skip to content

Conversation

@fakedev9999
Copy link
Member

Bumping to Jovian ready release of the kona-client.

@github-actions
Copy link
Contributor

github-actions bot commented Oct 30, 2025

Performance Comparison (ELF: eigenda-range-elf-embedded)

Range 2751087~2751092

Metric Base Branch Current PR Diff (%)
Total Instructions 524281818 518219771 -1.16%
Oracle Verify Cycles 42101285 42101285 0.00%
Derivation Cycles 335655697 329603907 -1.80%
Block Execution Cycles 7103836 7094034 -0.14%
Blob Verification Cycles 712 755 6.04%
Total SP1 Gas 842651982 836166418 -0.77%
Cycles per Block 104856363 103643954 -1.16%
Cycles per Transaction 104856363 103643954 -1.16%
BN Pair Cycles 0 0 0.00%
BN Add Cycles 0 0 0.00%
BN Mul Cycles 0 0 0.00%
KZG Eval Cycles 0 0 0.00%
EC Recover Cycles 0 0 0.00%
P256 Verify Cycles 0 0 0.00%

Copy link
Collaborator

@Farhad-Shabani Farhad-Shabani left a comment

Choose a reason for hiding this comment

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

Thanks a bunch for taking care of this.
Left a question and a few reminders before moving on.
By the way, shouldn't the cycle-count-diffCI job be passing now with kona v.1.2.0?

Comment on lines 105 to 113
hokulea-compute-proof = { git = "https://github.com/Layr-Labs/hokulea", rev = "e93d73d2207055a0fa89469d14845cbd0139fbe9" }
hokulea-eigenda = { git = "https://github.com/Layr-Labs/hokulea", rev = "e93d73d2207055a0fa89469d14845cbd0139fbe9" }
hokulea-host-bin = { git = "https://github.com/Layr-Labs/hokulea", rev = "e93d73d2207055a0fa89469d14845cbd0139fbe9" }
hokulea-proof = { git = "https://github.com/Layr-Labs/hokulea", rev = "e93d73d2207055a0fa89469d14845cbd0139fbe9" }
hokulea-witgen = { git = "https://github.com/Layr-Labs/hokulea", rev = "e93d73d2207055a0fa89469d14845cbd0139fbe9" }
hokulea-zkvm-verification = { git = "https://github.com/Layr-Labs/hokulea", rev = "e93d73d2207055a0fa89469d14845cbd0139fbe9" }
canoe-sp1-cc-host = { git = "https://github.com/Layr-Labs/hokulea", rev = "e93d73d2207055a0fa89469d14845cbd0139fbe9" }
canoe-sp1-cc-verifier = { git = "https://github.com/Layr-Labs/hokulea", rev = "e93d73d2207055a0fa89469d14845cbd0139fbe9" }
canoe-verifier-address-fetcher = { git = "https://github.com/Layr-Labs/hokulea", rev = "e93d73d2207055a0fa89469d14845cbd0139fbe9" }
Copy link
Collaborator

Choose a reason for hiding this comment

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

Just a reminder to update these to head once your PR in hokulea is merged

Copy link
Member Author

Choose a reason for hiding this comment

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

Do you think it should be blocking for merging this PR?

Copy link
Collaborator

Choose a reason for hiding this comment

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

If there’s no rush and they’re planning to merge today or tomorrow, let’s hold off for now.

Comment on lines 105 to 113
hokulea-compute-proof = { git = "https://github.com/Layr-Labs/hokulea", rev = "e93d73d2207055a0fa89469d14845cbd0139fbe9" }
hokulea-eigenda = { git = "https://github.com/Layr-Labs/hokulea", rev = "e93d73d2207055a0fa89469d14845cbd0139fbe9" }
hokulea-host-bin = { git = "https://github.com/Layr-Labs/hokulea", rev = "e93d73d2207055a0fa89469d14845cbd0139fbe9" }
hokulea-proof = { git = "https://github.com/Layr-Labs/hokulea", rev = "e93d73d2207055a0fa89469d14845cbd0139fbe9" }
hokulea-witgen = { git = "https://github.com/Layr-Labs/hokulea", rev = "e93d73d2207055a0fa89469d14845cbd0139fbe9" }
hokulea-zkvm-verification = { git = "https://github.com/Layr-Labs/hokulea", rev = "e93d73d2207055a0fa89469d14845cbd0139fbe9" }
canoe-sp1-cc-host = { git = "https://github.com/Layr-Labs/hokulea", rev = "e93d73d2207055a0fa89469d14845cbd0139fbe9" }
canoe-sp1-cc-verifier = { git = "https://github.com/Layr-Labs/hokulea", rev = "e93d73d2207055a0fa89469d14845cbd0139fbe9" }
canoe-verifier-address-fetcher = { git = "https://github.com/Layr-Labs/hokulea", rev = "e93d73d2207055a0fa89469d14845cbd0139fbe9" }
Copy link
Member Author

Choose a reason for hiding this comment

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

Do you think it should be blocking for merging this PR?

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.

3 participants