Skip to content

[ViPPET] Add dynamic gpu progress components and test utils #1641

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
KaliszWiktoria merged 9 commits into from
Jan 14, 2026
Merged

[ViPPET] Add dynamic gpu progress components and test utils #1641

KaliszWiktoria merged 9 commits into from
Jan 14, 2026

Conversation

@KaliszWiktoria
Copy link
Contributor

@KaliszWiktoria KaliszWiktoria commented Jan 9, 2026 •
edited
Loading

Description

  • Added GPU progress components that are added based on the number of GPUs available, with their tags
  • Deleted old GPU components
  • Added no shrink class parameter to cpu and gpu icons
  • Added test utils file, helpful for testing metrics related components
111

before:
shrink2
after:
shrink1

Checklist:

  • I agree to use the APACHE-2.0 license for my code changes.
  • I have not introduced any 3rd party components incompatible with APACHE-2.0.
  • I have not included any company confidential information, trade secret, password or security token.
  • I have performed a self-review of my code.

@KaliszWiktoria KaliszWiktoria self-assigned this Jan 9, 2026
@p-zak p-zak added the vippet label Jan 9, 2026
@p-zak p-zak changed the title Add dynamic gpu progress components and test utils [ViPPET] Add dynamic gpu progress components and test utils Jan 9, 2026
@KaliszWiktoria KaliszWiktoria marked this pull request as ready for review January 9, 2026 15:56
p-zak
p-zak previously approved these changes Jan 12, 2026
View reviewed changes
dlemiech
dlemiech reviewed Jan 13, 2026
View reviewed changes
dlemiech
dlemiech previously approved these changes Jan 13, 2026
View reviewed changes
@KaliszWiktoria KaliszWiktoria merged commit 032e7f6 into main Jan 14, 2026
31 of 36 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@p-zak p-zak p-zak approved these changes

@ktaube26 ktaube26 ktaube26 approved these changes

@dlemiech dlemiech dlemiech approved these changes

@pmalatyn pmalatyn Awaiting requested review from pmalatyn pmalatyn is a code owner

@dmichalo dmichalo Awaiting requested review from dmichalo dmichalo is a code owner

@msmiatac msmiatac Awaiting requested review from msmiatac msmiatac is a code owner

@grey-eminence grey-eminence Awaiting requested review from grey-eminence grey-eminence is a code owner

Assignees

@KaliszWiktoria KaliszWiktoria

Labels

vippet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@KaliszWiktoria @p-zak @ktaube26 @dlemiech