Skip to content

[Fix]: use default MAGI_COMPILE mode in tests and add CI CUDA check#25

Merged
jiahy0825 merged 1 commit into
SandAI-org:mainfrom
cennn:fix/test-default-magi-compile-and-ci-cuda-check
Apr 23, 2026
Merged

[Fix]: use default MAGI_COMPILE mode in tests and add CI CUDA check#25
jiahy0825 merged 1 commit into
SandAI-org:mainfrom
cennn:fix/test-default-magi-compile-and-ci-cuda-check

Conversation

@cennn
Copy link
Copy Markdown
Collaborator

@cennn cennn commented Apr 23, 2026

🗂️ PR Category

  • ✨ New Feature
  • 🚀 Optimization (performance, memory, etc.)
  • 💥 Breaking Change
  • 🐛 Bug Fix
  • 🛠️ Development / Refactoring
  • 📚 Documentation
  • 🧹 Chore (Dependencies, CI/CD, Configuration, etc.)
  • 🧪 Testing

📝 Description

  • Remove explicit CompileMode.TORCH_COMPILE from test fixtures in test_magi_compile.py and test_register_custom_op.py, letting them use the default MAGI_COMPILE mode
  • Add a "Check environment (torch & CUDA)" step as the first CI action to fail fast on misconfigured runners

- test_magi_compile.py and test_register_custom_op.py fixtures were
  explicitly using CompileMode.TORCH_COMPILE, masking any MAGI_COMPILE
  specific issues. Switch to default CompileConfig() (MAGI_COMPILE).
- Add a torch/CUDA availability check as the first CI step so failures
  surface immediately instead of after checkout + install.
@cennn cennn changed the title fix: use default MAGI_COMPILE mode in tests and add CI CUDA check [Fix]: use default MAGI_COMPILE mode in tests and add CI CUDA check Apr 23, 2026
Copy link
Copy Markdown
Collaborator

@jiahy0825 jiahy0825 left a comment

Choose a reason for hiding this comment

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

LGTM

@jiahy0825 jiahy0825 merged commit 8e2e47c into SandAI-org:main Apr 23, 2026
5 checks passed
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.

2 participants