Add plotly plots - Githubissues

GEMDAT-repos / GEMDAT

Python toolkit for molecular dynamics analysis

https://gemdat.readthedocs.io

Apache License 2.0

21 stars 3 forks source link

Add plotly plots #313

Closed stefsmeets closed 1 month ago

stefsmeets commented 1 month ago

This PR adds missing plotly plots. It also adds regression testing for the plotly plots. This improves coverage by about 10% to 85%.

Closes #302 Closes #312

Todo

[x] radial_distribution
[ ] shape -> later
[x] energy_along_path
[x] rectilinear
[x] bond_length_distribution
[x] autocorrelation
[x] Refactor shared bond_length_distribution code
[x] Add image compare for all plotly plots
[x] Set default plots to plotly were available
[x] Check why plotly plots cannot be reproduced in CI
[x] Regenerate reference images in clean virtual env

stefsmeets commented 1 month ago

Test fail seems related to #312

stefsmeets commented 1 month ago

Plots are failing because of font rendering issues, seems like the CI uses a different font than what I have locally.