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

Table Panel: Add Stat Cell #87780

Closed
wants to merge 1 commit into from
Closed

Conversation

codeincarnate
Copy link
Collaborator

This PR adds a stat cell to the table panel. This is useful as many users want to display prominent statistics on tables for a variety of use cases including ops monitoring (think CPU, Load, etc), finance (dollars, trends and more), science (monitor values), and more besides.

Currently not really working but added here to track progress.

Please check that:

  • It works as expected from a user's perspective.
  • If this is a pre-GA feature, it is behind a feature toggle.
  • The docs are updated, and if this is a notable improvement, it's added to our What's New doc.

@torkelo
Copy link
Member

torkelo commented May 14, 2024

not 100% I see the point here. It's not like it can be much more prominent?

@codeincarnate
Copy link
Collaborator Author

@torkelo it's from a customer request, in particular it's a potential solution to show trend indicators in the table which the stat panel already does. It may make more sense to do a trend cell or something like that. I've also seen the image cell used with value mappings to do something similar but I'm not sure it would cover this use case.

What do you think?

@torkelo
Copy link
Member

torkelo commented May 15, 2024

@codeincarnate yea, a trend cell could work but what would the trend be based on? (Same problem for cell)

Or are you thinking this is used like the chart cell (with cell value at are data frames ) ?

@codeincarnate
Copy link
Collaborator Author

@torkelo definitely thinking along the lines of the chart cell where the cell value is itself a data frame. We could add it as another mode for that potentially 😄

@torkelo
Copy link
Member

torkelo commented May 15, 2024

@codeincarnate do you have any doc that describes the feature request? Is the scenario for time series shown in table?

Is to show a trend number a long a chart cell?

Chart cell I think support diff, if not maybe we can add a trend reducer (value option)

@codeincarnate
Copy link
Collaborator Author

@torkelo bringing together these right now in a design doc

And yep, it is for the timeseries in the table. In terms of trend it would be either show a trend number, direction, or potentially percentage. Trend reducer seems like a really good route!

Given the above I'll close this one for now and reference this conversation in the design doc.

@grafana-delivery-bot grafana-delivery-bot bot removed this from the 11.1.x milestone May 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants