-
Notifications
You must be signed in to change notification settings - Fork 721
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
CAD-1385: Show KES values in LiveView. #1503
Conversation
90ede3c
to
39c02c9
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
137: CAD-1385: KES-info in RTView. r=CodiePP a=denisshevchenko Show KES-info in RTView. Corresponding changes in `cardano-node`: IntersectMBO/cardano-node#1503 Please note that, if the node is launched with an old protocol (without KES-info), corresponding fields don't hide, but show `"—"` instead. The reason is this: in `Grid view` we have a table where each row corresponds to metric. Suppose three nodes were connected to RTView, but one of them (node `A`) works with `Byron` and two another (`B` and `C`) - with `Cardano`. In this case I still have the same 3 KES-related rows, but for `A` it will be show `"—"`, and for `B` and `C` - real values. Co-authored-by: Denis Shevchenko <denis.shevchenko@iohk.io>
@denisshevchenko could you add a screenshot |
@CodiePP Done (the screenshot was added in a description). |
else [ padTop (Pad 1) $ txt "Start KES period:" | ||
, txt "KES period:" | ||
, txt "KES remaining:" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This alignment seems a bit odd. Perhaps you meant:
else [ padTop (Pad 1) $ txt "Start KES period:" | |
, txt "KES period:" | |
, txt "KES remaining:" | |
else [ padTop (Pad 1) $ txt "Start KES period:" | |
, txt "KES period:" | |
, txt "KES remaining:" |
bors r+ |
Build succeeded |
Updated BP to 1.19.0, and not seeing KES period info in the LiveView mode, FYI. It's just missing, output is identical to 1.18.0. BP node was created from scratch on mainnet 1.18.0 post-Shelley fork, so I don't think its' due to "was launched with Byron-protocol")... |
Please see #1742 |
Add KES period and KES remaining in
LiveView
.Related to #1372.
Please note that if the node doesn't have KES-information (if it was launched with Byron-protocol) KES-related fields in TUI will be hidden.
Screenshot: