Skip to content

define shift_labels in gemma#961

Merged
Tcc0403 merged 1 commit into
linkedin:mainfrom
akoumpa:akoumparouli/fix_gemma3_shift_labels
Dec 2, 2025
Merged

define shift_labels in gemma#961
Tcc0403 merged 1 commit into
linkedin:mainfrom
akoumpa:akoumparouli/fix_gemma3_shift_labels

Conversation

@akoumpa

@akoumpa akoumpa commented Dec 2, 2025

Copy link
Copy Markdown
Contributor

Summary

Minor fix: undefined variable in Gemma3/Paligemma

Testing Done

  • Hardware Type: 2x NVIDIA RTX 5880
  • run make test to ensure correctness
  • run make checkstyle to ensure code style
  • run make test-convergence to ensure convergence

Signed-off-by: Alexandros Koumparoulis <akoumparouli@nvidia.com>
@akoumpa akoumpa marked this pull request as ready for review December 2, 2025 08:43
@akoumpa

akoumpa commented Dec 2, 2025

Copy link
Copy Markdown
Contributor Author

Github does not allow me to ask reviewers, but @Tcc0403 would be best to review. This is a very minor fix. Thanks.

@Tcc0403 Tcc0403 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thank you for the fix!

@Tcc0403 Tcc0403 merged commit 3d075fd into linkedin:main Dec 2, 2025
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.

2 participants