Skip to content
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

Introduce QueryHardFork #2370

Merged
merged 3 commits into from
Jul 8, 2020
Merged

Introduce QueryHardFork #2370

merged 3 commits into from
Jul 8, 2020

Conversation

mrBliss
Copy link
Contributor

@mrBliss mrBliss commented Jul 8, 2020

Fixes #2365.

  • Introduce QueryHardFork for queries specific to the hard fork combinator,
    unrelated to any specific era. Provide a query to ask for an Interpreter,
    which can convert a slot to an epoch or time, etc.

  • Generalise QueryAnytime so that the first era can also be queried. We
    still require there to be more than one era. Add the QueryAnytimeByron
    pattern synonym.

  • Rename the EraStart constructor of QueryAnytime to GetEraStart for
    consistency.

  • Change the encoding format of hard fork queries. Since they are not part of
    the final Shelley release yet (right?!), this is fine.

@mrBliss mrBliss added the consensus issues related to ouroboros-consensus label Jul 8, 2020
@mrBliss mrBliss requested a review from edsko July 8, 2020 12:03
@mrBliss mrBliss force-pushed the mrBliss/interpreter-query branch from 78090dc to a1b03db Compare July 8, 2020 12:03
Copy link
Contributor

@edsko edsko left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed over meet.

This allows you to easily refer to the `xs`.
Fixes #2365.

* Introduce `QueryHardFork` for queries specific to the hard fork combinator,
  unrelated to any specific era. Provide a query to ask for an `Interpreter`,
  which can convert a slot to an epoch or time, etc.

* Generalise `QueryAnytime` so that the first era can also be queried. We
  still require there to be more than one era. Add the `QueryAnytimeByron`
  pattern synonym.

* Rename the `EraStart` constructor of `QueryAnytime` to `GetEraStart` for
  consistency.

* Change the encoding format of hard fork queries. Since they are not part of
  the final Shelley release yet (right?!), this is fine.
@mrBliss mrBliss force-pushed the mrBliss/interpreter-query branch from a1b03db to 43a6ab9 Compare July 8, 2020 13:00
@mrBliss
Copy link
Contributor Author

mrBliss commented Jul 8, 2020

bors merge

@iohk-bors
Copy link
Contributor

iohk-bors bot commented Jul 8, 2020

@iohk-bors iohk-bors bot merged commit 7f3277d into master Jul 8, 2020
@iohk-bors iohk-bors bot deleted the mrBliss/interpreter-query branch July 8, 2020 13:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
consensus issues related to ouroboros-consensus
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Make time conversion infrastructure available through the local state query protocol
2 participants