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 build and librosa numpy dependency errors #1311

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

evezers
Copy link

@evezers evezers commented Apr 27, 2024

Updated librosa function signatures (added missing positional arguments according to librosa/librosa#1521), so librosa and numpy dependency can be flawlessly updated. Also fixed scikit issues according to #1068.

Updating librosa fixes #473, fixes #620, fixes #1016.
Removing Dora fixes #765, fixes #1046, fixes #522, fixes #725, fixes #919.

I also suggest installing into Python virtual environment instead of breaking Arch system Python, so installed as follows:

cd /path/to/UVR
python -m venv venv
source venv/bin/activate
pip install -r requirements.txt
python UVR.py

Tested on Manjaro 23.1.4 and Python 3.11.

@DoiiarX
Copy link

DoiiarX commented May 24, 2024

good job. why not the author merge it? strange

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