sui icon indicating copy to clipboard operation
sui copied to clipboard

Add support for minimum randomness generation interval.

Open aschran opened this issue 1 year ago • 5 comments

Adds back deferral for randomness-using tx arriving in commits for which randomness is not generated.

aschran avatar Apr 25 '24 23:04 aschran

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
sui-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 30, 2024 6:41pm
3 Ignored Deployments
Name Status Preview Comments Updated (UTC)
multisig-toolkit ⬜️ Ignored (Inspect) Visit Preview Apr 30, 2024 6:41pm
sui-kiosk ⬜️ Ignored (Inspect) Visit Preview Apr 30, 2024 6:41pm
sui-typescript-docs ⬜️ Ignored (Inspect) Visit Preview Apr 30, 2024 6:41pm

vercel[bot] avatar Apr 25 '24 23:04 vercel[bot]

Just curious, will there still be randomness-only checkpoints, or it will be another discussion / change?

mwtian avatar Apr 26 '24 03:04 mwtian

Just curious, will there still be randomness-only checkpoints, or it will be another discussion / change?

Yes there are still randomness checkpoints. I don't follow what other discussion you're asking about?

aschran avatar Apr 26 '24 05:04 aschran

Just curious, will there still be randomness-only checkpoints, or it will be another discussion / change?

Yes there are still randomness checkpoints. I don't follow what other discussion you're asking about?

I mean with < 5/s randomness rounds and potentially ~10/s normal checkpoints, there seem to be less latency benefit for separate randomness checkpoints. I imagine there will be plan or discussion on if / how / when we would make changes to the randomness checkpoint logic.

mwtian avatar Apr 26 '24 07:04 mwtian

Just curious, will there still be randomness-only checkpoints, or it will be another discussion / change?

Yes there are still randomness checkpoints. I don't follow what other discussion you're asking about?

I mean with < 5/s randomness rounds and potentially ~10/s normal checkpoints, there seem to be less latency benefit for separate randomness checkpoints. I imagine there will be plan or discussion on if / how / when we would make changes to the randomness checkpoint logic.

Yes it's something that could be addressed separately, when code is written to coalesce pending checkpoints. At least it doesn't make as much of a difference either way after this change because we'll only have max 5/s extra and not 2x.

aschran avatar Apr 26 '24 17:04 aschran