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

scipy ImportError on multiple platforms #1138

Closed
2 tasks done
marcellocirelli opened this issue Apr 3, 2024 · 3 comments · Fixed by #1139
Closed
2 tasks done

scipy ImportError on multiple platforms #1138

marcellocirelli opened this issue Apr 3, 2024 · 3 comments · Fixed by #1139
Labels
bug Something isn't working

Comments

@marcellocirelli
Copy link

Describe the bug

When attempting to Infer, the process is cancelled with the following error message:

[10:53:57] ERROR [10:53:57] cannot import name 'kaiser' from 'scipy.signal' gui.py:764
(C:\Users\Marcello\AppData\Roaming\so-vits-svc-fork\venv\lib\site-packages\scipy\signal_
init_.py)
Traceback (most recent call last):
File
"C:\Users\Marcello\AppData\Roaming\so-vits-svc-fork\venv\lib\site-packages\so_vits_svc_fo
rk\gui.py", line 723, in main
from so_vits_svc_fork.inference.main import infer
File
"C:\Users\Marcello\AppData\Roaming\so-vits-svc-fork\venv\lib\site-packages\so_vits_svc_fo
rk\inference\main.py", line 14, in
from so_vits_svc_fork.inference.core import RealtimeVC, RealtimeVC2, Svc
File
"C:\Users\Marcello\AppData\Roaming\so-vits-svc-fork\venv\lib\site-packages\so_vits_svc_fo
rk\inference\core.py", line 18, in
from ..modules.synthesizers import SynthesizerTrn
File
"C:\Users\Marcello\AppData\Roaming\so-vits-svc-fork\venv\lib\site-packages\so_vits_svc_fo
rk\modules\synthesizers.py", line 13, in
from so_vits_svc_fork.modules.decoders.mb_istft import (
File
"C:\Users\Marcello\AppData\Roaming\so-vits-svc-fork\venv\lib\site-packages\so_vits_svc_fo
rk\modules\decoders\mb_istft_init_.py", line 1, in
from ._generators import (
File
"C:\Users\Marcello\AppData\Roaming\so-vits-svc-fork\venv\lib\site-packages\so_vits_svc_fo
rk\modules\decoders\mb_istft_generators.py", line 11, in
from .pqmf import PQMF
File
"C:\Users\Marcello\AppData\Roaming\so-vits-svc-fork\venv\lib\site-packages\so_vits_svc_fo
rk\modules\decoders\mb_istft_pqmf.py", line 9, in
from scipy.signal import kaiser
ImportError: cannot import name 'kaiser' from 'scipy.signal'
(C:\Users\Marcello\AppData\Roaming\so-vits-svc-fork\venv\lib\site-packages\scipy\signal_
init
.py)

To Reproduce

  1. Install so-vits-svc-fork using option 1, 2 or 3 natively or within Anaconda
  2. Select model & config
  3. Select input file, click on Infer

Additional context

The same error message appears on every operating system I've tried.

Version

4.1.58

Platform

Windows 11 / MacOS Sonoma 14.1.1 / Anaconda3

Code of Conduct

  • I agree to follow this project's Code of Conduct.

No Duplicate

  • I have checked existing issues to avoid duplicates.
@marcellocirelli marcellocirelli added the bug Something isn't working label Apr 3, 2024
@4ll0w3v1l
Copy link

@marcellocirelli try changing

from scipy.signal import kaiser
to
from scipy.signal import kaiserord
in this file
modules\decoders\mb_istft_pqmf.py

seems like scipy 0.14.0 is the last version that had "kaiser" as the module name
as far as i checked, it is the same method, though i might be wrong

@Mr-Mathias-F
Copy link

@marcellocirelli try changing

from scipy.signal import kaiser
to
from scipy.signal import kaiserord
in this file
modules\decoders\mb_istft_pqmf.py

seems like scipy 0.14.0 is the last version that had "kaiser" as the module name
as far as i checked, it is the same method, though i might be wrong

Just tried it, it worked for me

@marcellocirelli
Copy link
Author

@marcellocirelli try changing

from scipy.signal import kaiser to from scipy.signal import kaiserord in this file modules\decoders\mb_istft_pqmf.py

seems like scipy 0.14.0 is the last version that had "kaiser" as the module name as far as i checked, it is the same method, though i might be wrong

Worked for me as well. Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants