Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

implement reset! for FIRInterpolator and tests #626

Merged
merged 3 commits into from
Feb 10, 2025

Conversation

wheeheee
Copy link
Member

@wheeheee wheeheee commented Feb 8, 2025

Also bumps version to 0.8.2. Or I don't know, should 0.8.1 be pulled?

Copy link

codecov bot commented Feb 8, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 98.13%. Comparing base (4a88779) to head (bf10e67).
Report is 1 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #626   +/-   ##
=======================================
  Coverage   98.13%   98.13%           
=======================================
  Files          19       19           
  Lines        3277     3277           
=======================================
  Hits         3216     3216           
  Misses         61       61           

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

@wheeheee
Copy link
Member Author

wheeheee commented Feb 8, 2025

Forgot to mention, fixes #625

@wheeheee wheeheee linked an issue Feb 8, 2025 that may be closed by this pull request
@martinholters
Copy link
Member

Also bumps version to 0.8.2. Or I don't know, should 0.8.1 be pulled?

A quick 0.8.2 with the regression fixed seems appropriate.

@wheeheee wheeheee merged commit d08699a into master Feb 10, 2025
11 checks passed
@wheeheee wheeheee deleted the reset_FIRInterpolator branch February 10, 2025 12:03
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

resample with dims is broken
2 participants