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

[PyOV] Shared memory improvements for Tensor and Constant classes #15814

Merged
merged 8 commits into from
Feb 23, 2023
Merged

[PyOV] Shared memory improvements for Tensor and Constant classes #15814

merged 8 commits into from
Feb 23, 2023

Conversation

jiwaszki
Copy link

@jiwaszki jiwaszki commented Feb 20, 2023 •
edited
Loading

Details:

  • Added new constructors for Constant.
  • New property data for Constant that reflects shared memory effects. Name and concept is borrowed from Tensor class.
  • Added test cases.
  • Cleaned common code and unified it.

Tickets:

  • 96644

@jiwaszki jiwaszki added category: Python API OpenVINO Python bindings WIP work in progress labels Feb 20, 2023
@jiwaszki jiwaszki added this to the 2023.0 milestone Feb 20, 2023
@jiwaszki jiwaszki self-assigned this Feb 20, 2023
@jiwaszki jiwaszki removed the WIP work in progress label Feb 20, 2023
@jiwaszki jiwaszki marked this pull request as ready for review February 20, 2023 18:02
@jiwaszki jiwaszki requested a review from a team as a code owner February 20, 2023 18:02
mvafin
mvafin approved these changes Feb 22, 2023
View reviewed changes
Copy link
Contributor

@mvafin mvafin left a comment

Choose a reason for hiding this comment

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

Please create a ticket to remove API for pointer

@jiwaszki jiwaszki merged commit 3373c67 into openvinotoolkit:master Feb 23, 2023
andrei-cv pushed a commit to andrei-cv/openvino that referenced this pull request Mar 21, 2023
@andrei-cv
[PyOV] Shared memory improvements for Tensor and Constant classes (op…
af238fe 
…envinotoolkit#15814)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@p-wysocki p-wysocki p-wysocki approved these changes

@akuporos akuporos akuporos approved these changes

@mvafin mvafin mvafin approved these changes

@yury-gorbachev yury-gorbachev Awaiting requested review from yury-gorbachev

@slyalin slyalin Awaiting requested review from slyalin

Assignees

@jiwaszki jiwaszki

Labels
category: Python API OpenVINO Python bindings
Projects
None yet
Milestone
2023.0
Development

Successfully merging this pull request may close these issues.
4 participants
@jiwaszki @mvafin @akuporos @p-wysocki