IntersectMBO / IntersectMBO/ouroboros-consensus

Label the NTC events with the local peer identifier

Open
#599 0 comments 0 reactions 0 assignees View on GitHub
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

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.