Skip to content
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

unittest: Handle op_db to align xfail defined by CUDA #608

Merged
merged 30 commits into from
Aug 10, 2024

Conversation

yuchengliu1
Copy link
Contributor

@yuchengliu1 yuchengliu1 commented Jul 17, 2024

Functional coverage of PyTorch XPU operator is aligned with CUDA. So we are gradually align test scope with CUDA. The PR is to handle op_db to align xfail configuration. It helps us reduce efforts to triage failures which are expected.
#593

@fengyuan14 fengyuan14 changed the title pass unexpected xfail unittest: Handle op_db to align xfail defined by CUDA Jul 26, 2024
@fengyuan14
Copy link
Contributor

Please resolve the conflicts.

@fengyuan14
Copy link
Contributor

Please,

  1. Address the failure of extended case script
  2. Resolve the conflicts.

# Mismatched elements: 2 / 125 (1.6%)
# Greatest absolute difference: 0.001953125 at index (2, 0, 0) (up to 0.001 allowed)
# Greatest relative difference: 0.007568359375 at index (2, 0, 0) (up to 0.001 allowed)
"test_compare_cpu_cumprod_xpu_bfloat16",
Copy link
Contributor

Choose a reason for hiding this comment

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

We have got it in line 172. Please remove it.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

# Mismatched elements: 2 / 125 (1.6%)
# Greatest absolute difference: 0.001953125 at index (2, 0, 0) (up to 0.001 allowed)
# Greatest relative difference: 0.007568359375 at index (2, 0, 0) (up to 0.001 allowed)
"test_compare_cpu_cumprod_xpu_bfloat16",
Copy link
Contributor

Choose a reason for hiding this comment

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

Why we pass before?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

removed it

@fengyuan14
Copy link
Contributor

Please address the comments.

@fengyuan14 fengyuan14 added this pull request to the merge queue Aug 10, 2024
Merged via the queue into main with commit a1657ad Aug 10, 2024
3 checks passed
@fengyuan14 fengyuan14 deleted the unexpected_xfail branch August 10, 2024 10:48
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