Skip to content

[TSVB] Introducing Timerange Data Mode for Metric Style Visualizations - #37185

Merged
gospodarsky merged 55 commits into
elastic:masterfrom
gospodarsky:tsvb-timerange-support
Jul 18, 2019
Merged

[TSVB] Introducing Timerange Data Mode for Metric Style Visualizations#37185
gospodarsky merged 55 commits into
elastic:masterfrom
gospodarsky:tsvb-timerange-support

Conversation

@gospodarsky

@gospodarsky gospodarsky commented May 27, 2019 •
edited by timroes
Loading

Copy link
Copy Markdown

Fix: #23467

Summary

It has been a long time since the story was created.

This PR attempts to fix a common usability issue with the metric style visualizations by adding a new setting that controls the time range used for matching documents. There are two options:

  • Entire time range will match all the documents selected in the time picker.
  • Last value will match only the documents for the specified interval from the end of the time range.

Before this setting users would often try to use the TSVB metric visualization to display a number for the entire time range not realizing that it was the last bucket. In some instances, users would add an "overall average" then scratch their heads wondering why the average would not match up to the raw values they calculated by hand. This PR is an attempt to try and alleviate that confusion by introducing this new concept up front.

The PR embeds Data timerange mode to the:

  • Metric, Top N, Gauge, Markdown, Table visualizations with Entire time range and Last value options.
  • Rollup Jobs with the Last Value only.

How it looks.

Last value

image

Entire time range

image

Known issues

The value shown if you select an overall timerange of < 30 minutes might be off a bit, due to some Elasticsearch bug (elastic/elasticsearch#43577).

@gospodarsky gospodarsky added the WIP Work in progress label May 27, 2019
@gospodarsky
gospodarsky requested a review from a team May 27, 2019 13:38
@elasticmachine

elasticmachine commented May 27, 2019

Copy link
Copy Markdown
Contributor

💔 Build Failed

@elasticmachine

elasticmachine commented May 28, 2019

Copy link
Copy Markdown
Contributor

💚 Build Succeeded

@gospodarsky gospodarsky changed the title Add Time Range input to Panel on TSVB Add a Time Range input on TSVB May 28, 2019
@gospodarsky gospodarsky changed the title Add a Time Range input on TSVB [TSVB] Add a Time Range input May 28, 2019
@alexwizp alexwizp changed the title [TSVB] Add a Time Range input [TSVB] Introducing Timerange Data Mode for Metric Style Visualizations May 29, 2019
alexwizp
alexwizp reviewed May 30, 2019
View reviewed changes
Comment thread src/legacy/core_plugins/metrics/public/components/index_pattern.js Outdated
alexwizp
alexwizp reviewed May 30, 2019
View reviewed changes
Comment thread src/legacy/core_plugins/metrics/public/components/index_pattern.js
alexwizp
alexwizp reviewed May 30, 2019
View reviewed changes
Comment thread src/legacy/core_plugins/metrics/public/components/index_pattern.js Outdated
@elasticmachine

elasticmachine commented May 31, 2019

Copy link
Copy Markdown
Contributor

💚 Build Succeeded

@gospodarsky
gospodarsky requested a review from alexwizp May 31, 2019 14:58
@elasticmachine

elasticmachine commented May 31, 2019

Copy link
Copy Markdown
Contributor

💚 Build Succeeded

@elasticmachine

elasticmachine commented Jul 1, 2019

Copy link
Copy Markdown
Contributor

💔 Build Failed

@spalger spalger added the v7.4.0 label Jul 3, 2019
@elasticmachine

elasticmachine commented Jul 4, 2019

Copy link
Copy Markdown
Contributor

💔 Build Failed

@elasticmachine

elasticmachine commented Jul 5, 2019

Copy link
Copy Markdown
Contributor

💔 Build Failed

@elasticmachine

elasticmachine commented Jul 8, 2019

Copy link
Copy Markdown
Contributor

💚 Build Succeeded

@elasticmachine

elasticmachine commented Jul 11, 2019

Copy link
Copy Markdown
Contributor

💔 Build Failed

@elasticmachine

elasticmachine commented Jul 12, 2019

Copy link
Copy Markdown
Contributor

💚 Build Succeeded

@elasticmachine

elasticmachine commented Jul 15, 2019

Copy link
Copy Markdown
Contributor

💔 Build Failed

@elasticmachine

elasticmachine commented Jul 16, 2019

Copy link
Copy Markdown
Contributor

💔 Build Failed

@elasticmachine

elasticmachine commented Jul 18, 2019

Copy link
Copy Markdown
Contributor

💚 Build Succeeded

@gospodarsky
gospodarsky merged commit eeb92b4 into elastic:master Jul 18, 2019
@gospodarsky gospodarsky mentioned this pull request Jul 18, 2019
Merged
gospodarsky pushed a commit that referenced this pull request Jul 18, 2019
Artyom Gospodarsky
[TSVB] Introducing Timerange Data Mode for Metric Style Visualizations (
7e740ae 
#37185) (#41440)

* Add Time Range input to Panel on TSVB

* Disable YesNo Component

* Remove redundant translations

* add timerange_data_modes

* Fix reviews

* BE part

* add new ui restriction

* add new ui restriction - ui part

* override index pattern issue

* Take TIME_RANGE_MODE_KEY out

* Make TimeRange more stable

* Small improvements

* Small improvements - BE part

* Small improvements - BE part

* Small improvements - BE part

* Bind uiRestrictions to UI of the Time Range

* Get rid of context consumers

* Remove console.log

* Add translation and refactor a little

* Pretty Panel config
This was referenced Jul 19, 2019
Merged
Closed
@alexwizp alexwizp mentioned this pull request Aug 9, 2019
Closed
@timroes timroes mentioned this pull request Dec 2, 2019
Closed
@alexwizp alexwizp mentioned this pull request Aug 4, 2020
Closed
@alexwizp alexwizp mentioned this pull request Jun 30, 2021
Closed
patrykkopycinski pushed a commit to patrykkopycinski/kibana that referenced this pull request May 6, 2026
Artyom Gospodarsky
[TSVB] Introducing Timerange Data Mode for Metric Style Visualizations (
ff377d2 
elastic#37185)

* Add Time Range input to Panel on TSVB

* Disable YesNo Component

* Remove redundant translations

* add timerange_data_modes

* Fix reviews

* BE part

* add new ui restriction

* add new ui restriction - ui part

* override index pattern issue

* Take TIME_RANGE_MODE_KEY out

* Make TimeRange more stable

* Small improvements

* Small improvements - BE part

* Small improvements - BE part

* Small improvements - BE part

* Bind uiRestrictions to UI of the Time Range

* Get rid of context consumers

* Remove console.log

* Add translation and refactor a little

* Pretty Panel config
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@alexwizp alexwizp alexwizp approved these changes
@markov00 markov00 Awaiting requested review from markov00
@emmacunningham emmacunningham Awaiting requested review from emmacunningham
+1 more reviewer
@timroes timroes timroes approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

@timroes timroes

@markov00 markov00

@gospodarsky gospodarsky

@alexwizp alexwizp

@rayafratkina rayafratkina

Labels

Feature:TSVB TSVB (Time Series Visual Builder) release_note:enhancement v7.4.0 v8.0.0

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Visual Builder interval auto for Top N, Metric and Gauge

8 participants

@gospodarsky @elasticmachine @rayafratkina @polyfractal @timroes @alexwizp @spalger @markov00