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.

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

parallel VP runs and gas accounting #9

Closed
tzemanovic opened this issue Jun 14, 2021 · 2 comments
Closed

parallel VP runs and gas accounting #9

tzemanovic opened this issue Jun 14, 2021 · 2 comments

Comments

@tzemanovic
Copy link
Member

tzemanovic commented Jun 14, 2021

We might want to set e.g. maximum parallelism to some reasonable value that would be part of the minimum recommended spec for validator hardware.

The gas accounting formulae for parallelized VPs runs should then be adjusted, so that the parallelized runs are discounted only for the number of runs up to this limit (we currently take the full gas from the VP that used the most gas and divide the rest by a constant).

┆Issue is synchronized with this Asana task by Unito

@cwgoes
Copy link
Contributor

cwgoes commented Jun 16, 2021

Yes; this can be a configurable parameter but we should ensure that the gas costs reflect the actual underlying execution speed.

@sync-by-unito sync-by-unito bot closed this as completed Feb 3, 2022
@juped juped reopened this Feb 3, 2022
@tzemanovic tzemanovic transferred this issue from anoma/anoma-archive Jul 7, 2022
@grarco grarco self-assigned this Dec 5, 2022
@cwgoes
Copy link
Contributor

cwgoes commented Jan 12, 2023

@grarco I believe this is rendered obsolete by the simplified gas semantics but reopen if I'm wrong about that.

@cwgoes cwgoes closed this as completed Jan 12, 2023
@cwgoes cwgoes closed this as not planned Won't fix, can't repro, duplicate, stale Jan 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
Status: Tested in Devnet
Development

No branches or pull requests

6 participants