Skip to content

Adding support for negative filter ratios for TSVB#152053

Merged
simianhacker merged 2 commits intoelastic:mainfrom
simianhacker:issue-150738-support-negative-ratios
Feb 28, 2023
Merged

Adding support for negative filter ratios for TSVB#152053
simianhacker merged 2 commits intoelastic:mainfrom
simianhacker:issue-150738-support-negative-ratios

Conversation

@simianhacker
Copy link
Copy Markdown
Member

Summary

This PR fixes #150738 by adding support for negative denominators for TSVB's filter ratios.

Checklist

@simianhacker simianhacker marked this pull request as ready for review February 24, 2023 01:58
@simianhacker simianhacker requested a review from a team as a code owner February 24, 2023 01:58
@simianhacker simianhacker added release_note:fix v8.8.0 Feature:TSVB TSVB (Time Series Visual Builder) labels Feb 24, 2023
@stratoula stratoula added the Team:Visualizations Team label for Lens, elastic-charts, Graph, legacy editors (TSVB, Visualize, Timelion) t// label Feb 28, 2023
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/kibana-visualizations @elastic/kibana-visualizations-external (Team:Visualizations)

Copy link
Copy Markdown
Contributor

@stratoula stratoula left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changes LGTM

@kibana-ci
Copy link
Copy Markdown

💚 Build Succeeded

Metrics [docs]

Unknown metric groups

ESLint disabled line counts

id before after diff
securitySolution 428 430 +2

Total ESLint disabled count

id before after diff
securitySolution 506 508 +2

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@simianhacker simianhacker merged commit a6d988a into elastic:main Feb 28, 2023
@kibanamachine kibanamachine added the backport:skip This PR does not require backporting label Feb 28, 2023
bmorelli25 pushed a commit to bmorelli25/kibana that referenced this pull request Mar 10, 2023
## Summary

This PR fixes elastic#150738 by adding support for negative denominators for
TSVB's filter ratios.

### Checklist

- [X] [Unit or functional
tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)
were updated or added to match the most common scenarios

Co-authored-by: Stratoula Kalafateli <efstratia.kalafateli@elastic.co>
@simianhacker simianhacker deleted the issue-150738-support-negative-ratios branch April 17, 2024 15:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:skip This PR does not require backporting Feature:TSVB TSVB (Time Series Visual Builder) release_note:fix Team:Visualizations Team label for Lens, elastic-charts, Graph, legacy editors (TSVB, Visualize, Timelion) t// v8.8.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

TSVB Filter ratio denominator not handling negative values

5 participants