Pawel Raasz

Results 5 comments of Pawel Raasz

> @praasz, please comment on value outside Kaldi. Kaldi support was stopped a year ago. I see setting shapes for variables reasonable in the context that we started to produce...

@jane-intel could check if anything from ticket [CVS-94507](https://jira.devtools.intel.com/browse/CVS-94507) is missing. I think current shape inference works correctly and recalculate concatenation axis on each use. Added test to verify it.

> @praasz the problem was about this: https://github.com/openvinotoolkit/openvino/blob/master/src/core/src/op/concat.cpp#L45 https://github.com/openvinotoolkit/openvino/blob/master/src/core/include/openvino/op/concat.hpp#L43 https://github.com/openvinotoolkit/openvino/blob/master/src/core/include/openvino/op/concat.hpp#L60:L63 > ... Sure we can review if setter/getter could be marked as deprecated, I assume deprecation message should be like...

> General questions: > > * Why did we pick to use strings to identify variables? We could've used indices or instances instead. It looks like name is user friendly...