Skip to content

Merge test cases in OVClassModelTestP to OVClassQueryModelTest #26197

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
Closed
wants to merge 2 commits into from
Closed

Merge test cases in OVClassModelTestP to OVClassQueryModelTest #26197

wants to merge 2 commits into from

Conversation

@peterchen-intel
Copy link
Contributor

@peterchen-intel peterchen-intel commented Aug 23, 2024 •
edited
Loading

Details:

  • There are only 2 query_model test cases in OVClassModelTestP
    -- QueryModelActualNoThrow
    -- QueryModelWithKSO
  • Move them to OVClassQueryModelTest
  • Delete empty OVClassModelTestP

Tickets:

  • N/A

@peterchen-intel
Merge OVClassModelTestP to OVClassQueryModelTest
2e8b496 
There are only 2 query_model test cases in OVClassModelTestP
QueryModelActualNoThrow
QueryModelWithKSO

Signed-off-by: Chen Peter <[email protected]>
@peterchen-intel peterchen-intel requested review from a team as code owners August 23, 2024 08:03
@github-actions github-actions bot added category: IE Tests OpenVINO Test: plugins and common category: GPU OpenVINO GPU plugin category: TEMPLATE OpenVINO Template plugin category: NPU OpenVINO NPU plugin labels Aug 23, 2024
@peterchen-intel peterchen-intel changed the title Merge OVClassModelTestP to OVClassQueryModelTest Merge test cases in OVClassModelTestP to OVClassQueryModelTest Aug 23, 2024
@ilya-lavrenov ilya-lavrenov added this to the 2024.4 milestone Aug 23, 2024
@iefode
Copy link
Contributor

iefode commented Aug 26, 2024

Many thanks for removing of extra lines!

@iefode iefode enabled auto-merge August 26, 2024 10:16
@peterchen-intel
Copy link
Contributor Author

peterchen-intel commented Aug 26, 2024

Blocked by openvinotoolkit/openvino_contrib#928

This was referenced Aug 26, 2024
Open
Closed
@github-actions
Copy link
Contributor

github-actions bot commented Oct 17, 2024

This PR will be closed in a week because of 2 weeks of no activity.

@github-actions github-actions bot added the Stale label Oct 17, 2024
@ilya-lavrenov ilya-lavrenov removed this from the 2024.4 milestone Oct 21, 2024
@github-actions github-actions bot removed the Stale label Oct 22, 2024
@github-actions
Copy link
Contributor

github-actions bot commented Nov 6, 2024

This PR will be closed in a week because of 2 weeks of no activity.

@github-actions github-actions bot added the Stale label Nov 6, 2024
@github-actions
Copy link
Contributor

github-actions bot commented Nov 13, 2024

This PR was closed because it has been stalled for 2 week with no activity.

@github-actions github-actions bot closed this Nov 13, 2024
auto-merge was automatically disabled November 13, 2024 00:23

Pull request was closed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@sbalandi sbalandi sbalandi approved these changes

+1 more reviewer

@iefode iefode iefode approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@iefode iefode

@sbalandi sbalandi

Labels

category: GPU OpenVINO GPU plugin category: IE Tests OpenVINO Test: plugins and common category: NPU OpenVINO NPU plugin category: TEMPLATE OpenVINO Template plugin Stale

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@peterchen-intel @iefode @sbalandi @ilya-lavrenov