Skip to content

[DRAFT] [AMD] Update Minimax M2.5 MI325 image and adjust search space#953

Open
benenzhu wants to merge 8 commits intomainfrom
amd/zhu/minimax
Open

[DRAFT] [AMD] Update Minimax M2.5 MI325 image and adjust search space#953
benenzhu wants to merge 8 commits intomainfrom
amd/zhu/minimax

Conversation

@benenzhu
Copy link
Copy Markdown
Collaborator

@benenzhu benenzhu commented Mar 27, 2026

@github-actions
Copy link
Copy Markdown
Contributor

Thanks for the contribution! For vLLM & SGLang, please ensure that your recipes is similar to the official vLLM recipes and/or the SGLang cookbook

If it is not, please create a PR first before we can merge your PR into the master branch. Let's ensure that the documentation is first class such that the entire ML community can benefit from your hard work! Thank you

1 similar comment
@github-actions
Copy link
Copy Markdown
Contributor

Thanks for the contribution! For vLLM & SGLang, please ensure that your recipes is similar to the official vLLM recipes and/or the SGLang cookbook

If it is not, please create a PR first before we can merge your PR into the master branch. Let's ensure that the documentation is first class such that the entire ML community can benefit from your hard work! Thank you

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@benenzhu benenzhu changed the title [AMD] Update minimax mi325/mi300 image and adjust search space [DRAFT] [AMD] Update Minimax M2.5 MI325 image and adjust search space Mar 27, 2026
@functionstackx
Copy link
Copy Markdown
Contributor

@chunfangamd is @benenzhu part of AMD? can u confirm internally? if so, plz add him to the upstream repo. better developer experience to create branches in upstream then do forks. for example, forks, we can do sweep-enabled label to validate PRs

@benenzhu
Copy link
Copy Markdown
Collaborator Author

@chunfangamd is @benenzhu part of AMD? can u confirm internally? if so, plz add him to the upstream repo. better developer experience to create branches in upstream then do forks. for example, forks, we can do sweep-enabled label to validate PRs

Hi I am from AMD and already in the repo with the help of @chunfangamd. Though this branch name is starting with amd/ maybe a little confusing .

Copy link
Copy Markdown
Collaborator

@chunfangamd chunfangamd left a comment

Choose a reason for hiding this comment

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

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

3 participants