-
Notifications
You must be signed in to change notification settings - Fork 2.7k
[TF FE]: Support MatrixInverse operation for TensorFlow #23881
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
[TF FE]: Support MatrixInverse operation for TensorFlow #23881
Conversation
Good first issue taken by Hongbo on #22957
…/openvino into hongbo/matrix_inverse
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please also update other files and add test.
See other PRs as examples: #23418
Co-authored-by: Roman Kazantsev <roman.kazantsev@intel.com>
Good first issue taken by Hongbo on #22957
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@rkazants Code updated accordingly, please review and guide me, thank you! : )
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
translator gets better. Now please register this translator in op_table etc. See other PRs.
Also test is needed:)
Thx Co-authored-by: Roman Kazantsev <roman.kazantsev@intel.com>
unused variable "outputs", not needed Co-authored-by: Roman Kazantsev <roman.kazantsev@intel.com>
build_jenkins |
build_jenkins |
Looks that we have an issue with |
build_jenkins |
Thank you, Roman! I'm so so so happy that I'm going to have my first real GitHub contribution to be merged. All the best for OpenVINO and Intel. 😃 |
build_jenkins |
build_jenkins |
…kit#23881) This pull request adds support for the TensorFlow "**MatrixInverse**" operation to OpenVINO's TensorFlow Frontend (TF FE). It implements a new loader function translate_matrix_inverse_op that translates the TensorFlow operation into the equivalent OpenVINO Inverse operation (from opset-14) for both real-valued and complex-valued inputs. Addresses issue: #[22957](openvinotoolkit#22957) Hi Roman @rkazants, could you please review my work, thank you! --------- Co-authored-by: Roman Kazantsev <roman.kazantsev@intel.com>
This pull request adds support for the TensorFlow "MatrixInverse" operation to OpenVINO's TensorFlow Frontend (TF FE). It implements a new loader function translate_matrix_inverse_op that translates the TensorFlow operation into the equivalent OpenVINO Inverse operation (from opset-14) for both real-valued and complex-valued inputs.
Addresses issue: #22957