lightly-ai / lightly

A python library for self-supervised learning on images.
https://docs.lightly.ai/self-supervised-learning/
MIT License
3.19k stars 286 forks source link

Cleanup docstrings in lightly/models/modules subpackage. #1687

Open guarin opened 1 month ago

guarin commented 1 month ago

See https://github.com/lightly-ai/lightly/issues/1216 for more information

goelnaman66 commented 1 month ago

Hii, can you assign this to me?

guarin commented 1 month ago

Hi, yes of course :)

Abhrajitdas02 commented 1 month ago

hii, can you ssign this issue to me?

guarin commented 1 month ago

Hi @Abhrajitdas02! Thanks for your interest in this issue. We only assign issues to one person at a time.

@goelnaman66 did you already have time to look into this?

fadkeabhi commented 1 month ago

Hey @guarin I would like to work on this issue as discussed here https://github.com/lightly-ai/lightly/issues/1216. For some reason I did not receive the notification about it and I could not comment on it, can you assign it to me now. I have already started to work on it.

ayush22iitbhu commented 1 month ago

@guarin I see this issue has been open for more than a week. I have already started working on this. Also, I have already completed the work you assigned to me on documentation of lightly/models/utils.py and lightly/loss subpackage. Please assign this issue to me. I'll complete this soon as well.

guarin commented 1 month ago

Hi @fadkeabhi and @ayush22iitbhu could you post which files you already updated? Then we can avoid that you work both on the same thing.

ayush22iitbhu commented 1 month ago

@guarin I have already updated center.py, heads.py, heads_timm.py and currently working on ijepa.py. But you can assign more files to me.

guarin commented 1 month ago

Could you already make a PR for these files? Then we can merge and avoid conflicts later on. The fewer files a PR touches the easier it is to merge for us :)

ayush22iitbhu commented 1 month ago

@guarin review PR