FluxML / DataAugmentation.jl

Flexible data augmentation library for machine and deep learning
https://fluxml.ai/DataAugmentation.jl/dev/
MIT License
41 stars 17 forks source link

Fix Cropping #53

Closed mfalt closed 2 years ago

mfalt commented 2 years ago

Fixes https://github.com/lorenzoh/DataAugmentation.jl/issues/52 I have no idea if this is how the bounds are meant to be used, but this change fixes the issue, and seems to be identical to how apply! is implemented later in the file.

lorenzoh commented 2 years ago

Yeah, that must have been it! I think I wrote that conditional as a performance improvement, but forgot that that would also not do the cropping. Thanks for the fix! I'll tag a new release right away.

terasakisatoshi commented 2 years ago

@mfalt @lorenzoh thank you !!!