Multiple values for PitchShift and TimeStretch deformers

bmcfee / muda

A library for augmenting annotated audio data

ISC License

230 stars 33 forks source link

Closed bmcfee closed 7 years ago

bmcfee commented 7 years ago

This PR implements #41. PitchShift can now accept an array of shift values, in addition to a single scalar. Similarly for TimeStretch.

bmcfee commented 7 years ago

Testing this guy also revealed a small bug in the test coverage.

I added serialization round-trip tests to all deformation objects, so hopefully everything should work as expected.

bmcfee commented 7 years ago

@justinsalamon i think this one should be good to go. Care to give it a once-over to make sure it meets your needs?

justinsalamon commented 7 years ago

Just had a look, LGTM.