You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
sortByProfitabilityUsingMaxCover is doing maxCover for the attestations from same slot.
However I guess there might be some attestations from different committee index.
The test codes are not caring about attestations from different committee as well
The whole attestations are guaranteed to be from same committee ?
The proposer is fetching those attestations from the pools,
so I'm not getting how it's guaranteed.
reacted with thumbs up emoji reacted with thumbs down emoji reacted with laugh emoji reacted with hooray emoji reacted with confused emoji reacted with heart emoji reacted with rocket emoji reacted with eyes emoji
-
prysm/beacon-chain/rpc/prysm/v1alpha1/validator/proposer_attestations.go
Line 148 in a85b444
sortByProfitabilityUsingMaxCover is doing maxCover for the attestations from same slot.
However I guess there might be some attestations from different committee index.
The test codes are not caring about attestations from different committee as well
The whole attestations are guaranteed to be from same committee ?
The proposer is fetching those attestations from the pools,
so I'm not getting how it's guaranteed.
prysm/beacon-chain/rpc/prysm/v1alpha1/validator/proposer_attestations.go
Line 28 in a85b444
Please fix me if I'm wrong. 🙏
Beta Was this translation helpful? Give feedback.
All reactions