Reasoning: interTxSpace = 250 ms p90 target (Asia) inflightTimeout = 600 ms 2x interTxSpace + ~100 ms headroom
Exposed `nodeTo{Node,Client}VersionDataCodec` instead.
We used to need to call `cborTermVersionDataCodec` to get `VersionDataCodec` from `CodecCBORTerm`. The `VersionDataCodec` and `cborTermVersionDataCodec` is moved to `ouroboros-network:api` library (`Ouroboros.network.CodecCBORTerm` module). `cardano-diffusion` provides both `nodeToNodeVersionDataCodec` and `nodeToClientVersionDataCodec`. This required a refactorisation in tests to get rid of some newtype wrappers, however the semantics of all the tests is preserved.
The tests started to fail due to usage of version data codec which doesn't satisfy round robin property on invalid version data in the previous commit.
The tests started to fail due to usage of version data codec which doesn't satisfy round robin property on invalid version data in the previous commit.
Exposed `nodeTo{Node,Client}VersionDataCodec` instead.
We used to need to call `cborTermVersionDataCodec` to get `VersionDataCodec` from `CodecCBORTerm`. The `VersionDataCodec` and `cborTermVersionDataCodec` is moved to `ouroboros-network:api` library (`Ouroboros.network.CodecCBORTerm` module). `cardano-diffusion` provides both `nodeToNodeVersionDataCodec` and `nodeToClientVersionDataCodec`. This required a refactorisation in tests to get rid of some newtype wrappers, however the semantics of all the tests is preserved.
Shorten the timeout to 1 second. Moved the delay from each client to the main thread. This way the output is written quicker to stdout.
Ping runs in either of the two modes query tip of ping, so a sum type is better than a product of two boolean values.
We used to need to call `cborTermVersionDataCodec` to get `VersionDataCodec` from `CodecCBORTerm`. The `VersionDataCodec` and `cborTermVersionDataCodec` is moved to `ouroboros-network:api` library (`Ouroboros.network.CodecCBORTerm` module). `cardano-diffusion` provides both `nodeToNodeVersionDataCodec` and `nodeToClientVersionDataCodec`. This required a refactorisation in tests to get rid of some newtype wrappers, however the semantics of all the tests is preserved.
The tests started to fail due to usage of version data codec which doesn't satisfy round robin property on invalid version data in the previous commit.
Exposed `nodeTo{Node,Client}VersionDataCodec` instead.