-
Notifications
You must be signed in to change notification settings - Fork 287
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Cleanup docstrings in lightly/models/modules subpackage. #1687
Comments
Hii, can you assign this to me? |
Hi, yes of course :) |
hii, can you ssign this issue to me? |
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? |
@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. |
Hi @fadkeabhi and @ayush22iitbhu could you post which files you already updated? Then we can avoid that you work both on the same thing. |
@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. |
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 :) |
@guarin review PR |
See #1216 for more information
The text was updated successfully, but these errors were encountered: