Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Minor updates to VLM Multimodal notebook #1299

Merged
merged 6 commits into from
Jan 29, 2025
Merged

Conversation

xrdaukar
Copy link
Collaborator

@xrdaukar xrdaukar commented Jan 29, 2025

Description

-- Refactor memory cleanup into a helper function and call it in more places.
-- Double the number of training steps (with the previous count I observed when adapter model still produces verbose outputs)
-- Retain some cells' outputs (they can serve as illustration)
-- More-consistent handling of variables .
-- Other minor changes

Related issues

Towards OPE-911

Before submitting

  • This PR only changes documentation. (You can ignore the following checks in that case)
  • Did you read the contributor guideline Pull Request guidelines?
  • Did you link the issue(s) related to this PR in the section above?
  • Did you add / update tests where needed?

Reviewers

At least one review from a member of oumi-ai/oumi-staff is required.

@xrdaukar xrdaukar requested review from optas and oelachqar January 29, 2025 00:32
@xrdaukar xrdaukar marked this pull request as ready for review January 29, 2025 00:32
@xrdaukar xrdaukar changed the title Minor updated to Multimodal notebooks Minor updates to VLM Multimodal notebook Jan 29, 2025
@xrdaukar xrdaukar merged commit f0c3990 into main Jan 29, 2025
@xrdaukar xrdaukar deleted the xrdaukar/mm-notebook-v4 branch January 29, 2025 00:49
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants