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

Interactive plots in report #225

Merged
merged 5 commits into from
Jul 5, 2022
Merged

Interactive plots in report #225

merged 5 commits into from
Jul 5, 2022

Conversation

sbrugman
Copy link
Collaborator

Conflicts resolved from pradyot-09:feat/plotly-express

Primary changes:

  • Interactive plots through plotly (replacing matplotlib base64 encoded images)
  • Reduced report file size

@sbrugman sbrugman force-pushed the feat/plotly-express branch from 86a907d to 657d83a Compare June 27, 2022 10:18
@pradyot-09 pradyot-09 force-pushed the feat/plotly-express branch from 84171d2 to 9b3ee72 Compare June 27, 2022 13:07
@sbrugman sbrugman force-pushed the feat/plotly-express branch 3 times, most recently from d919b68 to 3ed9f2b Compare June 27, 2022 15:41
@sbrugman sbrugman changed the base branch from develop to refactor-config June 27, 2022 15:42
@pradyot-09 pradyot-09 assigned pradyot-09 and unassigned pradyot-09 Jun 27, 2022
@sbrugman sbrugman force-pushed the feat/plotly-express branch from 8b38b16 to 4e4423f Compare June 28, 2022 15:55
popmon/config.py Outdated Show resolved Hide resolved
Base automatically changed from refactor-config to develop July 4, 2022 12:42
@sbrugman sbrugman force-pushed the feat/plotly-express branch 2 times, most recently from c740e32 to 70916f2 Compare July 5, 2022 09:45
@sbrugman sbrugman mentioned this pull request Jul 5, 2022
@sbrugman sbrugman force-pushed the feat/plotly-express branch 6 times, most recently from 1ca9c43 to e1b6a46 Compare July 5, 2022 13:40
pradyot-09 and others added 5 commits July 5, 2022 16:41
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.

BREAKING CHANGE: matplotlib-related config is removed
When the `online_report` config parameter is set to True, bootstrap and jquery
are loaded from a CDN as ploty.js. This results in a much smaller report.

Co-authored-by: pradyot-09 <[email protected]>
The report title, primary color and traffic light colors are now configurable.

Co-authored-by: pradyot-09 <[email protected]>
@sbrugman sbrugman force-pushed the feat/plotly-express branch from e1b6a46 to fc77a07 Compare July 5, 2022 14:41
@pradyot-09 pradyot-09 merged commit b5b7626 into develop Jul 5, 2022
@pradyot-09 pradyot-09 deleted the feat/plotly-express branch July 5, 2022 15:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

CDN option for Js imports Label axes for profiles and comparisons are missing
2 participants