Closed bmcfee closed 6 months ago
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 88.32%. Comparing base (
7997fdf
) to head (76d9500
).
:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Test failure is unrelated, appears to be a resurgence of a heisenbug in bsseval on osx. 🤦
I added one more test here to cover the case where the input length is shorter than the provided input.
Otherwise I think this is good to go. We shouldn't let the OSX failure block merging this PR.
Lol, rerunning the tests passed this time with no changes. OSX is great.
This PR fixes #371 by separating out the slicing logic for time_frequency generation.
I've unskipped the test that was previously failing and things seem good to go.