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.

Sign up for GitHub

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

Jump to bottom

Add data2vec loss class, rename modules #16

Merged
merged 6 commits into from
Sep 27, 2024
Merged

Add data2vec loss class, rename modules #16

merged 6 commits into from
Sep 27, 2024

Conversation

amrit110
Copy link
Member

@amrit110 amrit110 commented Sep 13, 2024 •
edited
Loading

PR Type

Feature

Short Description

  • Add Data2VecLoss into the losses sub-package
  • Rename modules to simpler and shorter names

Tests Added

TODO

@amrit110 amrit110 changed the title Add Data2VecVisionEncoder and Data2VecLoss classes, rename modules Add data2vec loss class, rename modules Sep 25, 2024
@amrit110 amrit110 marked this pull request as ready for review September 25, 2024 13:15
@amrit110 amrit110 requested a review from fcogidi September 25, 2024 13:15
@amrit110 amrit110 self-assigned this Sep 25, 2024
@amrit110 amrit110 added refactor Change exisiting code while maintaining the same functionality enhancement New feature or request labels Sep 25, 2024
@fcogidi fcogidi merged commit 0602c6d into main Sep 27, 2024
4 checks passed
@fcogidi fcogidi deleted the add_d2v_loss_enc branch September 27, 2024 23:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@fcogidi fcogidi fcogidi approved these changes

Assignees

@amrit110 amrit110

Labels
enhancement New feature or request refactor Change exisiting code while maintaining the same functionality
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@amrit110 @fcogidi @actions-user