-
Notifications
You must be signed in to change notification settings - Fork 179
raise_account_cu_limits #306
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
raise_account_cu_limits #306
Conversation
|
||
## Summary | ||
|
||
Increase the per account CU limit from 12M to 20M. |
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.
can we propose implementing this as "40% of block limit" instead of static 20M? rather take it to 24M when block limit goes to 60M and not have to dance around more SIMDs and feature gates
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.
I'm okay with it. Although if we fix execution concurrency issues, anything beyond 25% may get a bit dicey (or leave capacity on the table)
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.
only way to fix exec concurrency is to delete priority fees. gfl with that. most practical option is to crash the pf market with surplus capacity. accelerate
- Larger blocks may cause unforseen issues in infrastructure beyond the | ||
validators. |
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.
how will the blocks get larger? the proposal only changes a constraint on the internal composition. we're still bound by block limit
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.
yeah, this is bad copy pasta. Will fix
9f7ca38
to
e5bacce
Compare
@topointon-jump - would appreciate FD review |
e5bacce
to
06edbea
Compare
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.
lgtm now
No description provided.