Skip to content

Comments

reduce modal test time#433

Merged
msaroufim merged 1 commit intomainfrom
modal_takes_too_long
Feb 5, 2026
Merged

reduce modal test time#433
msaroufim merged 1 commit intomainfrom
modal_takes_too_long

Conversation

@msaroufim
Copy link
Member

No description provided.

Copilot AI review requested due to automatic review settings February 5, 2026 05:00
@github-actions
Copy link

github-actions bot commented Feb 5, 2026

Coverage report

Click to see where and how coverage changed

FileStatementsMissingCoverageCoverage
(new stmts)
Lines missing
  src/libkernelbot
  utils.py
Project Total  

This report was generated by python-coverage-comment-action

Copy link
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 optimizes CI test execution time by reducing the number of GPU types tested in the Modal launcher integration tests.

Changes:

  • Reduced GPU type parametrization from 5 types (T4, L4, A100, H100, B200) to 2 types (T4, H100) in test_modal_launcher_python_script

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

@msaroufim msaroufim merged commit 911b94e into main Feb 5, 2026
11 of 12 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.

1 participant