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.

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

[PyOV] Extend Python API with Squeeze-15 #27281

Merged
merged 49 commits into from
Oct 31, 2024

Conversation

p-wysocki
Copy link
Contributor

@p-wysocki p-wysocki commented Oct 28, 2024

Details:

Tickets:



@pytest.mark.parametrize(("input_shape", "axes", "allow_axis_skip", "expected_shape"), [
((1, 2, 1, 3, 1, 1), [2, 4], True, [1, 2, 3, 1]),
Copy link
Contributor

Choose a reason for hiding this comment

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

what will be if axis corresponds to static dimension not equal to 1 for allow_axis_skip True and False?

Copy link
Contributor Author

@p-wysocki p-wysocki Oct 28, 2024

Choose a reason for hiding this comment

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

In that case the dimension will remain unchanged, I just checked on master. @barnasm1 could you please confirm? I also modified the test to cover this case.

@p-wysocki p-wysocki requested a review from rkazants October 29, 2024 10:47
@github-actions github-actions bot removed category: IE Tests OpenVINO Test: plugins and common category: CPU OpenVINO CPU plugin category: build OpenVINO cmake script / infra labels Oct 31, 2024
Signed-off-by: p-wysocki <[email protected]>
@github-actions github-actions bot removed category: Core OpenVINO Core (aka ngraph) category: CPP API OpenVINO CPP API bindings labels Oct 31, 2024
@mlukasze mlukasze enabled auto-merge October 31, 2024 09:01
@mlukasze mlukasze added this pull request to the merge queue Oct 31, 2024
Merged via the queue into openvinotoolkit:master with commit a0b73e0 Oct 31, 2024
166 checks passed
@p-wysocki p-wysocki deleted the squeeze_py branch October 31, 2024 12:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category: Python API OpenVINO Python bindings
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants