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

Edit colab notebooks #24

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

Conversation

amrzv
Copy link

@amrzv amrzv commented Jan 19, 2021

Description

All changes were made in quick_demo.ipynb and quick_demo_tdlr.ipynb:

  • removed unnecessary import time
  • removed second import glob in _tdlr.ipynb
  • used f-strings
  • prettified a little (according pep8)

How Has This Been Tested?

Tested in google colab by "run all", no new errors appeared, notebook works as expected.
P.S: outputs left the same.

Checklist:

  • I have signed the Adobe Open Source CLA.
  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

@manu-polsani
Copy link

While running the notebook, Ihave got this error. Can you help me in resolving this?
<>:4: SyntaxWarning: "is not" with a literal. Did you mean "!="?
<>:4: SyntaxWarning: "is not" with a literal. Did you mean "!="?
:4: SyntaxWarning: "is not" with a literal. Did you mean "!="?
ains = [item for item in ains if item is not 'tmp.wav']
Loaded the voice encoder model on cuda in 7.96 seconds.
Processing audio file M6_04_16k.wav
0 out of 0 are in this portion

TypeError Traceback (most recent call last)
in <cell line: 6>()
16 c = AutoVC_mel_Convertor('examples')
17
---> 18 au_data_i = c.convert_single_wav_to_autovc_input(audio_filename=os.path.join('examples', ain),
19 autovc_model_path=opt_parser.load_AUTOVC_name)
20 au_data += au_data_i

1 frames
/content/MakeItTalk/src/autovc/retrain_version/vocoder_spec/extract_f0_func.py in extract_f0_func_audiofile(audio_file, gender)
95 def extract_f0_func_audiofile(audio_file, gender='M'):
96 floor_sp, ceil_sp = -80, 30
---> 97 mel_basis = mel(16000, 1024, fmin=90, fmax=7600, n_mels=80).T
98 min_level = np.exp(-100 / 20 * np.log(10))
99 b, a = butter_highpass(30, 16000, order=5)

TypeError: mel() takes 0 positional arguments but 2 positional arguments (and 3 keyword-only arguments) were given

@Enes-s
Copy link

Enes-s commented Sep 29, 2023

While running the notebook, Ihave got this error. Can you help me in resolving this?

<>:4: SyntaxWarning: "is not" with a literal. Did you mean "!="?
<>:4: SyntaxWarning: "is not" with a literal. Did you mean "!="?
:4: SyntaxWarning: "is not" with a literal. Did you mean "!="?
ains = [item for item in ains if item is not 'tmp.wav']
Loaded the voice encoder model on cuda in 7.96 seconds.
Processing audio file M6_04_16k.wav
0 out of 0 are in this portion
TypeError Traceback (most recent call last) in <cell line: 6>() 16 c = AutoVC_mel_Convertor('examples') 17 ---> 18 au_data_i = c.convert_single_wav_to_autovc_input(audio_filename=os.path.join('examples', ain), 19 autovc_model_path=opt_parser.load_AUTOVC_name) 20 au_data += au_data_i

1 frames /content/MakeItTalk/src/autovc/retrain_version/vocoder_spec/extract_f0_func.py in extract_f0_func_audiofile(audio_file, gender) 95 def extract_f0_func_audiofile(audio_file, gender='M'): 96 floor_sp, ceil_sp = -80, 30 ---> 97 mel_basis = mel(16000, 1024, fmin=90, fmax=7600, n_mels=80).T 98 min_level = np.exp(-100 / 20 * np.log(10)) 99 b, a = butter_highpass(30, 16000, order=5)

TypeError: mel() takes 0 positional arguments but 2 positional arguments (and 3 keyword-only arguments) were given

is this problem solved ?

@amrzv
Copy link
Author

amrzv commented Jan 28, 2024

The notebook is still broken
image

AttributeError: _3D

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.

4 participants