8000 Enable skipIfXpu to support class-level skipping by EikanWang · Pull Request #151420 · pytorch/pytorch · GitHub
[go: up one dir, main page]

Skip to content

Enable skipIfXpu to support class-level skipping #151420

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 8 commits into
base: gh/EikanWang/81/base
Choose a base branch
from

Conversation

EikanWang
Copy link
Collaborator
@EikanWang EikanWang commented Apr 16, 2025

[ghstack-poisoned]
Copy link
pytorch-bot bot commented Apr 16, 2025

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/151420

Note: Links to docs will display an error until the docs builds have been completed.

✅ You can merge normally! (4 Unrelated Failures)

As of commit c5787b8 with merge base cd7bc60 (image):

FLAKY - The following jobs failed but were likely due to flakiness present on trunk:

This comment was automatically generated by Dr. CI and updates every 15 minutes.

EikanWang added a commit that referenced this pull request Apr 16, 2025
ghstack-source-id: c2810e8
Pull Request resolved: #151420
@EikanWang EikanWang changed the title Enable skipIfXpu support class-level skip Enable skipIfXpu to support class-level skipping Apr 16, 2025
@EikanWang EikanWang requested a review from malfet April 16, 2025 07:20
@EikanWang EikanWang added ciflow/xpu Run XPU CI tasks topic: not user facing topic category labels Apr 16, 2025
)
expected_ref = torch.xpu.is_available()
expected_res = "test_case_1" not in rc
self.assertEqual(expected_ref, expected_res)
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Since the test class is decorated by skipIfXpu, all the test cases are skipped only for XPU.

[ghstack-poisoned]
Comment on lines +765 to +767
expected_ref = torch.xpu.is_available()
expected_res = "test_case_2" not in rc
self.assertEqual(expected_ref, expected_res)
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

The test_case_2 is decoreated by skipIfXpu while the other test cases are not. Therefore, the literal "test_case_2" should not appear in the output when the XPU is available.

EikanWang added a commit that referenced this pull request Apr 16, 2025
ghstack-source-id: e045b00
Pull Request resolved: #151420
[ghstack-poisoned]
EikanWang added a commit that referenced this pull request Apr 16, 2025
ghstack-source-id: 5b43df4
Pull Request resolved: #151420
[ghstack-poisoned]
EikanWang added a commit that referenced this pull request Apr 16, 2025
ghstack-source-id: 25d3f80
Pull Request resolved: #151420
[ghstack-poisoned]
EikanWang added a commit that referenced this pull request Apr 16, 2025
ghstack-source-id: 0cbd4d1
Pull Request resolved: #151420
[ghstack-poisoned]
EikanWang added a commit that referenced this pull request Apr 16, 2025
ghstack-source-id: 9ec61f6
Pull Request resolved: #151420
[ghstack-poisoned]
EikanWang added a commit that referenced this pull request Apr 16, 2025
ghstack-source-id: 82b00e1
Pull Request resolved: #151420
[ghstack-poisoned]
EikanWang added a commit that referenced this pull request Apr 17, 2025
ghstack-source-id: 51da282
Pull Request resolved: #151420
@EikanWang
Copy link
Collaborator Author

@pytorchbot rebase

@pytorchmergebot
Copy link
Collaborator

@pytorchbot started a rebase job onto refs/remotes/origin/viable/strict. Check the current status here

@pytorchmergebot
Copy link
Collaborator

Rebase failed due to Command git -C /home/runner/work/pytorch/pytorch rebase refs/remotes/origin/viable/strict gh/EikanWang/81/orig returned non-zero exit code 1

Rebasing (1/2)
Rebasing (2/2)
Auto-merging test/test_xpu.py
CONFLICT (content): Merge conflict in test/test_xpu.py
Auto-merging torch/testing/_internal/common_utils.py
error: could not apply ecdcaf193d4... Enable skipIfXpu support class-level skip
hint: Resolve all conflicts manually, mark them as resolved with
hint: "git add/rm <conflicted_files>", then run "git rebase --continue".
hint: You can instead skip this commit: run "git rebase --skip".
hint: To abort and get back to the state before "git rebase", run "git rebase --abort".
hint: Disable this message with "git config set advice.mergeConflict false"
Could not apply ecdcaf193d4... Enable skipIfXpu support class-level skip

Raised by https://github.com/pytorch/pytorch/actions/runs/14999417333

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants
0