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

[SIP] Network Labeling for Charts #10582

Closed
juen1jp opened this issue Aug 11, 2020 · 4 comments
Closed

[SIP] Network Labeling for Charts #10582

juen1jp opened this issue Aug 11, 2020 · 4 comments
Labels
enhancement:request Enhancement request submitted by anyone from the community sip Superset Improvement Proposal

Comments

@juen1jp
Copy link

juen1jp commented Aug 11, 2020

Please make sure you are familiar with the SIP process documented
(here)[https://github.com//issues/5602]

[SIP] Proposal for Network Chart Labels

Motivation

We currently use superset for graphing network traffic flows. This leads to charts having large numerical suffixes such as k, m etc. It would be nice to be able to label Bytes transferred as kB, MB, GB, TB etc as well as flows as kb/s, Mb/s, Gb/s etc. Furthermore, we could also support IEC (base 2 instead of base 10) ie 1 kB = 1024 Bytes. These additions would greatly benefit folks using Superset for network monitoring.

Proposed Change

This appears to a relatively straightforward addition with the way the number formatting is written. I would just add a new number format factory that would implement a labeling type for Bytes, bits, bits/s and bytes/s. We could throw in the IEC as well.

New or Changed Public Interfaces

This should have minimal impact. The new labeling would be added to the Format options in the chart controls.

New dependencies

I would not anticipate needing any new dependencies.

Migration Plan and Compatibility

I would not anticipate changes.

Rejected Alternatives

Not quite sure this is applicable.

@juen1jp juen1jp added the sip Superset Improvement Proposal label Aug 11, 2020
@issue-label-bot issue-label-bot bot added the enhancement:request Enhancement request submitted by anyone from the community label Aug 11, 2020
@issue-label-bot
Copy link

Issue-Label Bot is automatically applying the label #enhancement to this issue, with a confidence of 0.79. Please mark this comment with 👍 or 👎 to give our bot feedback!

Links: app homepage, dashboard and code for this bot.

@juen1jp juen1jp changed the title Network Labeling for Charts [SIP] Network Labeling for Charts Aug 11, 2020
@Ayesha-Siddiqa2
Copy link

@amitmiran137 Is this enhancement already available on Superset as I see the issue is marked as closed ?

@Praytic
Copy link

Praytic commented Aug 9, 2022

@amitmiran137 Also interested to know that.

@juen1jp
Copy link
Author

juen1jp commented Aug 9, 2022

pls let me know if this was merged somewhere. I actually did some work on it and could most likely get a PR together if it is still needed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement:request Enhancement request submitted by anyone from the community sip Superset Improvement Proposal
Projects
None yet
Development

No branches or pull requests

4 participants