-
Notifications
You must be signed in to change notification settings - Fork 80
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
Optimize deal activation by enabling sector activation batching #1278
Comments
See also #1101 as a likely angle on reducing gas here. |
Here is a trace of ProveCommitAggregate with 8 sectors/deals from https://github.com/anorth/fvm-workbench. This is a baseline against which we can measure improvement. Note that this trace is under constructed (repeatable) conditions of empty state, so it will tend to underestimate the cost of updating large linked data structures vs mainnet state (which also means that improvements are relatively more impactful). https://gist.github.com/alexytsu/8981c9bfe9525fc18d5a058f7d590c30
|
The changes in #1310 were profiled to a savings of ~11% against the previous changes and in total around 34% down since baseline. builtin-actors: commit
|
Profile
activate_deals_and_claim_allocations
batches deal activations for deals in one sector and see how much gas it saves. We might want something to enable activating multiple sectors within one PCA message.Tasks
The text was updated successfully, but these errors were encountered: