Skip to content

update test to use smaller hf_text model #4235

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
Merged
merged 1 commit into from
May 28, 2024
Merged

update test to use smaller hf_text model #4235

merged 1 commit into from
May 28, 2024

Conversation

suzhoum
Copy link
Contributor

@suzhoum suzhoum commented May 28, 2024 •
edited
Loading

Issue #, if available:

This is to fix the OOM issue on CI test_dump_hf_text https://github.com/autogluon/autogluon/actions/runs/9274764166/job/25518041355

Description of changes:

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@suzhoum suzhoum added model list checked You have updated the model list after modifying multimodal unit tests/docs run-multi-gpu Run multimodal multi-gpu tests labels May 28, 2024
@yinweisu
Copy link
Contributor

Previous CI Run Current CI Run

@suzhoum suzhoum changed the title update to smaller model update test to use smaller hf_text model May 28, 2024
@suzhoum suzhoum requested a review from Innixma May 28, 2024 22:09
Innixma
Innixma approved these changes May 28, 2024
View reviewed changes
Copy link
Contributor

@Innixma Innixma left a comment

Choose a reason for hiding this comment

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

LGTM!

@Innixma Innixma added this to the 1.1.1 Release milestone May 28, 2024
@suzhoum suzhoum merged commit 0e3bc0e into master May 28, 2024
@Innixma Innixma deleted the update_hf_text_test branch June 13, 2024 23:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Innixma Innixma Innixma approved these changes

Assignees
No one assigned
Labels
model list checked You have updated the model list after modifying multimodal unit tests/docs run-multi-gpu Run multimodal multi-gpu tests
Projects
None yet
Milestone
1.1.1 Release
Development

Successfully merging this pull request may close these issues.
3 participants
@suzhoum @yinweisu @Innixma