Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[Bug] [C-API]: some non-string inputs of remote context are processed as string #28803

Open
3 tasks done
xygnal opened this issue Feb 4, 2025 · 0 comments
Open
3 tasks done
Assignees
Labels
bug Something isn't working category: C API OpenVINO C API bindings support_request

Comments

@xygnal
Copy link

xygnal commented Feb 4, 2025

OpenVINO Version

Master branch

Operating System

Ubuntu 20.04 (LTS)

Device used for inference

GPU

Framework

None

Model used

No response

Issue description

When I use the remote context of VASurfaceID, ov_remote_context_create_tensor() does not work.

I found the cause in src/bindings/c/src/ov_remote_context.cpp.
check_intel_gpu_property_value_is_ptr( ) classifies inputs into "char *" or "void *".
At line 28-29, the current classification is

  • void *
    : ov_property_key_intel_gpu_ocl_context
    : ov_property_key_intel_gpu_ocl_queue
    : ov_property_key_intel_gpu_va_device
    : ov_property_key_intel_gpu_mem_handle
  • char * : the others

But it does not match the documentation.
https://docs.openvino.ai/2024/openvino-workflow/running-inference/inference-devices-and-modes/gpu-device/remote-tensor-api-gpu-plugin.html
(I confirmed that mis-classification by unintended working with casting VASurafceID and plane number into strings.)

ov_property_key_intel_gpu_dev_object_handle
ov_property_key_intel_gpu_va_plane
The 2 properties should be added to void * class.

(ov_property_key_intel_gpu_ocl_context_device_id is not shown in the documentation. Please check.)

Thanks.

Step-by-step reproduction

No response

Relevant log output

Issue submission checklist

  • I'm reporting an issue. It's not a question.
  • I checked the problem with the documentation, FAQ, open issues, Stack Overflow, etc., and have not found a solution.
  • There is reproducer code and related data files such as images, videos, models, etc.
@xygnal xygnal added bug Something isn't working support_request labels Feb 4, 2025
@ilya-lavrenov ilya-lavrenov added the category: C API OpenVINO C API bindings label Feb 4, 2025
@ilya-lavrenov ilya-lavrenov changed the title [Bug]: some non-string inputs of remote context are processed as string [Bug] [C-API]: some non-string inputs of remote context are processed as string Feb 4, 2025
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
bug Something isn't working category: C API OpenVINO C API bindings support_request
Projects
None yet
Development

No branches or pull requests

6 participants