Skip to content

Conversation

@ShwethaSelma
Copy link
Contributor

Existing Sample Changes

Description

Fix for error on PVC machine, since sample uses sycl::image class - Code checks for devices that are compatible with sycl image class. CMakefile change to execute on gpu and cpu seperately.

Type of change

Please delete options that are not relevant. Add a 'X' to the one that is applicable.

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Implement fixes for ONSAM Jiras

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

  • Command Line
  • oneapi-cli
  • Visual Studio
  • Eclipse IDE
  • VSCode
  • When compiling the compliler flag "-Wall -Wformat-security -Werror=format-security" was used

Signed-off-by: ShwethaSelma <[email protected]>
@ShwethaSelma ShwethaSelma changed the title updated makefiles and README HSOptical Sample May 9, 2023
Copy link
Contributor

@IgorOchocki IgorOchocki left a comment

Choose a reason for hiding this comment

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

Looks good to me. @jkinsky can you please accept the changes in the README file?

@IgorOchocki IgorOchocki requested a review from a user May 9, 2023 13:44
@IgorOchocki IgorOchocki merged commit 7f846f4 into oneapi-src:development May 10, 2023
@krisrak
Copy link
Contributor

krisrak commented May 10, 2023

use d.has(aspect::image) instead of d.get_info<info::device::image_support>(), this will remove deprecation warning

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