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

ImportError: cannot import name 'kaiser' from 'scipy.signal' #430

Open
unilight opened this issue Jun 27, 2024 · 1 comment
Open

ImportError: cannot import name 'kaiser' from 'scipy.signal' #430

unilight opened this issue Jun 27, 2024 · 1 comment
Labels
bug Something isn't working

Comments

@unilight
Copy link

unilight commented Jun 27, 2024

I encountered the following error:

  File "/data/group1/z44476r/Experiments/ppg-vc/tools/venv/lib/python3.10/site-packages/parallel_wavegan/layers/__init__.py", line 2, in <module>
    from .pqmf import *  # NOQA
  File "/data/group1/z44476r/Experiments/ppg-vc/tools/venv/lib/python3.10/site-packages/parallel_wavegan/layers/pqmf.py", line 11, in <module>
    from scipy.signal import kaiser
ImportError: cannot import name 'kaiser' from 'scipy.signal' (/data/group1/z44476r/Experiments/ppg-vc/tools/venv/lib/python3.10/site-packages/scipy/signal/__init__.py)

The problem comes from this line:
https://github.com/kan-bayashi/ParallelWaveGAN/blob/master/parallel_wavegan/layers/pqmf.py#L11
I am not sure when scipy changed it, but it should be from scipy.signal.windows import kaiser. (tested on scipy version 1.14.0)

@spencerchubb
Copy link

I also ran into this

I looked through the scipy history, and it looks like they made the change from 1.0.0 to 1.0.1

For now I am working around this by directly change the code of parallel_wavegan

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

No branches or pull requests

3 participants