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

XFAIL failing cpu comparisons #698

Merged
merged 2 commits into from
Aug 23, 2024
Merged

Conversation

makslevental
Copy link
Collaborator

@makslevental makslevental commented Aug 23, 2024

The Windows E2E PR #689 is blocked because in order to get Windows to work I had to use a particular commit of XRT (required by XRT-MCDM). At that commit, the current linux driver (date 20240819) fails some tests while the older driver (such as the one on the sharkbox phoenix nuc) passes. I do not know what the exact issue is but even after a couple recent flake-fixes (#697, #694), the disparity persists.

This PR XFAILs (effectively) those tests in order to unblock windows E2E. I will reconcile the disparity shortly.

Tracking issue #700.

cc @newling

@makslevental makslevental force-pushed the makslevental/updated-driver branch from c2d99c4 to 1c981a0 Compare August 23, 2024 17:15
@makslevental makslevental force-pushed the makslevental/updated-driver branch from 1c981a0 to 4b6bc6e Compare August 23, 2024 17:43
@makslevental makslevental force-pushed the makslevental/updated-driver branch from 4b6bc6e to 10dab9e Compare August 23, 2024 18:05
@makslevental makslevental changed the title try runner with updated driver [NO MERGE] try runner with updated driver Aug 23, 2024
@makslevental makslevental force-pushed the makslevental/updated-driver branch from 41fb0cd to 2944541 Compare August 23, 2024 19:06
@makslevental makslevental changed the title [NO MERGE] try runner with updated driver XFAIL failing cpu comparisons Aug 23, 2024
@makslevental makslevental force-pushed the makslevental/updated-driver branch from 2944541 to acacc14 Compare August 23, 2024 19:35
@makslevental makslevental force-pushed the makslevental/updated-driver branch 3 times, most recently from 844c633 to c788bba Compare August 23, 2024 20:38
@makslevental makslevental force-pushed the makslevental/updated-driver branch from c788bba to bccd6f8 Compare August 23, 2024 21:08
@makslevental makslevental merged commit 652d648 into main Aug 23, 2024
4 checks passed
@makslevental makslevental deleted the makslevental/updated-driver branch August 23, 2024 22:29
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