ouroboros-consensus icon indicating copy to clipboard operation
ouroboros-consensus copied to clipboard

Move TPraos logic from Ledger into Consensus

Open amesgen opened this issue 2 years ago • 1 comments

For historical reasons, the logic for TPraos lives in Ledger instead of Consensus (which would be the natural place for the implementation of a ConsensusProtocol, as is already the case for eg PBft and Praos).

#340 already vendors one function (tickChainDepState) into Consensus, so it makes sense to also migrate the remaining logic.

amesgen avatar Sep 18 '23 12:09 amesgen

I've looked through cardano-ledger, and only test packages currently depend on cardano-protocol-tpraos. However, it's probably still going to take a while to figure out how to completely decouple the TPraos logic from the ledger test code, and we possibly can't do it by ourselves (i.e., consensus team only)

jorisdral avatar Jan 22 '24 11:01 jorisdral