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

[CORE] Try to correct 'undefined' and 'dynamic' type serialization #28524

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

EgorDuplensky
Copy link
Contributor

@EgorDuplensky EgorDuplensky commented Jan 17, 2025

Details:

  • There is misalignment between ov::element::type string serialization / de-serialization methods
    and the ir serializer itself.
    After introduction of internal opset FullyConnected operation the 'undefined' element type basically
    became valid and serializable
  • Try to update the serializer and see which tests are going to fail
  • There are potential issues with IR backward compatibility which should be addressed carefully

@EgorDuplensky EgorDuplensky requested a review from a team as a code owner January 17, 2025 15:04
@EgorDuplensky EgorDuplensky requested review from itikhono and removed request for a team January 17, 2025 15:04
@github-actions github-actions bot added the category: Core OpenVINO Core (aka ngraph) label Jan 17, 2025
@EgorDuplensky
Copy link
Contributor Author

EgorDuplensky commented Jan 17, 2025

@praasz @slyalin @itikhono FYI

@EgorDuplensky EgorDuplensky force-pushed the try_to_correct_element_type_serialization branch from b16492a to 799051c Compare January 17, 2025 15:13
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