Skip to content

[SYCL][E2E] increase test coverage for subregions copies #14046

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

Merged
merged 11 commits into from
Jun 26, 2024

Conversation

cppchedy
Copy link
Contributor

@cppchedy cppchedy commented Jun 5, 2024

  • Updated 2D subregion copy test so that it follows the other subregion tests.
  • Updated the 3D and 2D subregion tests to cover test cases for copying with non-zero offsets from device to host.

@cppchedy cppchedy requested review from a team as code owners June 5, 2024 10:11
@cppchedy cppchedy marked this pull request as draft June 5, 2024 10:11
@cppchedy cppchedy force-pushed the chedy/fix-copy-in-bytes-bug branch from e6854c6 to 6b52b98 Compare June 5, 2024 10:14
@cppchedy
Copy link
Contributor Author

cppchedy commented Jun 5, 2024

Associated UR PR

* Updated 2D subregion copy test so that it follows the other
  subregion tests.
* Updated the 3D and 2D subregion tests to cover test cases
  for copying with non-zero offsets from device to host.
@cppchedy cppchedy force-pushed the chedy/fix-copy-in-bytes-bug branch from 6b52b98 to 4f1019f Compare June 5, 2024 12:34
@cppchedy cppchedy temporarily deployed to WindowsCILock June 5, 2024 12:34 — with GitHub Actions Inactive
@cppchedy cppchedy temporarily deployed to WindowsCILock June 5, 2024 12:58 — with GitHub Actions Inactive
@cppchedy cppchedy temporarily deployed to WindowsCILock June 5, 2024 14:18 — with GitHub Actions Inactive
@cppchedy cppchedy temporarily deployed to WindowsCILock June 5, 2024 14:41 — with GitHub Actions Inactive
@cppchedy cppchedy marked this pull request as ready for review June 6, 2024 11:09
@cppchedy
Copy link
Contributor Author

cppchedy commented Jun 21, 2024

The failed tests in this job aren't related to the bindless images tests updated in this PR.

@callumfare
Copy link
Contributor

oneapi-src/unified-runtime#1714 has merged, please pull in the latest sycl branch changes and update the UR repo/tag as suggested.

@callumfare
Copy link
Contributor

@intel/llvm-gatekeepers Please merge

@sommerlukas sommerlukas merged commit 3eef33b into intel:sycl Jun 26, 2024
13 checks passed
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.

7 participants