Skip to content

Conversation

@wujingyue
Copy link
Collaborator

No description provided.

@wujingyue
Copy link
Collaborator Author

!test

@github-actions
Copy link

Description

  • Skip failing test to maintain CI stability

Changes walkthrough 📝

Relevant files
Tests
test_overlap.py
Skip failing allgather matmul test                                             

tests/python/multidevice/test_overlap.py

  • Added @pytest.mark.skip decorator to disable failing test
  • Preserved existing test parameters and structure
  • +1/-0     

    PR Reviewer Guide 🔍

    Here are some key observations to aid the review process:

    🧪 No relevant tests
    ⚡ Recommended focus areas for review

    Test Skipped

    The test test_overlap_allgather_matmul_stream_outermost is marked with @pytest.mark.skip, which may hide underlying issues in the implementation. Consider addressing the root cause instead of skipping the test to maintain test coverage and ensure correctness.

    @pytest.mark.skip(reason="Broke jit_python_distributed_tests_20_GB200_TNVF")

    @wujingyue wujingyue requested a review from rdspring1 October 28, 2025 01:28
    @wujingyue wujingyue marked this pull request as draft October 28, 2025 17:23
    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