libffcv / ffcv

FFCV: Fast Forward Computer Vision (and other ML workloads!)
https://ffcv.io
Apache License 2.0
2.8k stars 180 forks source link

Bordesf add colorjitter #226

Open lengstrom opened 2 years ago

rtaori commented 1 year ago

Btw what's the status of this? Are there any blockers to it being merged?

andrewilyas commented 1 year ago

Hi @rtaori ! Are you more interested in the grayscale support, or just the colorjitter augmentation? (trying to figure out priorities for next release)

rtaori commented 1 year ago

Just the colorjitter augmentation

andrewilyas commented 1 year ago

@rtaori looks like #162 might have accomplished something similar already? Does this PR add something new?

rtaori commented 1 year ago

I see. I guess it may be a good enough approximation. There is no support for Hue jitter that I saw in that PR.

lengstrom commented 1 year ago

@rtaori: This branch should work, it is what I use every day!

rtaori commented 1 year ago

Ok thanks will try it out!