Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Fix sample rate type annotations #1789

Merged
merged 4 commits into from Dec 21, 2023
Merged

Fix sample rate type annotations #1789

merged 4 commits into from Dec 21, 2023

Conversation

bmcfee
Copy link
Member

@bmcfee bmcfee commented Dec 21, 2023

Reference Issue

Fixes #1748

What does this implement/fix? Explain your changes.

This PR completes the type annotation corrections started in #1753

Any other comments?

Nothing left to do here I think. Once it passes CR, I'll merge.

@bmcfee bmcfee added documentation Issues relating to docstrings, examples, and documentation build testing Issues with our test design and continuous integration services labels Dec 21, 2023
@bmcfee bmcfee added this to the 0.10.2 milestone Dec 21, 2023
Copy link

codecov bot commented Dec 21, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (07d7295) 98.77% compared to head (0607375) 98.77%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1789   +/-   ##
=======================================
  Coverage   98.77%   98.77%           
=======================================
  Files          34       34           
  Lines        4661     4661           
=======================================
  Hits         4604     4604           
  Misses         57       57           
Flag Coverage Δ
unittests 98.77% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

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

@bmcfee bmcfee merged commit 6e0d0f5 into main Dec 21, 2023
12 checks passed
@bmcfee bmcfee deleted the fix/sr_type branch December 21, 2023 20:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Issues relating to docstrings, examples, and documentation build testing Issues with our test design and continuous integration services
Development

Successfully merging this pull request may close these issues.

librosa.load returns wrongly-typed int object
2 participants