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

[PT FE] Add tests for detectron2 models #19888

Merged
merged 4 commits into from
Sep 18, 2023
Merged

[PT FE] Add tests for detectron2 models #19888

merged 4 commits into from
Sep 18, 2023

Conversation

mvafin
Copy link
Contributor

@mvafin mvafin commented Sep 15, 2023

Details:

  • Add tests for detectron2 models

Tickets:

  • 120265

@mvafin mvafin requested review from a team as code owners September 15, 2023 18:07
@mvafin mvafin requested review from AlexKoff88 and eaidova and removed request for a team September 15, 2023 18:07
@github-actions github-actions bot added category: CI OpenVINO public CI category: dependency_changes Pull requests that update a dependency file labels Sep 15, 2023
rkazants
rkazants requested changes Sep 16, 2023
View reviewed changes
Copy link
Member

@rkazants rkazants left a comment

Choose a reason for hiding this comment

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

Please revert add check that both results ov_res and fw_res to be the same

@github-actions github-actions bot added the github_actions Pull requests that update GitHub Actions code label Sep 18, 2023
@mvafin mvafin merged commit a4cbac3 into openvinotoolkit:master Sep 18, 2023
@mvafin mvafin deleted the mvafin/pt_fe/det2_tests branch September 18, 2023 11:40
alvoron pushed a commit to alvoron/openvino that referenced this pull request Nov 6, 2023
@mvafin @rkazants @alvoron
[PT FE] Add tests for detectron2 models (openvinotoolkit#19888)
ae7495d 
* [PT FE] Add tests for detectron2 models

* Fix names of tests

* Apply suggestions from code review

Co-authored-by: Roman Kazantsev <[email protected]>

* Create secondary requirements file

---------

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

@akladiev akladiev akladiev left review comments

@ilya-lavrenov ilya-lavrenov ilya-lavrenov approved these changes

@rkazants rkazants rkazants approved these changes

@AlexKoff88 AlexKoff88 Awaiting requested review from AlexKoff88 AlexKoff88 is a code owner automatically assigned from openvinotoolkit/openvino-configuration-mgmt

@eaidova eaidova Awaiting requested review from eaidova eaidova is a code owner automatically assigned from openvinotoolkit/openvino-configuration-mgmt

Assignees
No one assigned
Labels
category: CI OpenVINO public CI category: dependency_changes Pull requests that update a dependency file github_actions Pull requests that update GitHub Actions code
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@mvafin @ilya-lavrenov @akladiev @rkazants