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.

Sign up for GitHub

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

Jump to bottom

Cache shufflings outside of regen state cache #2848

Closed
dapplion opened this issue Jul 15, 2021 · 1 comment · Fixed by #6030
Closed

Cache shufflings outside of regen state cache #2848

dapplion opened this issue Jul 15, 2021 · 1 comment · Fixed by #6030
Assignees
@twoeths
Labels
prio-low This is nice to have. scope-memory Issues to reduce and improve memory usage. scope-performance Performance issue and ideas to improve performance.

Comments

@dapplion
Copy link
Contributor

What is your question?

We use the regen + state cache to retrieve shufflings to validate attestations. Instead we could just cache the shufflings by decision root which requires way less memory than a full CachedBeaconState
@dapplion dapplion self-assigned this Jul 15, 2021
@dapplion dapplion added scope-memory Issues to reduce and improve memory usage. scope-performance Performance issue and ideas to improve performance. labels Jul 15, 2021
@philknows philknows mentioned this issue Nov 29, 2021
Closed
10 tasks
@philknows philknows added the prio-low This is nice to have. label Nov 29, 2021
@dapplion dapplion removed their assignment Jul 11, 2022
@wemeetagain
Copy link
Member

will be resolved by #6008

@dapplion dapplion mentioned this issue Oct 11, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@twoeths twoeths

Labels
prio-low This is nice to have. scope-memory Issues to reduce and improve memory usage. scope-performance Performance issue and ideas to improve performance.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat: implement shuffling cache ChainSafe/lodestar
4 participants
@wemeetagain @twoeths @dapplion @philknows