Skip to content

[NVIDIA] fix: gptoss b200 after refactor#168

Merged
functionstackx merged 1 commit intomainfrom
fix-nvs
Nov 5, 2025
Merged

[NVIDIA] fix: gptoss b200 after refactor#168
functionstackx merged 1 commit intomainfrom
fix-nvs

Conversation

@functionstackx
Copy link
Copy Markdown
Contributor

@functionstackx functionstackx requested a review from a team as a code owner November 5, 2025 20:27
Copilot AI review requested due to automatic review settings November 5, 2025 20:27
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR fixes a configuration issue introduced during a refactor where the b200-nvs runner group was incorrectly used instead of b200-trt. The changes restore the correct runner assignment for the gptoss-fp4-b200-trt configuration and clean up the runner definitions to remove the now-unused b200-nvs group.

  • Updated gptoss-fp4-b200-trt to use the correct b200-trt runner group
  • Removed the obsolete b200-nvs runner group definition
  • Reorganized runner group definitions for better clarity

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
.github/configs/runners.yaml Removed unused b200-nvs runner group and reorganized b200 group placement
.github/configs/nvidia-master.yaml Corrected runner assignment from b200-nvs to b200-trt for gptoss-fp4-b200-trt configuration

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Nov 5, 2025

📊 Line Count Report

File: utils/matrix-logic/generate_sweep_configs.py

Total Lines: 956

Copy link
Copy Markdown
Collaborator

@cquil11 cquil11 left a comment

Choose a reason for hiding this comment

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

W fix

@functionstackx functionstackx merged commit 2cf106d into main Nov 5, 2025
1 check passed
@functionstackx functionstackx deleted the fix-nvs branch November 5, 2025 20:29
@cquil11 cquil11 added the NVIDIA label Apr 8, 2026
@cquil11 cquil11 changed the title fix: gptoss b200 after refactor [NVIDIA] fix: gptoss b200 after refactor Apr 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants