Skip to content

moved warmup to metal#316

Open
nostojicTT wants to merge 6 commits intodevfrom
nostojic/move_warmup_to_metal
Open

moved warmup to metal#316
nostojicTT wants to merge 6 commits intodevfrom
nostojic/move_warmup_to_metal

Conversation

@nostojicTT
Copy link

@nostojicTT nostojicTT commented Feb 9, 2026

Purpose

Move the parameter creation of decode warmup to metal side for the needs of metal e2e and demo tests of the warmup method.

tt-metal PR - tenstorrent/tt-metal#37447


Essential Elements of an Effective PR Description Checklist
  • The purpose of the PR, such as "Fix some issue (link existing issues this PR will resolve)".
  • The test plan, such as providing test command.
  • The test results, such as pasting the results comparison before and after, or e2e results
  • (Optional) The necessary documentation update, such as updating supported_models.md and examples for a new model.
  • (Optional) Release notes update. If your change is user facing, please update the release notes draft in the Google Doc.

@bgoelTT
Copy link

bgoelTT commented Feb 11, 2026

Ah I see the logging was moved Generator-side 👍

Signed-off-by: Nikola Ostojic <nostojic@tenstorrent.com>
Signed-off-by: Nikola Ostojic <nostojic@tenstorrent.com>
@nostojicTT nostojicTT force-pushed the nostojic/move_warmup_to_metal branch from c9699ee to 3dd01a8 Compare February 13, 2026 20:17
Signed-off-by: Nikola Ostojic <nostojic@tenstorrent.com>
Copy link
Collaborator

@skhorasganiTT skhorasganiTT left a comment

Choose a reason for hiding this comment

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

lgtm assuming no new vllm nightly ci failures

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.

3 participants