Skip to content

Comments

Library fixes#10

Open
giannifloriddia wants to merge 3 commits intoLemonATsu:masterfrom
giannifloriddia:library-fixes
Open

Library fixes#10
giannifloriddia wants to merge 3 commits intoLemonATsu:masterfrom
giannifloriddia:library-fixes

Conversation

@giannifloriddia
Copy link

Changes from Original

This fork includes the following updates for compatibility with modern Python and libraries:

  • Removed external pYIN vamp plugin dependency: Now uses LibROSA's built-in pyin function for pitch detection
  • Removed pypropack dependency: Replaced with scipy.sparse.linalg.svds for truncated SVD
  • Updated scipy imports: Fixed deprecated scipy.signal.hamming to scipy.signal.windows.hamming
  • Updated librosa API: Fixed librosa.load() to use keyword arguments
  • Simplified dependencies: Only requires librosa and scipy (NumPy is installed as a dependency of these)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant