openvino icon indicating copy to clipboard operation
openvino copied to clipboard

Fix Subgraph serialization/de-serialization

Open mmikolajcz opened this issue 2 years ago • 4 comments

Details:

  • Add default constructor for Subgraph object
  • Add Subgraph operation into extensions
  • Fix Subgraph attribute serialization

Tickets:

  • 63594

mmikolajcz avatar Jul 04 '22 15:07 mmikolajcz

@chenhu-wang Could you please take a review? thanks!

yuxu42 avatar Jul 06 '22 07:07 yuxu42

Is it possible to add test to cover this kind of serialization/deserialization failure in https://github.com/openvinotoolkit/openvino/tree/master/src/tests/functional/inference_engine/serialization?

chenhu-wang avatar Jul 07 '22 02:07 chenhu-wang

Thanks for review, I'm looking on how to test this case

mmikolajcz avatar Jul 08 '22 15:07 mmikolajcz

@IvanNovoselov could you please take a look as well?

dmitry-gorokhov avatar Aug 03 '22 07:08 dmitry-gorokhov

The changes will be merged into master as part of #14064

v-Golubev avatar Nov 18 '22 09:11 v-Golubev

Implemented in https://github.com/openvinotoolkit/openvino/pull/14064

dmitry-gorokhov avatar Dec 23 '22 10:12 dmitry-gorokhov