JuliaDynamics / TransitionsInTimeseries.jl

Transition Indicators / Early Warning Signals / Regime Shifts / Change Point Detection

MIT License

18 stars 5 forks source link

Improvement of the visualization code #68

Closed JanJereczek closed 7 months ago

JanJereczek commented 7 months ago

Major changes

visualization of SlidingWindowResults changed to one indicator = one row. Gives the following:
visualization now supports SegmentedWindowResults and gives the following: . Note that the visualisation of the surrogate change metrics is a bit tricky here, since it is a single value per segment. Suggestions to improve the current solution (scatterplot) are welcome!
visualization for SlidingWindowResults and SegmentedWindowResults rely on same functions so that maintenance easier

Minors

tutorial.jl adapted to make use of the new visualization
tutorial.jl adapted to include segmented windows in the description
TITvisualization.jl changed for less ambiguous name TransitionVisualizations.jl
fix references in docs

codecov-commenter commented 7 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

:exclamation: No coverage uploaded for pull request base (main@9943887). Click here to learn what that means.

Additional details and impacted files

```diff @@ Coverage Diff @@ ## main #68 +/- ## ======================================= Coverage ? 45.22% ======================================= Files ? 17 Lines ? 356 Branches ? 0 ======================================= Hits ? 161 Misses ? 195 Partials ? 0 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

Datseris commented 7 months ago

also, please add a top-level codecov.yml file that ingores the files with plotting from the test coverage so that we don't get reports of low coverage, similar to here: https://github.com/JuliaDynamics/Attractors.jl/blob/main/codecov.yml

JanJereczek commented 7 months ago

My answer above points out that I was missing the use of contents instead of content, which you did not comment. But ok, now it is implemented and should be looking good.