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

generic: skip unsupported examples #2171

Merged
merged 1 commit into from
Oct 21, 2024

Conversation

sgeor255
Copy link
Contributor

@sgeor255 sgeor255 commented Oct 16, 2024

This PR skips the unsupported examples on generic vendor and fixes an issue in deconvolution where problems post-ops were incorrectly returning unimplemented (and as a result the deconvolution example was failing).

This PR is currently on top of #2159 (996e949 contains the new changes). I will rebase it on main when #2159 is merged.

@sgeor255 sgeor255 requested review from a team as code owners October 16, 2024 10:41
@sgeor255 sgeor255 changed the title generic: fix several gtest issues generic: skip unsupported examples Oct 16, 2024
@github-actions github-actions bot added the platform:gpu-generic Codeowner: @oneapi-src/onednn-gpu-generic label Oct 16, 2024
@mgouicem
Copy link
Contributor

Thanks for the PR @sgeor255. It seems clang-format is picking on src/gpu/generic/sycl/ref_convolution.cpp:139 (it seems to expect a line break before return statement). Could you run clang-format on that file please?

@mgouicem
Copy link
Contributor

make test
disable device_cpu
enable device_gpu
enable thr_generic
enable arch_rtx

@sgeor255 sgeor255 force-pushed the svet/example-fixes branch 2 times, most recently from 1d2d20d to e7a827d Compare October 21, 2024 15:25
@mgouicem mgouicem merged commit 2962128 into uxlfoundation:main Oct 21, 2024
5 of 16 checks passed
@vpirogov vpirogov added this to the v3.7 milestone Dec 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
platform:gpu-generic Codeowner: @oneapi-src/onednn-gpu-generic
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants