The check is removed and CI is green. Looks ready to merge.

@ilyachur, I tried to implement the changes with disable reshape as we discussed yesterday. It turned out that in order to use GenericIE in serialization file I need to add plugin_api to cmake. Does it changes something or is it ok to add this dependency?

Could you please add new tests to cover this bug?

Could you please add new tests to cover this bug?

The initial report says it will be issue with NMS. So @bszmelcz let's try to reproduce this issue with NMS by introducing Serialization SLT for it, and then check if your fix is resolving issue.

Hi, I realized that we merged #3475 but the fix was actually wrong because node->is_dynamic() method checks only input shapes but not output. So it won’t work for NMS which always produce dynamic output shape. I think we need to remove this check at all.

Could you please add new tests to cover this bug?

The initial report says it will be issue with NMS. So @bszmelcz let's try to reproduce this issue with NMS by introducing Serialization SLT for it, and then check if your fix is resolving issue.

Hi, I realized that we merged #3475 but the fix was actually wrong because node->is_dynamic() method checks only input shapes but not output. So it won’t work for NMS which always produce dynamic output shape. I think we need to remove this check at all.

I wasn't right about removing this check at all. See my comment above.

@GlebKazantaev I added the SLT for serialization for NMS, but it seems like that all the tests are passing. Could you give me some hint what might be wrong, since I cannot reproduce the bug?

@ilyachur I think we can merge, reported issue was reproduced with test and fixed. Additionally we got Serialization SLT for NMS op.