Skip to content
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: ijepa example #1747

Merged

Conversation

vectorvp
Copy link
Contributor

Resolves #1712

Moved apply_masks together with repeat_interleave_batch to utils.py from lightly/models/modules/ijepa_timm.py.

Copy link

codecov bot commented Nov 29, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 85.23%. Comparing base (1e7274a) to head (9fa4820).
Report is 1 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #1747   +/-   ##
=======================================
  Coverage   85.22%   85.23%           
=======================================
  Files         162      162           
  Lines        6729     6731    +2     
=======================================
+ Hits         5735     5737    +2     
  Misses        994      994           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Contributor

@guarin guarin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the PR! I updated the docstrings to match our format but otherwise everything LGTM :)

@guarin guarin enabled auto-merge (squash) December 3, 2024 08:41
@guarin guarin merged commit fac3dcb into lightly-ai:master Dec 3, 2024
12 checks passed
@vectorvp vectorvp deleted the issue-1712-fix-broken-ijepa-example branch December 3, 2024 08:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Fix broken IJEPA Example
2 participants