open-mmlab / Amphion

Amphion (/æmˈfaɪən/) is a toolkit for Audio, Music, and Speech Generation. Its purpose is to support reproducible research and help junior researchers and engineers get started in the field of audio, music, and speech generation research and development.
https://openhlt.github.io/amphion/
MIT License
4.41k stars 373 forks source link

[BUG]: missing reference to `modules.generic.conv` in `modules/encoder/conv_encoder.py` #178

Closed karljeon44 closed 4 months ago

karljeon44 commented 4 months ago

Describe the bug

(A clear and concise description of what the bug is.)

modules/encoder/conv_encoder.py attempts to import from modules.generic package, as follows:

import torch
...
from modules.generic.conv import Conv1d

but generic is clearly missing from inside modules

Perhaps you meant conv.py inside wavenet??

How To Reproduce

Steps to reproduce the behavior:

  1. Config/File changes: N/A
  2. Run command:
    • Run any file that imports from conv_encoder, for instance:
      from modules.encoder.conv_encoder import ConvEncoder
  3. See error:

Screenshot 2024-04-08 at 4 33 41 PM

Expected behavior

(A clear and concise description of what you expected to happen.)

Screenshots

(If applicable, add screenshots to help explain your problem.)

Environment Information

Additional context

(Add any other context about the problem here.)

Adorable-Qin commented 4 months ago

Hi @karljeon44 ! Thank you very much for your feedback about Amphion!

conv_encoder.py is a deprecated file left over from the old implementation, that's why it contains invalid reference. I've removed it to avoid unintended import operation which may cause crash. If you want to check our self-defined Conv1d function, you may find it at /modules/general/utils.py.

Feel free to ask if you have any other question, we are delight to help!