-
Notifications
You must be signed in to change notification settings - Fork 661
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
Compose sox_effects and torchaudio.transforms #889
Comments
Hello, To start, I just tried to make your RuntimeError: torchaudio.sox_effects.sox_effects.apply_effects_tensor requires module: torchaudio._torchaudio I carefully read the source code and couldn't find the RuntimeError: Tried to instantiate class 'torchaudio.TensorSignal', but it does not exist! Ensure that it is registered via torch::class_ |
Hi @leocances Which OS and version of torchaudio are you using? sox-related functionalities are only available in Linux/macOS distribution. Windows is not supported. |
I do not think adding it to transforms provide value for the maintenance cost. It is just a pass-through to the function form, which users can do as a part of their own transform easily. |
@leocances FYI: I am adding on-the-fly sample rate conversion here #816. Please give a +1 if you think this is useful. |
Hi @mthrok I am new to the PyTorch source code in general, and I may have missed something. |
@leocances |
So, this did the trick.
Thank a lot |
I'd like to suggest a way of composing
sox_effects
withtorchaudio.transforms
. The latter is made oftorch.nn.Module
so they can be composed usingtorch.nn.Sequential
. This would also follow the design pattern we established in torchaudio with transforms/functionals.Would it be valuable to offer something like
SoxEffectsTransform
directly in torchaudio? It is currently in the docstring.The text was updated successfully, but these errors were encountered: