-
Notifications
You must be signed in to change notification settings - Fork 286
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
Fix broken IJEPA Example #1712
Comments
@MalteEbner, hello, so the idea is to return apply_masks() to utils.py? |
Yes that would be a great start. I'm not sure if something else is broken as well. The relevant code from the original PR is:
|
@guarin, apply_masks together with repeat_interleave_batch were moved to lightly/models/modules/ijepa_timm.py. However, timm is used nowhere. Does this module make sense? Do we want to remove it and just add the functions we need to utils.py? |
I've delved into the conversations from past issues regarding this timm module and plans to move to it from the torchvision. Based on that I just moved Now the example works. PR is attached. |
When running
python examples/pytorch/ijepa.py
, the code fails:The utils are imported as
from lightly.models import utils
.The function
lightly.models/utils.py:apply_masks()
was added in a55cf44 and removed later.The text was updated successfully, but these errors were encountered: