-
Notifications
You must be signed in to change notification settings - Fork 155
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Added pulsing incremental computation of the DRep distribution #3592
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this looks great. Just some minor cleanup is needed and one minor question.
We still need to remember all of the votes at the epoch boundary as well, so they don't change, but that will be done as a separate ticket in #3590
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Very nice. Thank you.
c3edea6
to
921173b
Compare
This was added as a 3rd field to VState. Changed Serialisation and Golden Tests. Integrated the DRepDistr into NEWEPOCH, EPOCH, RATIFY, and TICK rules. Reorgnized (and added missing) Lenses for types in and below NewEpochState
Added module Cardano.Ledger.DRepDistr, defines pulsing DRepDistr.
Integrated the DRepDistr into NEWEPOCH, EPOCH, and RATIFY rules.
Description
Addresses issue #3446
Checklist
.cabal
andCHANGELOG.md
files according to theversioning process.
.cabal
files for all affected packages are updated. If you change the bounds in a cabal file, that package itself must have a version increase. (See RELEASING.md)CHANGELOG.md
for the affected packages. New section is never added with the code changes. (See RELEASING.md)fourmolu
(usescripts/fourmolize.sh
)scripts/cabal-format.sh
)hie.yaml
has been updated (usescripts/gen-hie.sh
)