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: modify minimal preset's field elements per blob to mainnet for dev runs #5484
Conversation
…value for dev runs
Performance Report✔️ no performance regression detected Full benchmark results
|
@@ -44,6 +47,9 @@ else if (network) { | |||
else if (process.argv[2] === "dev") { | |||
process.env.LODESTAR_PRESET = "minimal"; | |||
process.env.LODESTAR_NETWORK = "dev"; | |||
// "c-kzg" has hardcoded the mainnet value, do not use presets | |||
// eslint-disable-next-line @typescript-eslint/naming-convention | |||
setActivePreset(PresetName.minimal, {FIELD_ELEMENTS_PER_BLOB: 4096}); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Adding this in the production code path just for a local test of us is pretty ugly. Please use presetFile functionality in that test to override the values on the test runtime only.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this for running lodestar via --dev
network flag on cli (mostly via prebuild/docker images), test override is a different file
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you detail more who / where this is necessary? Is someone else dependant on this fix?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
generally its used in ethereumjs sims for 4844 (https://github.com/ethereumjs/ethereumjs-monorepo/blob/master/packages/client/test/sim/4844.md) , also we would want our --dev
network to be 4844 compatible with EL clients
🎉 This PR is included in v1.9.0 🎉 |
As part of by parts integration of free the blobs PR
For devnets started using cli in minimal and for the merge interop/inline tests, the
FIELD_ELEMENTS_PER_BLOB
needs to be set to4096
which is the mainnet valuebecause all kzg libraries and ELs process the blobs on this size and is a hardcoded value in them. We only need minimal value for spec tests which does not use the cli path or will not be providing DEV_RUN flag on test setup