Skip to content

[PYTHON API] fix tensor naming #8918

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
Merged
merged 7 commits into from
Dec 1, 2021
Merged

[PYTHON API] fix tensor naming #8918

merged 7 commits into from
Dec 1, 2021

Conversation

AlexeyLebedev1
Copy link
Contributor

Tickets:

  • 72349

@AlexeyLebedev1 AlexeyLebedev1 requested review from akuporos, jiwaszki and a team November 30, 2021 09:03
popovaan
popovaan reviewed Nov 30, 2021
View reviewed changes
src/bindings/python/src/openvino/ie_api.py Outdated
@@ -38,7 +38,7 @@ def normalize_inputs(py_dict: dict, py_types: dict) -> dict:

def get_input_types(obj: Union[InferRequestBase, ExecutableNetworkBase]) -> dict:
"""Get all precisions from object inputs."""
return {i.get_node().get_friendly_name(): i.get_node().get_element_type() for i in obj.inputs}
return {i.get_any_name(): i.get_element_type() for i in obj.inputs}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you add tests for this method? And for infer() method?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done

@openvino-pushbot openvino-pushbot added the category: inference OpenVINO Runtime library - Inference label Nov 30, 2021
@akuporos akuporos added this to the 2022.1 milestone Nov 30, 2021
@akuporos akuporos added bug Something isn't working category: Python API OpenVINO Python bindings and removed category: inference OpenVINO Runtime library - Inference labels Nov 30, 2021
@openvino-pushbot openvino-pushbot added the category: inference OpenVINO Runtime library - Inference label Nov 30, 2021
@jiwaszki jiwaszki self-requested a review November 30, 2021 15:33
@akuporos akuporos merged commit 246e628 into openvinotoolkit:master Dec 1, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@popovaan popovaan popovaan left review comments

@akuporos akuporos akuporos approved these changes

@jiwaszki jiwaszki jiwaszki approved these changes

Assignees
No one assigned
Labels
bug Something isn't working category: inference OpenVINO Runtime library - Inference category: Python API OpenVINO Python bindings
Projects
None yet
Milestone
2022.1
Development

Successfully merging this pull request may close these issues.
5 participants
@AlexeyLebedev1 @akuporos @jiwaszki @popovaan @openvino-pushbot