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

Re-enable Praos tests and update for KES #1545

Closed
edsko opened this issue Jan 31, 2020 · 3 comments
Closed

Re-enable Praos tests and update for KES #1545

edsko opened this issue Jan 31, 2020 · 3 comments
Labels
consensus issues related to ouroboros-consensus testing
Milestone

Comments

@edsko
Copy link
Contributor

edsko commented Jan 31, 2020

No description provided.

@edsko edsko added consensus issues related to ouroboros-consensus shelley testnet testing labels Jan 31, 2020
@edsko
Copy link
Contributor Author

edsko commented Jan 31, 2020

Alternatively, if the TPraos implementation is parameterized over crypto and can be run with our mock blocks, we could consider removing BFT and Praos entirely, leaving only PBFT and TPraos (PBFT is supported by the mock blocks).

@edsko
Copy link
Contributor Author

edsko commented Feb 18, 2020

Related: IntersectMBO/ouroboros-consensus#714 .

@edsko
Copy link
Contributor Author

edsko commented Feb 20, 2020

This will be done as part of #1677 , although it's currently a bit of a hack; see IntersectMBO/cardano-base#81.

edsko added a commit that referenced this issue Feb 20, 2020
@edsko edsko added this to the S7 2020-02-27 milestone Feb 20, 2020
@iohk-bors iohk-bors bot closed this as completed in e7cfaab Feb 20, 2020
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 testing
Projects
None yet
Development

No branches or pull requests

1 participant