Fix stuck post init on no VRF nonce #317
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Closes #316
The problem is that the default
ComputeBatchSize
is just too large for small inits on testnet or fasnat presets. To avoid an initialization that didn't find a VRF nonce to run much longer than it needs to when searching for a nonce outside of the initialization range the actual usedComputeBatchSize
is limited to the total size of the initialization. Larger values don't make sense, since a nonce should statistically be found with ~ 99.97% probability during initialization every subsequent batch of the same size as the PoST should have another 99.97% of containing a valid nonce. Failing to find a nonce more than 2x outside of the init range is statistically impossible, so there is no need to have a batch size bigger than the total init size 🙂