Add guard when fill_prev_epoch
is True
in epoch 0
#3570
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.
state_transition_with_full_block
takes two flags,fill_cur_epoch
andfill_prev_epoch
, to control what attestations to include into the new testing block.While
fill_cur_epoch
usesMIN_ATTESTATION_INCLUSION_DELAY
as the minimum slot before taking effect,fill_prev_epoch
results in an annoying assert, requiring caller to work around the case of calling the function as usual at the start.This adds a similar minimum slot
SLOTS_PER_EPOCH
forfill_prev_epoch
before which the flag is ignored, same as forfill_cur_epoch
.