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

Vits2 prototype #3355

Closed
wants to merge 4 commits into from
Closed

Vits2 prototype #3355

wants to merge 4 commits into from

Conversation

p0p4k
Copy link
Contributor

@p0p4k p0p4k commented Dec 2, 2023

continuing where i left earlier, adding some test cases.

  • to do -> fix test cases to take in mel instead of linear spectrogram (if anywhere)

@erogol
Copy link
Member

erogol commented Dec 4, 2023

Looks like a bug (from CI)

======================================================================
ERROR: test_d_vector_forward (tests.tts_tests.test_vits2.TestVits2)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/runner/work/TTS/TTS/tests/tts_tests/test_vits2.py", line 214, in test_d_vector_forward
    model = Vits2.init_from_config(config, verbose=False).to(device)
  File "/home/runner/work/TTS/TTS/TTS/tts/models/vits2.py", line 1862, in init_from_config
    return Vits2(new_config, ap, tokenizer, speaker_manager, language_manager)
  File "/home/runner/work/TTS/TTS/TTS/tts/models/vits2.py", line 652, in __init__
    self.use_transformer_flow_layer = self.args.use_transformer_flow_layer
  File "/opt/hostedtoolcache/Python/3.9.18/x64/lib/python3.9/site-packages/coqpit/coqpit.py", line 626, in __getattribute__
    value = super().__getattribute__(arg)
AttributeError: 'Vits2Args' object has no attribute 'use_transformer_flow_layer'

@p0p4k p0p4k marked this pull request as draft December 27, 2023 06:05
Copy link

stale bot commented Jan 28, 2024

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. You might also look our discussion channels.

@stale stale bot added the wontfix This will not be worked on but feel free to help. label Jan 28, 2024
@stale stale bot closed this Feb 4, 2024
@Marioando
Copy link

Changing use_transformer_flow to use_transformer_flow_layer in Vits2Args get arround of this but the I get this error :
/opt/conda/lib/python3.10/site-packages/TTS/tts/layers/vits2/transformer.py:477, in ConditionalRelativePositionTransformer.init(self, in_channels, out_channels, hidden_channels, hidden_channels_ffn, num_heads, num_layers, kernel_size, dropout_p, rel_attn_window_size, input_length, layer_norm_type, cond_channels, cond_layer_idx)
475 super().init()
476 self.cond_channels = cond_channels
--> 477 if cond_layer_idx < 0 or cond_layer_idx >= num_layers:
478 raise ValueError(" [!] cond_layer_idx should be in [0, num_layers)")
479 self.cond_layer_idx = cond_layer_idx

TypeError: '<' not supported between instances of 'NoneType' and 'int'

@pivolan
Copy link

pivolan commented Nov 7, 2024

is it work for single gpu train and inference?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
wontfix This will not be worked on but feel free to help.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants