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] Extend Python API with ROIAlignRotated-15 #25755

Merged
merged 5 commits into from
Aug 8, 2024
Merged

[PyOV] Extend Python API with ROIAlignRotated-15 #25755

merged 5 commits into from
Aug 8, 2024

Conversation

hub-bla
Copy link
Contributor

@hub-bla hub-bla commented Jul 26, 2024

Details:

  • Add Python API for ROIAlignRotated-15

Tickets:

  • 141569

@hub-bla hub-bla requested a review from a team as a code owner July 26, 2024 13:21
@github-actions github-actions bot added the category: Python API OpenVINO Python bindings label Jul 26, 2024
@sys-openvino-ci sys-openvino-ci added the ExternalPR External contributor label Jul 26, 2024
@hub-bla hub-bla force-pushed the roi-align-rotated-python branch from 5be7227 to 3496513 Compare July 26, 2024 13:22
@mitruska mitruska self-requested a review July 26, 2024 13:33
@mitruska mitruska linked an issue Jul 26, 2024 that may be closed by this pull request
[Good First Issue]: Extend Python API with ROIAlignRotated-15 #25325
Closed
mitruska
mitruska approved these changes Jul 26, 2024
View reviewed changes
Copy link
Contributor

@mitruska mitruska left a comment

Choose a reason for hiding this comment

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

Thank you for your contribution 🚀
This PR LGTM.

src/bindings/python/src/openvino/runtime/opset15/ops.py Show resolved Hide resolved
src/bindings/python/tests/test_graph/test_create_op.py Show resolved Hide resolved
@mitruska mitruska self-assigned this Jul 26, 2024
@mitruska mitruska added the category: Opset OpenVINO Opset label Jul 26, 2024
p-wysocki
p-wysocki approved these changes Jul 26, 2024
View reviewed changes
Copy link
Contributor

@p-wysocki p-wysocki left a comment

Choose a reason for hiding this comment

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

Looks great, thank you for your contribution!

@hub-bla
Copy link
Contributor Author

hub-bla commented Jul 26, 2024 •
edited
Loading

Thank you for your dedication to developing and maintaining. I'm genuinely impressed by how well everything is organized :)

@mlukasze mlukasze enabled auto-merge July 29, 2024 04:43
@mlukasze
Copy link
Contributor

build_jenkins

auto-merge was automatically disabled July 29, 2024 07:40

Head branch was pushed to by a user without write access

@hub-bla hub-bla force-pushed the roi-align-rotated-python branch from 3496513 to 8a4abd9 Compare July 29, 2024 07:40
@mlukasze
Copy link
Contributor

build_jenkins

@mitruska mitruska enabled auto-merge July 29, 2024 08:46
@hub-bla
Copy link
Contributor Author

hub-bla commented Jul 29, 2024

Hey @mlukasze, I noticed that there's a failed Jenkins check. Could you provide me with some hints on how to fix this issue? Thanks!

@mitruska
Copy link
Contributor

build_jenkins

@mitruska
Copy link
Contributor

Could you provide me with some hints on how to fix this issue? Thanks!

@hub-bla That looks like an infrastructure issue, update to the master branch and CI rebuild should help (already done).
It's on our side, further action is not required so far:) Thanks!

@mitruska mitruska added this pull request to the merge queue Jul 29, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jul 29, 2024
@mlukasze mlukasze added this pull request to the merge queue Jul 30, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jul 30, 2024
@mlukasze
Copy link
Contributor

build_jenkins

@mlukasze mlukasze enabled auto-merge July 30, 2024 12:34
@mlukasze mlukasze added this pull request to the merge queue Jul 30, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jul 30, 2024
@hub-bla
Copy link
Contributor Author

hub-bla commented Aug 8, 2024

@mitruska It looks like the CI process has stalled on two ubuntu checks. I took a quick glance at them yesterday, and there haven’t been any changes since then.

@mitruska
Copy link
Contributor

mitruska commented Aug 8, 2024

build_jenkins

@mitruska mitruska added this pull request to the merge queue Aug 8, 2024
Merged via the queue into openvinotoolkit:master with commit cb09e4f Aug 8, 2024
141 checks passed
@mitruska mitruska mentioned this pull request Aug 8, 2024
Closed
@mlukasze mlukasze added this to the 2024.4 milestone Aug 9, 2024
mory91 pushed a commit to mory91/openvino that referenced this pull request Aug 13, 2024
@hub-bla @mitruska
@mlukasze @mory91
[PyOV] Extend Python API with ROIAlignRotated-15 (openvinotoolkit#25755)
64ffd54 
### Details:
 - Add Python API for `ROIAlignRotated-15`

### Tickets:
 - 141569

---------

Co-authored-by: Katarzyna Mitrus <[email protected]>
Co-authored-by: Michal Lukaszewski <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mitruska mitruska mitruska approved these changes

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

Assignees

@mitruska mitruska

Labels
category: Opset OpenVINO Opset category: Python API OpenVINO Python bindings ExternalPR External contributor
Projects
None yet
Milestone
2024.4
Development

Successfully merging this pull request may close these issues.

[Good First Issue]: Extend Python API with ROIAlignRotated-15
5 participants
@hub-bla @mlukasze @mitruska @p-wysocki @sys-openvino-ci