IntersectMBO / IntersectMBO/ouroboros-consensus
Label the NTC events with the local peer identifier
Open
- Dominant language
- Haskell
- Stars
- 67
- Forks
- 43
- Avg merge
- 5d 13h
- Merged PRs (30d)
- 43
Description
@Infinisil noticed that the `localPeer` type parameter for `Ouroboros.Consensus.Node.Tracers.Tracers'` isn't used at all.
This Issue is to add the `TraceLabelPeer localPeer` wrapper to the Node-to-Client events in order to match the existing `TraceLabelPeer remotePeer` wrapper on the Node-to-Node events.
We're thinking that under normal circumstances, there's probably only one (main?) local peer: the wallet. But that's not necessarily the case. So this currently-missing label could be a helpful disambiguator in some scenarios.
Contributor guide
Assessment
This issue has not been assessed yet.