Skip to content

[Logger] Add dev_api log callback setup #31093

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

Conversation

t-jankowski
Copy link
Contributor

@t-jankowski t-jankowski commented Jun 24, 2025

Details:

  • Adds log dispatching functionality; to either std::cout or callback function.
  • Applies log dispatch to ov::util::LogHelper with a trick. Common/Util is stateless, independent from Core and unique per component. But log dispatching must be stateful and single to all the components. Though the declaration to exported function is out of its definition component.

Tickets:

Signed-off-by: Tomasz Jankowski <tomasz1.jankowski@intel.com>
Signed-off-by: Tomasz Jankowski <tomasz1.jankowski@intel.com>
Signed-off-by: Tomasz Jankowski <tomasz1.jankowski@intel.com>
Signed-off-by: Tomasz Jankowski <tomasz1.jankowski@intel.com>
Signed-off-by: Tomasz Jankowski <tomasz1.jankowski@intel.com>
Signed-off-by: Tomasz Jankowski <tomasz1.jankowski@intel.com>
Signed-off-by: Tomasz Jankowski <tomasz1.jankowski@intel.com>
Signed-off-by: Tomasz Jankowski <tomasz1.jankowski@intel.com>
@github-actions github-actions bot added the category: Core OpenVINO Core (aka ngraph) label Jun 24, 2025
Signed-off-by: Tomasz Jankowski <tomasz1.jankowski@intel.com>
Signed-off-by: Tomasz Jankowski <tomasz1.jankowski@intel.com>
to avoid -Werror=missing-declarations in CI

Signed-off-by: Tomasz Jankowski <tomasz1.jankowski@intel.com>
@t-jankowski t-jankowski marked this pull request as ready for review June 25, 2025 09:14
@t-jankowski t-jankowski requested review from a team as code owners June 25, 2025 09:14
Signed-off-by: Tomasz Jankowski <tomasz1.jankowski@intel.com>
Signed-off-by: Tomasz Jankowski <tomasz1.jankowski@intel.com>
Debugging in CI - no repro locally.

Signed-off-by: Tomasz Jankowski <tomasz1.jankowski@intel.com>
Signed-off-by: Tomasz Jankowski <tomasz1.jankowski@intel.com>
Signed-off-by: Tomasz Jankowski <tomasz1.jankowski@intel.com>
@t-jankowski t-jankowski closed this Jul 1, 2025
@t-jankowski
Copy link
Contributor Author

t-jankowski commented Jul 1, 2025

Closed as replaced with #31165

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category: Core OpenVINO Core (aka ngraph)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant