Skip to content

CI: add dpnp - #471

Open
Nin17 wants to merge 2 commits into
data-apis:mainfrom
Nin17:dpnp-ci
Open

CI: add dpnp#471
Nin17 wants to merge 2 commits into
data-apis:mainfrom
Nin17:dpnp-ci

Conversation

@Nin17

@Nin17 Nin17 commented Aug 26, 2026

Copy link
Copy Markdown
Contributor

Adds dpnp to Tests workflow

@Nin17
Nin17 marked this pull request as ready for review August 26, 2026 15:17
Copilot AI lite review requested due to automatic review settings August 26, 2026 15:17
@ev-br ev-br closed this Aug 26, 2026
@ev-br ev-br reopened this Aug 26, 2026
@ev-br

ev-br commented Aug 26, 2026

Copy link
Copy Markdown
Member

close/reopen to flush the CI

@Nin17 Nin17 closed this Aug 26, 2026
@Nin17 Nin17 reopened this Aug 26, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@ev-br

ev-br commented Aug 26, 2026

Copy link
Copy Markdown
Member

CI looks unhappy:

...
FAILED tests/test_common.py::test_asarray_cross_library[dpnp-dpnp] - dpctl._sycl_device.SyclDeviceCreationError: Default device is unavailable.
==== 35 failed, 487 passed, 207 skipped

@Nin17

Nin17 commented Aug 26, 2026

Copy link
Copy Markdown
Contributor Author

Perhaps I'm misunderstanding the workflow file, but shouldn't it have installed dpnp in the test (latest, 3.10), (latest, 3.13) and (latest, 3.14) jobs as well?

@Nin17

Nin17 commented Aug 27, 2026

Copy link
Copy Markdown
Contributor Author

I've seen you've got an open issue for adding GPU runners to CI: #197. Just thought I'd mention that this isn't strictly necessary for dpnp. It should be possible to use an intel CPU with integrated graphics, for instance I was using an i7 8565U. There's an open issue asking for clarification on the conditions for using dpnp on CPU: IntelPython/dpnp#2923

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