Skip to content
This repository has been archived by the owner on May 11, 2024. It is now read-only.

A normal prover should at least wait protocolConfig.targetProofTime seconds before submitting proofs #231

Closed
davidtaikocha opened this issue May 17, 2023 · 3 comments · Fixed by #232

Comments

@davidtaikocha
Copy link
Member

No description provided.

@davidtaikocha davidtaikocha changed the title A normal prover should at least wait protocolConfig.targetProofTime seconds before submit proofs A normal prover should at least wait protocolConfig.targetProofTime seconds before submitting proofs May 17, 2023
@RogerLamTd RogerLamTd moved this from 📝 Todo to 🏗 In progress in Taiko Project Board May 17, 2023
@github-project-automation github-project-automation bot moved this from 🏗 In progress to ✅ Done in Taiko Project Board May 19, 2023
@Brechtpd
Copy link

Targeting a fixed proof target time probably not very stable, because that would mean the base fee would change regardless of what the proof reward is. Calculating the proof delay based on the expected reward the proof would get when it ends up onchain seems like a more stable way to handle this. The expected prover reward would be something the prover could set personally to whatever value it wants.

@davidtaikocha
Copy link
Member Author

Targeting a fixed proof target time probably not very stable, because that would mean the base fee would change regardless of what the proof reward is. Calculating the proof delay based on the expected reward the proof would get when it ends up onchain seems like a more stable way to handle this. The expected prover reward would be something the prover could set personally to whatever value it wants.

agreed, maybe we can change the current codebase to do that? cc @RogerLamTd

@RogerLamTd
Copy link
Contributor

RogerLamTd commented May 19, 2023

Thoughts on adding another flag to allow provers to set target reward? I could work on that! @davidtaikocha @Brechtpd

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
Status: Done
3 participants