Simplify and expand filtering tests for bboxes

sfmig commented 1 month ago

Description

What is this PR

[ ] Bug fix
[x] Addition of a new feature
[ ] Other

Why is this PR needed? We want to ensure the filtering methods work for bboxes datasets.

What does this PR do?

Simplifies filtering tests
- Combines median and savgol filters tests into a single test, to reduce code duplication
- Make test for interpolation more compact
Expands tests to bboxes datasets.

References

Overflow from PR #246

How has this PR been tested?

Tests pass locally and in CI.

Is this a breaking change?

No.

Does this PR require an update to the documentation?

I reviewed the docstrings to check references to poses data but only did a minor change.

I changed some references from "filtering" to "smoothing" for consistency within the docstring, but open to other opinions.

Checklist:

[x] The code has been tested locally
[x] Tests have been added to cover all new functionality
[ ] The documentation has been updated to reflect any changes
[x] The code has been formatted with pre-commit

codecov[bot] commented 1 month ago

Codecov Report

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

Project coverage is 99.77%. Comparing base (96c3cd7) to head (f556832). Report is 1 commits behind head on main.

Additional details and impacted files

```diff @@ Coverage Diff @@ ## main #267 +/- ## ======================================= Coverage 99.77% 99.77% ======================================= Files 14 14 Lines 883 883 ======================================= Hits 881 881 Misses 2 2 ```

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