Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[core]Migrate Eye to new API #20258

Merged
merged 6 commits into from
Oct 11, 2023
Merged

[core]Migrate Eye to new API #20258

merged 6 commits into from
Oct 11, 2023

Conversation

praasz
Copy link
Contributor

@praasz praasz commented Oct 5, 2023

Details:

  • Migrate Eye operator to new API. Remove legacy headers, namespace, functions, classes.
  • Refactor Eye for binary size reduction.

Tickets:

@praasz praasz added this to the 2023.2 milestone Oct 5, 2023
@praasz praasz requested a review from a team as a code owner October 5, 2023 08:23
@github-actions github-actions bot added category: Core OpenVINO Core (aka ngraph) category: CPP API OpenVINO CPP API bindings labels Oct 5, 2023
@praasz praasz requested review from mitruska and t-jankowski October 5, 2023 09:08
@praasz praasz requested a review from ilyachur October 9, 2023 09:55
@ilyachur ilyachur merged commit ac11751 into openvinotoolkit:master Oct 11, 2023
40 checks passed
@praasz praasz deleted the api-2.0/migrate-eye-op-to-new-api branch October 12, 2023 07:03
alvoron pushed a commit to alvoron/openvino that referenced this pull request Nov 6, 2023
@praasz @alvoron
[core]Migrate Eye to new API (openvinotoolkit#20258)
9ccbb26 
* Migrate Eye to new API

* Fix `matrix_offset` initialization

* get_tensors_shapes -> get_tensors_partial_shapes
allnes pushed a commit to allnes/openvino that referenced this pull request Nov 23, 2023
@praasz @allnes
[core]Migrate Eye to new API (openvinotoolkit#20258)
edd6079 
* Migrate Eye to new API

* Fix `matrix_offset` initialization

* get_tensors_shapes -> get_tensors_partial_shapes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ilyachur ilyachur ilyachur left review comments

@t-jankowski t-jankowski t-jankowski approved these changes

@mitruska mitruska Awaiting requested review from mitruska

Assignees
No one assigned
Labels
category: Core OpenVINO Core (aka ngraph) category: CPP API OpenVINO CPP API bindings
Projects
None yet
Milestone
2023.2
Development

Successfully merging this pull request may close these issues.
3 participants
@praasz @ilyachur @t-jankowski