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

Some text inputs throw expanded size of the tensor #14

Open
Cabeda opened this issue Feb 17, 2024 · 3 comments
Open

Some text inputs throw expanded size of the tensor #14

Cabeda opened this issue Feb 17, 2024 · 3 comments
Labels
bug Something isn't working

Comments

@Cabeda
Copy link

Cabeda commented Feb 17, 2024

First of all thanks a lot for providing this package!

I was doing some tests for large inputs and found that when trying to send a large string (i.e. len >1000) it would throw the following error:

The expanded size of the tensor (546) must match the existing size (512) at non-singleton dimension 1.  Target sizes: [1, 546].  Tensor sizes: [1, 512]

In terms of code, I used all the defaults:

out = my_tts.inference(
    text, 
    output_wav_file="test_nb.wav"
)
@sidharthrajaram
Copy link
Owner

Thanks for the note @Cabeda , are you observing this issue with the latest release of the package?

And if it's okay, could you share the text that triggered the tensor size error? (just so it's easier to test on the same footing)

@sidharthrajaram sidharthrajaram added the bug Something isn't working label Feb 17, 2024
@Cabeda
Copy link
Author

Cabeda commented Feb 18, 2024

Hi, I'm using the version 0.1.6. For the test, I'm using the text from the book Martian: Lost Sols (it's a free one).

Lost_sols.txt

@Cabeda
Copy link
Author

Cabeda commented Feb 18, 2024

Another test that throwed the same error. I've split the text to blocks of 500 chars. The first one worked fine, the second one threw the error of this issue.

However, the NSA assisted us with software they declined to explain and we now have these additional log entries. ATTACHMENT: LOG ENTRY: SOL 488 Well. Fuck me raw. I navigated my way around the dust storm, so I thought the “pain in the ass” portion of my journey was over. But no, no. Apparently, Mars isn’t done handing me bullshit. There I was, driving along in Meridiani Planum. Smooth sailing from here on out – or so I thought. The terrain was rough but nothing the rover couldn’t handle.

However, if I further divide this text in half and run separately, it succeeds

@Cabeda Cabeda changed the title Large input of text throws error Some text inputs throw expanded size of the tensor Feb 19, 2024
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

2 participants