Central location for transforms

[matsumotosan]

Data transform classes for various models are included in their own folders in pl_bolts/models. It may be better to have them in pl_bolts/transforms instead.

Issue discussed with @otaj

[senarvi]

I was going to suggest the same thing. Also, at least SimCLR and MoCo can use the same transforms. Maybe we want to provide some transforms that can be used to replicate the paper results on some dataset, but do we want to provide a separate set of transforms for every model and every dataset?

Related to this, I feel like it would be good to somehow standardize the format in which the data is processed (data types, dimensions of the tensors), for different tasks, so that any classification model can be used with any classification dataset and transforms, and same for detection and SSL models. I don't know how much this is true already.

[matsumotosan]

In general, I think the existing transforms are based on what the original papers specified (for whatever datasets they used). If a model by default uses transforms from another (i.e. BYOL using SimCLR transforms), I think we can just import SimCLRTransforms for BYOL instead of creating a copy and calling it BYOLTransforms. I think creating transforms for every model for every dataset might be too much though.

I'm all for standardizing how transforms are set up. I'm almost done moving everything over to pl_bolts/transforms so I'll let you know when I'm finished with that.

[otaj]

Hi, @matsumotosan, that would be great, thank you for kicking it off!

[matsumotosan]

All self-supervised transforms have been moved #905

[stale]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

[matsumotosan]

Commenting so this issue doesn't get closed.

All changes have been made in #905. Waiting for CI to be fixed so we can finish testing.

[stale]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.