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

Annotation Styles for Time Series Annotations #5437

Merged
merged 1 commit into from
Aug 1, 2018

Conversation

JamshedRahman
Copy link
Contributor

@JamshedRahman JamshedRahman commented Jul 20, 2018

This PR adds styling functionality to time series annotations. The 2 styles added here are "Show Markers" and "Hide Lines". Specifically this has been shown to be useful when overlaying an annotation layer that is filtered by some condition (e.g. above Upper Threshold / below Lower Threshold). This presents a view where data points that are exceeding normal values show up as Red Dots and can be identified as anomalies / outliers.

annotationmenu

annotationstyle

@codecov-io
Copy link

Codecov Report

Merging #5437 into master will decrease coverage by 0.01%.
The diff coverage is 0%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #5437      +/-   ##
==========================================
- Coverage    59.1%   59.08%   -0.02%     
==========================================
  Files         372      372              
  Lines       23742    23750       +8     
  Branches     2758     2762       +4     
==========================================
  Hits        14033    14033              
- Misses       9694     9702       +8     
  Partials       15       15
Impacted Files Coverage Δ
superset/assets/src/visualizations/nvd3_vis.js 8.93% <0%> (-0.04%) ⬇️
...rc/explore/components/controls/AnnotationLayer.jsx 22.66% <0%> (-0.7%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 41447e8...0e51c21. Read the comment docs.

@mistercrunch
Copy link
Member

@fabianmenges

Copy link
Contributor

@fabianmenges fabianmenges left a comment

Choose a reason for hiding this comment

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

LGTM

@JamshedRahman
Copy link
Contributor Author

@mistercrunch can we please get this merged since Fabian's approved? Thanks in advance.

@graceguo-supercat graceguo-supercat merged commit 9831730 into apache:master Aug 1, 2018
wenchma pushed a commit to wenchma/incubator-superset that referenced this pull request Nov 16, 2018
@mistercrunch mistercrunch added 🏷️ bot A label used by `supersetbot` to keep track of which PR where auto-tagged with release labels 🚢 0.28.0 labels Feb 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🏷️ bot A label used by `supersetbot` to keep track of which PR where auto-tagged with release labels 🚢 0.28.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants