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

Add Polygon Primitive #254

Merged
merged 3 commits into from
Jan 16, 2025
Merged

Add Polygon Primitive #254

merged 3 commits into from
Jan 16, 2025

Conversation

ashwinvaidya17
Copy link
Collaborator

What does this PR do?

  • Add Polygon primitive to the visualizer

Fixes # (issue)

Before submitting

  • This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
  • Did you make sure to update the documentation with your changes?
  • Did you write any new necessary tests?

@github-actions github-actions bot added tests Related to tests python python related changes labels Jan 16, 2025
sovrasov
sovrasov previously approved these changes Jan 16, 2025
View reviewed changes
@sovrasov
Copy link
Collaborator

sovrasov commented Jan 16, 2025 •
edited
Loading

@ashwinvaidya17 please rebase to reflect changes from #253

@ashwinvaidya17
Refactor primitives
b7037ae 
Signed-off-by: Ashwin Vaidya <[email protected]>
@ashwinvaidya17
Add polygon primitive
8f67398 
Signed-off-by: Ashwin Vaidya <[email protected]>
@ashwinvaidya17
Add docstrings
23426c5 
Signed-off-by: Ashwin Vaidya <[email protected]>
@sovrasov sovrasov enabled auto-merge January 16, 2025 13:03
@sovrasov sovrasov added this pull request to the merge queue Jan 16, 2025
Merged via the queue into openvinotoolkit:master with commit ae3241a Jan 16, 2025
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sovrasov sovrasov sovrasov approved these changes

Assignees
No one assigned
Labels
python python related changes tests Related to tests
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ashwinvaidya17 @sovrasov