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

[C API] Add ov_core_add_extension C API #26670

Open
wants to merge 13 commits into
base: master
Choose a base branch
from

Conversation

rahulchaphalkar
Copy link

Details:

  • Added ov_core_add_extension to C API
  • Added test

Discussed here #26645

@rahulchaphalkar rahulchaphalkar requested a review from a team as a code owner September 19, 2024 00:47
@github-actions github-actions bot added the category: C API OpenVINO C API bindings label Sep 19, 2024
@sys-openvino-ci sys-openvino-ci added the ExternalIntelPR External contributor from Intel label Sep 19, 2024
src/bindings/c/src/ov_core.cpp Show resolved Hide resolved
src/bindings/c/tests/ov_test.hpp Outdated Show resolved Hide resolved
@rahulchaphalkar rahulchaphalkar requested a review from a team as a code owner September 21, 2024 00:32
@rahulchaphalkar rahulchaphalkar requested review from tsavina and removed request for a team September 21, 2024 00:32
@github-actions github-actions bot added the category: docs OpenVINO documentation label Sep 21, 2024
@github-actions github-actions bot added the category: build OpenVINO cmake script / infra label Sep 24, 2024
Copy link
Contributor

@riverlijunjie riverlijunjie left a comment

Choose a reason for hiding this comment

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

LGTM

@rahulchaphalkar
Copy link
Author

Any other changes required here? Thanks.

@rahulchaphalkar
Copy link
Author

Does CI need to rerun here? Or is this waiting on additional reviews?

@praasz praasz enabled auto-merge October 25, 2024 06:56
@praasz
Copy link
Contributor

praasz commented Oct 25, 2024

build_jenkins

@praasz praasz added this pull request to the merge queue Oct 29, 2024
@akladiev akladiev removed this pull request from the merge queue due to a manual request Oct 29, 2024
Copy link
Contributor

github-actions bot commented Nov 9, 2024

This PR will be closed in a week because of 2 weeks of no activity.

@github-actions github-actions bot added the Stale label Nov 9, 2024
@rahulchaphalkar
Copy link
Author

Can someone point me to logs for failures? I don't have access to logs for ie_tests_cldnn_unit_dg2_ubuntu22_release and couple other failing tests here. I can only see ci/jenkins logs, which are

[2024-10-25T07:03:03.693Z] Scheduling project: private-ci » ie » build-linux-ubuntu20
[2024-10-25T07:03:11.707Z] Starting building: private-ci » ie » build-linux-ubuntu20 #206827
[2024-10-25T09:27:27.703Z] Build private-ci » ie » build-linux-ubuntu20  | type: release | repo: openvino | branch: refs/pull/26670/head | #206827 completed: UNSTABLE

@github-actions github-actions bot removed the Stale label Nov 16, 2024
Copy link
Contributor

This PR will be closed in a week because of 2 weeks of no activity.

@github-actions github-actions bot added the Stale label Nov 30, 2024
@github-actions github-actions bot removed the Stale label Dec 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category: build OpenVINO cmake script / infra category: C API OpenVINO C API bindings category: docs OpenVINO documentation ExternalIntelPR External contributor from Intel
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants