-
Notifications
You must be signed in to change notification settings - Fork 22
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
Default genesis parameters yielding positive treasury #425
Conversation
@@ -129,6 +134,8 @@ shelleyGenesisDefaults = | |||
, sgMaxLovelaceSupply = 0 | |||
} | |||
where | |||
unsafeUI :: Rational -> Ledger.UnitInterval |
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.
Ledger has: unsafeBoundRational
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.
But that's a test module. Do we want to introduce a dependency on a test module?
a10ca6e
to
e461e2a
Compare
82e3d8f
to
8447322
Compare
b1fb00b
to
2a2d0dd
Compare
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.
LGTM but move the new function to Cardano.Api.Utils
2a2d0dd
to
a7f5ace
Compare
…t-types-for-drep-commands Command argument types for `drep` commands
Changelog
Context
Without non-zero rho and tau, treasury is not growing.
Checklist