Add missing tests

[martin-springer]

This is a draft. Mainly to get Codecov updated.

• Code changes are covered by tests
• Code changes have been evaluated for compatibility/integration with TrendAnalysis
• New functions added to __init__.py
• API.rst is up to date, along with other sphinx docs pages
• Example notebooks are rerun and differences in results scrutinized
• Updated changelog

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 75.25%. Comparing base (1f25b2f) to head (f2c48cd).

Additional details and impacted files

@@                        Coverage Diff                        @@
##           aggregated_filters_for_trials     #413      +/-   ##
=================================================================
+ Coverage                          71.70%   75.25%   +3.55%     
=================================================================
  Files                                 13       13              
  Lines                               2820     2825       +5     
=================================================================
+ Hits                                2022     2126     +104     
+ Misses                               798      699      -99     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[mdeceglie]

@martin-springer Thanks for putting this together, a few questions but otherwise looks great.

[mdeceglie]

@martin-springer is it possible to re-trigger the codecov report?

[martin-springer]

@martin-springer is it possible to re-trigger the codecov report?

@mdeceglie I do believe so, but haven't done that much myself yet. Looking through the quick start guide (https://docs.codecov.com/docs/quick-start), you can execute the upload of coverage reports from the command line. This page outlines the process (https://docs.codecov.com/docs/codecov-uploader).

We might want to tweak the github action, to not only run when a new PR is created but also when commits are made to a PR. This should then take care of it. I can take a look at how we would need to change the config for that.