Skip to content
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

Deoxys-TUI: an optional TUI dashboard #17

Merged
merged 20 commits into from
Mar 24, 2024
Merged

Deoxys-TUI: an optional TUI dashboard #17

merged 20 commits into from
Mar 24, 2024

Conversation

noamoulin
Copy link
Contributor

@noamoulin noamoulin commented Mar 16, 2024

What is the new behavior ?

When running the node with the --tui command-line argument (after building it with the tui feature), a dashboard shows some node's metrics like CPU usage, RAM usage, storage usage, network traffic, and classified L1 and L2 logs.

Does this introduce a breaking change?

Not really, just a cool feature

Other information

The dashboard is made with ratatui, implemented as a crate and launched in a thread from node/commands/run.rs in run_node if the --tui argument is provided.

To be able to redirect substrate logs to the TUI, it is necessary to define a writer for tracing (used by substrate). However, in its current state, substrate doesn't allow us to define a custom subscriber, so I had to write a script to directly modify the sources of the version of substrate cloned by cargo. This script is automatically executed in Deoxys if built with the tui flag, so the user doesn't have to worry about it. This modification will not affect the default behavior of substrate and therefore of Deoxys (it ensures that if an environment variable SUBSCRIBER_DEFINED specifying that a subscriber has already been defined, substrate does not redefine it and does not cause a panic).

Coming soon: Fully modular widgets and tabs, more data, p2p map...

@noamoulin noamoulin closed this Mar 20, 2024
@noamoulin noamoulin reopened this Mar 20, 2024
@Trantorian1
Copy link
Collaborator

Trantorian1 commented Mar 21, 2024

🎵 It just works 🎶

@antiyro antiyro merged commit 0e3bb60 into madara-alliance:main Mar 24, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants