guarin comments

Results 180 comments of


                                            guarin

integrate with Lightning ecosystem CI

Thanks for the message @Borda, we'll look into it!

SimMIM and non-ViT backbones

Hi @faris-k! We noticed this as well but I didn't look into it yet. From a quick glance at a code I believe the linear decoder head might be missing....

Move config files and setup.py to pyproject.toml

Hi, yes `.coveragerc` or `mypy.ini` could be a good start and a PR would be very welcome. I agree that adding `pylint` doesn't make sense for now, especially because we...

Move config files and setup.py to pyproject.toml

Completed in #1559

A²MIM

Hi! Thanks for your interest in lightly and suggesting the method! It looks interesting and could be a good addition to our package. I'll add it to the our [SSL...

Hi @Atharva-Phatak I believe you have to call `super().__init__()` in the `__init__` function of the loss. Otherwise the `nn.Module` is not correctly initialized. The error message indicates that some modules...

Added directclr loss

Hi @Atharva-Phatak, sorry for the late reply but I fixed the merge conflict. There still seems to be an issue with the loss implementation though.

Enable accelerator: "auto" and devices: "auto"

Hi @julien-blanchon, thanks for the issue. You are right that we should update this and use `"auto"` by default.

Add SimCLR with temperature schedule

Added it to #1172

pytorch_lightning -> lightning

Hi, from the link it seems that it is not yet clear when the change will happen. I would propose to wait a bit until we know more, especially because...