Skip to content

looks like locate file was the issue#26

Merged
holgerd77 merged 1 commit into
ethereumjs:masterfrom
ETHCF:fix-locate-file
Oct 31, 2025
Merged

looks like locate file was the issue#26
holgerd77 merged 1 commit into
ethereumjs:masterfrom
ETHCF:fix-locate-file

Conversation

@n8wb
Copy link
Copy Markdown
Contributor

@n8wb n8wb commented Oct 31, 2025

@n8wb n8wb mentioned this pull request Oct 31, 2025
@holgerd77
Copy link
Copy Markdown
Member

🙏 🙏 🙏 🙏 🙏

@holgerd77 holgerd77 merged commit 43e33f8 into ethereumjs:master Oct 31, 2025
1 check passed
@holgerd77
Copy link
Copy Markdown
Member

Hmm, tests are now timing out for me locally:

grafik

Any idea what's happening?

@holgerd77
Copy link
Copy Markdown
Member

(so: the test directly here on the library, not the ones over on the monorepo)

Comment thread package.json
"webdriverio": "^9.20.0"
},
"dependencies": {
"@vitest/browser-webdriverio": "^4.0.6"
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Ah, and this we need to fix as well, we can't have this as a production dependency. I can do along release though.

@holgerd77
Copy link
Copy Markdown
Member

Anyhow: have increased the timeout

@holgerd77
Copy link
Copy Markdown
Member

Have released as rc.4 and updated ethereumjs/ethereumjs-monorepo#4169 along.

@holgerd77
Copy link
Copy Markdown
Member

Ok, works, super great! 💯

I won't ask for any optimizations anymore for now, but one last question, since this is the "core of it all":

So, are you really sure/confident that this (the whole library) makes things a lot faster in production? So at least on the testing side, things are dramatically slow compared to the JS version.

So and if proof verification is faster: have you got some (very rough) ballpark numbers how much faster this is compared to the JS version (did you try the JS version?)? Because if we are not confident here, the selling point is rather gone (likely for yourself as well).

@n8wb
Copy link
Copy Markdown
Contributor Author

n8wb commented Oct 31, 2025

Ok, works, super great! 💯

I won't ask for any optimizations anymore for now, but one last question, since this is the "core of it all":

So, are you really sure/confident that this (the whole library) makes things a lot faster in production? So at least on the testing side, things are dramatically slow compared to the JS version.

So and if proof verification is faster: have you got some (very rough) ballpark numbers how much faster this is compared to the JS version (did you try the JS version?)? Because if we are not confident here, the selling point is rather gone (likely for yourself as well).

This is the average time over 30 runs each, run in the same way as the tests in the ethereumjs-monorepo (so running in chrome essentially to make it a fair comparison)

WASM KZG: 27.866666666666667 ms per commitment
JS KZG: 4.466666666666667 ms per commitment
WASM KZG: 276.06666666666666 ms per proof
JS KZG: 644.7666666666667 ms per proof
WASM KZG: 12.233333333333333 ms per verify proof
JS KZG: 28.333333333333332 ms per verify proof

@holgerd77
Copy link
Copy Markdown
Member

@n8wb Have published https://github.com/ethereumjs/kzg-wasm/releases/tag/kzg-wasm%401.0.0, thanks a lot again! 🙏

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