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 OneHot segmentation fault when the second input node is not a Constant node #27655

Open
wants to merge 4 commits into
base: releases/2024/5
Choose a base branch
from
Open

[GPU] Fix OneHot segmentation fault when the second input node is not a Constant node #27655

wants to merge 4 commits into from
+58 −20

Conversation

yuanxion
Copy link
Contributor

Details:

  • Get the 'depth' value from 'Select' node instead of a Constant node

Tickets:

Migrated from PR #27442.

@github-actions github-actions bot added the category: GPU OpenVINO GPU plugin label Nov 21, 2024
@sys-openvino-ci sys-openvino-ci added the ExternalPR External contributor label Nov 21, 2024
@yuanxion yuanxion marked this pull request as ready for review November 21, 2024 03:02
@yuanxion yuanxion requested review from a team as code owners November 21, 2024 03:02
@sshlyapn sshlyapn added this to the 2024.6 milestone Nov 21, 2024
@sshlyapn
Copy link
Contributor

build_jenkins

@yuanxion
Fix one_hot segmentation fault of getting depth value from the second…
f9a6bee 
… input node

Signed-off-by: yuan.xiong <yuan.xiong@intel.com>
@yuanxion
Get 'depth' value from then second input ('Select' node) of OneHot no…
cf44f44 
…de during inference

Signed-off-by: yuan.xiong <yuan.xiong@intel.com>
@yuanxion
Use empty const_data map to get dynamic output_shapes when no depth v…
c6e8f84 
…alue

Signed-off-by: yuan.xiong <yuan.xiong@intel.com>
@yuanxion
Update const_data map of one-hot instead of re-initialize it
76b82cc 
Signed-off-by: yuan.xiong <yuan.xiong@intel.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
category: GPU OpenVINO GPU plugin ExternalPR External contributor
Projects
None yet
Milestone
2024.6
Development

Successfully merging this pull request may close these issues.
3 participants
@yuanxion @sshlyapn @sys-openvino-ci