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.

Sign up for GitHub

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

Jump to bottom

[GPU] Fix warnings #16196

Merged
merged 12 commits into from
Mar 16, 2023
Merged

[GPU] Fix warnings #16196

merged 12 commits into from
Mar 16, 2023

Conversation

andrei-cv
Copy link
Contributor

Details:

  • fix warning in GPU plugin and GPU tests

Tickets:

  • 93716

@github-actions github-actions bot added category: build OpenVINO cmake script / infra category: GPU OpenVINO GPU plugin category: inference OpenVINO Runtime library - Inference labels Mar 10, 2023
@andrei-cv andrei-cv marked this pull request as ready for review March 10, 2023 00:57
@andrei-cv andrei-cv requested review from a team as code owners March 10, 2023 00:57
ilya-lavrenov
ilya-lavrenov reviewed Mar 10, 2023
View reviewed changes
Copy link
Contributor

@ilya-lavrenov ilya-lavrenov left a comment

Choose a reason for hiding this comment

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

In some places it looks like we need to change the types instead of simply adding static_casts

src/inference/dev_api/performance_heuristics.hpp Outdated Show resolved Hide resolved
src/plugins/intel_gpu/include/intel_gpu/plugin/graph.hpp Outdated Show resolved Hide resolved
src/plugins/intel_gpu/src/graph/layout_optimizer.cpp Outdated Show resolved Hide resolved
src/plugins/intel_gpu/src/graph/network.cpp Show resolved Hide resolved
...lugins/intel_gpu/src/kernel_selector/kernels/scatter_update/scatter_nd_update_kernel_ref.cpp Outdated Show resolved Hide resolved
src/plugins/intel_gpu/tests/test_cases/deconvolution_gpu_test.cpp Outdated Show resolved Hide resolved
src/plugins/intel_gpu/tests/test_cases/lru_caches_gpu_test.cpp Outdated Show resolved Hide resolved
@github-actions github-actions bot removed the category: build OpenVINO cmake script / infra label Mar 11, 2023
@andrei-cv andrei-cv force-pushed the warnings_v3 branch 2 times, most recently from 9a1a427 to eca99f3 Compare March 15, 2023 17:38
@p-durandin p-durandin merged commit 2f3ae45 into openvinotoolkit:master Mar 16, 2023
@vladimir-paramuzov vladimir-paramuzov added this to the 2023.0 milestone Mar 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ilya-lavrenov ilya-lavrenov ilya-lavrenov left review comments

@sshlyapn sshlyapn sshlyapn approved these changes

Assignees
No one assigned
Labels
category: GPU OpenVINO GPU plugin category: inference OpenVINO Runtime library - Inference
Projects
None yet
Milestone
2023.0
Development

Successfully merging this pull request may close these issues.
5 participants
@andrei-cv @ilya-lavrenov @sshlyapn @vladimir-paramuzov @p-durandin