Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Fix cmake FindDpctl.cmake#1484

Merged
oleksandr-pavlyk merged 1 commit intoIntelPython:masterIntelPython/dpctl:masterfrom
ZzEeKkAa:fix/cmake_searchZzEeKkAa/dpctl:fix/cmake_searchCopy head branch name to clipboard
Dec 6, 2023
Merged

Fix cmake FindDpctl.cmake#1484
oleksandr-pavlyk merged 1 commit intoIntelPython:masterIntelPython/dpctl:masterfrom
ZzEeKkAa:fix/cmake_searchZzEeKkAa/dpctl:fix/cmake_searchCopy head branch name to clipboard

Conversation

@ZzEeKkAa
Copy link
Contributor

@ZzEeKkAa ZzEeKkAa commented Dec 6, 2023

Update environment names for dpctl find cmake file. Issue most likely was introduced in #1389 but was reveled only now under certain circumstances: I was trying to build numba-dpex in vscode. I've tested locally that both scikit build and direct cmake build are working.

  • Have you provided a meaningful PR description?
  • Have you added a test, reproducer or referred to an issue with a reproducer?
  • Have you tested your changes locally for CPU and GPU devices?
  • Have you made sure that new changes do not introduce compiler warnings?
  • Have you checked performance impact of proposed changes?
  • If this PR is a work in progress, are you opening the PR as a draft?

@coveralls
Copy link
Collaborator

Coverage Status

coverage: 85.979%. remained the same
when pulling 6950a63 on ZzEeKkAa:fix/cmake_search
into 07dea79 on IntelPython:master.

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

Morty Proxy This is a proxified and sanitized view of the page, visit original site.