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

Feat/plotly express #216

Closed
wants to merge 33 commits into from

Conversation

pradyot-09
Copy link
Collaborator

@pradyot-09 pradyot-09 commented Jun 7, 2022

  • The histograms, heatmaps and comparisons have been replaced with interactive Plotly graphs. Plotly.js is used to build the graphs on the go from json.
  • Initial tests show that plotly reports are smaller in size compared to matplotlib and the takes way less time for report generation compared to matplotlib.
  • use parameter 'online_report' to use plotly.js from cdn server and use report online. Else, plotly.js is embedded in the report and can be used offline too.

newplot

newplot (3)

newplot (2)

closes #164

sbrugman and others added 20 commits May 20, 2022 20:42
aggregate alerts per feature (ordered from descending)
click the feature name to directly switch to that feature
closes ing-bank#202
Explicitly choose the rgb values for red, as matplotlib's "r" does not coincide with css "red"

[link]: https://matplotlib.org/3.5.0/tutorials/colors/colors.html#comparison-between-x11-css4-and-xkcd-colors
- update SEA description
- full and partial reference for synthetic data streams
- hyperplane logistic regression example
- Performance speedup in histogram summation
  o Replace for loop with np.sum
- Protection against outliers in sparse histograms
  o Protect against extreme outliers in sparse histograms by avoiding the use
    of its dense representation where possible.
- Working analysis function unit tests
  o update of test_report_generator
  o section_generator now passes np arrays to plot_bars_b64, no longer pd.Series.
- Added crop_range option to get_consistent_numpy_1dhists function
  o return a cropped range version of the histogram, between 5-95% quantiles.
  o Plot cropped histograms in histogram section of report.
- Bump up versions of histogrammar
@pradyot-09 pradyot-09 added enhancement New feature or request reporting Visualisation of the monitoring results labels Jun 7, 2022
@pradyot-09 pradyot-09 requested review from sbrugman and mbaak June 7, 2022 14:46
@pradyot-09 pradyot-09 self-assigned this Jun 7, 2022
@pradyot-09 pradyot-09 linked an issue Jun 8, 2022 that may be closed by this pull request
popmon/visualization/histogram_section.py Outdated Show resolved Hide resolved
popmon/visualization/templates/core.html Outdated Show resolved Hide resolved
popmon/visualization/utils.py Outdated Show resolved Hide resolved
popmon/visualization/utils.py Outdated Show resolved Hide resolved
requirements.txt Outdated Show resolved Hide resolved
@sbrugman
Copy link
Collaborator

Quick wins

Improvements

Follow-up features:

  • By default, the plotly express JS could be integrally included in the HTML file. The CDN option could be a new feature, as bootstrap could also be hosted by a third party. (the online_report option)
  • Cross filter functionality would be awesome, where selecting a time-window on one plot also updates the others, e.g. https://plotly.github.io/plotly.js-crossfilter.js/

@sbrugman sbrugman deleted the branch ing-bank:develop June 14, 2022 09:44
@sbrugman sbrugman closed this Jun 14, 2022
@sbrugman sbrugman reopened this Jun 14, 2022
@sbrugman
Copy link
Collaborator

sbrugman commented Jul 5, 2022

Closing in favour of #225

@sbrugman sbrugman closed this Jul 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request reporting Visualisation of the monitoring results
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Interactive plots
4 participants