Explicit round to solve issue #94

asteroid-team / torch-audiomentations

Fast audio data augmentation in PyTorch. Inspired by audiomentations. Useful for deep learning.

MIT License

969 stars 88 forks source link

Closed MorenoLaQuatra closed 2 years ago

MorenoLaQuatra commented 2 years ago

Rounded value using math.ceil function to solve error in size for background noise.

iver56 commented 2 years ago

Hi :) Thanks for the PR.

Could you please add a unit test that covers this case (that failed before, but passes after your patch)? That will help against regressions

iver56 commented 2 years ago

This PR, when merged, closes #94

iver56 commented 2 years ago

This fix seems solid. I'll take care of the test.