Skip to content

Conversation

@wujingyue
Copy link
Collaborator

@wujingyue wujingyue commented Sep 11, 2025

Partially reverts #5101

@wujingyue
Copy link
Collaborator Author

!test

@wujingyue wujingyue requested a review from rdspring1 September 11, 2025 17:47
@github-actions
Copy link

Description

  • Disabled failing test in cutlass_gemm

  • Removed pre-Blackwell device check

  • Updated skip reason with failure context


Changes walkthrough 📝

Relevant files
Bug fix
test_cutlass_gemm.py
Disable failing grouped mm test                                                   

tests/python/direct/test_cutlass_gemm.py

  • Disabled test_grouped_mm due to failures
  • Replaced device-specific skip with general skip
  • Removed unused is_pre_blackwell import
  • Updated skip reason to reference specific failure
  • +1/-4     

    PR Reviewer Guide 🔍

    Here are some key observations to aid the review process:

    🧪 PR contains tests
    ⚡ Recommended focus areas for review

    Test Disabled

    The test test_grouped_mm has been disabled by adding a @pytest.mark.skip decorator with a reason referencing a broken CI job. This suppression should be validated to ensure it's a temporary measure and that there is a plan or ticket to address the underlying issue causing the failure.

    @pytest.mark.skip(reason="broke jit_python_tests_20_B100_TNVF")

    @wujingyue wujingyue merged commit 79b1b2b into main Sep 11, 2025
    50 of 51 checks passed
    @wujingyue wujingyue deleted the wjy/gmm branch September 11, 2025 19:13
    rdspring1 added a commit that referenced this pull request Oct 3, 2025
    )
    
    Revert #5155 to reenable bf16 grouped gemm on blackwell. This PR runs
    the CI to Reproduce failure in `jit_python_tests_20_B100_TNVF`
    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.

    3 participants