Skip to content

[core] Make StreamSerializer and XmlSerializer customizable by other component #31639

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
Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

[core] Make StreamSerializer and XmlSerializer customizable by other component #31639

wants to merge 7 commits into from
+1,456 −1,245

Conversation

praasz
Copy link
Contributor

@praasz praasz commented Aug 7, 2025

Details:

  • Extract the XmlSerializer to dev API component which can apply some specific implementation e.g. weightless usage on CPU.
  • Add protected/private functions (minimum required) to override them in other module.
  • These utils are part of core dev API as serializer is already used in core library.

Related PR:

Tickets:

@praasz
Make StreamSerializer and XmlSerializer possible to customize by othe…
2073aa7 
…r components e.g. plugin

- The XmlSerializer is part of dev API for internal modifications

Signed-off-by: Raasz, Pawel <[email protected]>
@praasz praasz requested a review from nshchego August 7, 2025 09:55
@praasz praasz requested review from a team as code owners August 7, 2025 09:55
@praasz praasz requested review from itikhono and removed request for a team August 7, 2025 09:56
@github-actions github-actions bot added category: Core OpenVINO Core (aka ngraph) category: transformations OpenVINO Runtime library - Transformations category: CPP API OpenVINO CPP API bindings labels Aug 7, 2025
@github-actions github-actions bot added the category: build OpenVINO cmake script / infra label Aug 7, 2025
@praasz
Apply code style
8e0198d 
Signed-off-by: Raasz, Pawel <[email protected]>
t-jankowski
t-jankowski approved these changes Aug 8, 2025
View reviewed changes
@nshchego
Copy link
Contributor

Does it make sense to keep xml_serialize_util and xml_deserialize_util in one place?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@t-jankowski t-jankowski t-jankowski approved these changes

@nshchego nshchego Awaiting requested review from nshchego

@itikhono itikhono Awaiting requested review from itikhono itikhono is a code owner automatically assigned from openvinotoolkit/openvino-ie-transformations-maintainers

Assignees

@nshchego nshchego

Labels
category: build OpenVINO cmake script / infra category: Core OpenVINO Core (aka ngraph) category: CPP API OpenVINO CPP API bindings category: transformations OpenVINO Runtime library - Transformations do_not_merge
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@praasz @nshchego @t-jankowski