Hubert Błaszczyk

Results 29 comments of Hubert Błaszczyk

Hi @mohammad7t, I agree that existing code can support that operations using loop on named_parameters. To be honest I didn't checked if it works before I started implementing the enhancement....

Thank you for clarification! Now, I get it and I agree with everything you said. Regarding the ">" selector, I'm not sure if it's unnecessary when applying bfs. The way...

Hi @almilosz @vishniakov-nikolai, I have encountered an issue while testing the implementation of `CompiledModel::set_property()` ``` Exception from src/inference/src/cpp/compiled_model.cpp:136: Exception from src/plugins/intel_cpu/src/compiled_model.h:39: Not Implemented: It's not possible to set property of...

Hi @nashez, the issue is still under development. I have everything implemented but this test for `set_property()`. I'm waiting for a response to my issue.

@almilosz I found that if you change `1` in `cm.set_property({'AUTO_BATCH_TIMEOUT': 1})` to be a string it doesn't throw error so I guess there is a problem with converting this variable...

I looked into the problem with conversion and I think I found the reason why. https://github.com/openvinotoolkit/openvino/blob/dd2f6141b0e162789332bfccc7dfaa90d6584d2a/src/bindings/js/node/src/helper.cpp#L153-L180 `js_to_cpp` which is used in to_anyMap returns `int32_t` But in auto-batch `CompiledModel::set_property` we can...

Hi @yuhuishi-convect, please change `NO` to `YES` in the line below. Without it CI can't run some tests. https://github.com/openvinotoolkit/openvino/blob/538374f94d73a3dcd2393bfb0b0b11a9e2130472/src/frontends/tensorflow/docs/supported_ops.md?plain=1#L1010

@mlukasze @mvafin This issue can be closed now because of PR #26162.