-
Notifications
You must be signed in to change notification settings - Fork 86
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
Shelley: update query type of LocalStateQuery protocol #1848
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.
LGTM
@@ -239,20 +248,81 @@ instance LedgerDerivedInfo (ShelleyBlock c) where | |||
QueryLedger | |||
-------------------------------------------------------------------------------} | |||
|
|||
newtype NonMyopicMemberRewards c = NonMyopicMemberRewards { |
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.
Do we want to move this over to cardano-ledger-specs
at some point?
ouroboros-consensus-shelley/src/Ouroboros/Consensus/Shelley/Ledger/Ledger.hs
Outdated
Show resolved
Hide resolved
@@ -261,6 +331,9 @@ instance Crypto c => ShowQuery (Query (ShelleyBlock c)) where | |||
getPParams :: SL.ShelleyState c -> SL.PParams | |||
getPParams = SL.esPp . SL.nesEs | |||
|
|||
getProposedPPUpdates :: SL.ShelleyState c -> SL.ProposedPPUpdates c |
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.
What's the proposed use of this? I'm tempted to say that in its current form it isn't terribly useful, since this gives no indication of when the proposal will come in or whether it's likely to do so...
Is IntersectMBO/cardano-ledger#1337 likely to be useful here?
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 must admit that I'm not familiar with all these types. I wouldn't be surprised that we'll need to extend the Query
type in the future. But that shouldn't be too difficult. This PR adds the necessary infrastructure (+ test infrastructure) so that future updates should be straightforward.
It will be needed for Shelley.
0ff6990
to
2b3be37
Compare
bors r+ |
Fixes #1442.