cerc-io/lighthouse
10
0
Fork 0
Code Issues 1 Pull Requests Packages Projects Releases Wiki Activity

Fix attestations bug in block builder.

Browse Source 
It was previously producing too many attestations in some scenarios.
This commit is contained in:
Paul Hauner 2019-03-10 08:31:40 +11:00
parent 90d00773cb
commit 5f3da0732f
No known key found for this signature in database
GPG Key ID: D362883A9218FCC6
1 changed files with 7 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
eth2/state_processing/benching_utils/src/beacon_block_bencher.rs
View File

@ -92,18 +92,20 @@ impl BeaconBlockBencher {
// - The slot is too old to be included in a block at this slot. // - The slot is too old to be included in a block at this slot.
// - The `MAX_ATTESTATIONS`. // - The `MAX_ATTESTATIONS`.
loop { loop {
if attestations_added == spec.max_attestations {
break;
}
if state.slot >= slot + spec.slots_per_epoch { if state.slot >= slot + spec.slots_per_epoch {
break; break;
} }
for (committee, shard) in state.get_crosslink_committees_at_slot(slot, spec)? { for (committee, shard) in state.get_crosslink_committees_at_slot(slot, spec)? {
committees.push((slot, committee.clone(), committee.clone(), *shard)) if attestations_added >= spec.max_attestations {
break;
} }
committees.push((slot, committee.clone(), committee.clone(), *shard));
attestations_added += 1; attestations_added += 1;
}
slot -= 1; slot -= 1;
} }