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
Call torch.stft directly #531
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Test are failing, see here.
Interesting! It does not fail on my environment 🤔 |
9bca512
to
7afd581
Compare
The fix pytorch/pytorch#33504 isn't part of pytorch 1.4.0 on which the torchaudio test environment depends, see |
We'll merge this PR once Pytorch 1.5.0 has landed. Can you update the title to reflect that? |
Actually, that's not where test environment is defined. it's in https://github.com/pytorch/audio/blob/master/requirements.txt#L1 via https://github.com/pytorch/audio/blob/master/build_tools/travis/install.sh#L51 via https://github.com/pytorch/audio/blob/master/.travis.yml |
Yes, torchaudio master should depend on pytorch master; and the latest torchaudio release should depend on the latest pytorch release. Do you want to a PR to make torchaudio master depend on pytorch nightly? |
7afd581
to
b9230f9
Compare
b9230f9
to
d12b635
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Found the note
# TODO: remove this once https://github.com/pytorch/pytorch/issues/21478 gets solved
And the issue is resolved and we should not need to wrap
torch.stft
anymore.