Skip to content

Conversation

@wujingyue
Copy link
Collaborator

@wujingyue wujingyue commented Oct 7, 2025

@wujingyue
Copy link
Collaborator Author

!test

@github-actions
Copy link

github-actions bot commented Oct 7, 2025

Description

  • Update test file reference in timeout configuration

Changes walkthrough 📝

Relevant files
Enhancement
run_nvfuser_tests.py
Update test name in timeout logic                                               

tools/run_nvfuser_tests.py

  • Updated test name from test_ops.py to test_direct_ops.py in timeout
    configuration
  • Maintains 1-hour timeout for key test files
  • +1/-1     
    Additional files
    test_legacy_ops.py +0/-299 
    test_inline_definitions.py +0/-70   
    test_normalization.py +0/-201 
    test_python_frontend.py +0/-5319
    test_schedule_ops.py +0/-1158

    PR Reviewer Guide 🔍

    Here are some key observations to aid the review process:

    🧪 No relevant tests
    ⚡ Recommended focus areas for review

    Possible Issue

    The test timeout configuration still references test_ops.py in the legacy list, but the file test_python_frontend.py and others were removed. This may indicate outdated logic or a mismatch in test name handling.

    if test_name in ["test_nvfuser", "test_matmul", "test_direct_ops.py"]:
        return 3600  # 1 hour

    @wujingyue wujingyue requested a review from rdspring1 October 7, 2025 23:41
    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