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

[Tmpl test] CumSum: enable whole Tensor comparison #23862

Merged
merged 20 commits into from
Apr 10, 2024
Merged

[Tmpl test] CumSum: enable whole Tensor comparison #23862

merged 20 commits into from
Apr 10, 2024

Conversation

t-jankowski
Copy link
Contributor

Details:

  • Used actual shapes for Tensors creation.

Tickets:

t-jankowski and others added 20 commits March 12, 2024 11:39
@t-jankowski
Enhance ref vs out data comparison in template test
74a6436
@t-jankowski
Enhance ref vs out data comparison in template test
213bfc0
@t-jankowski
Add Tensor comparizon
2bfcf1a
@t-jankowski
Enhance ref vs out data comparison in template test
b12e3a8
@t-jankowski
Keep legacy comparison
8c8cbff
@t-jankowski
Fix style
e15d719
@t-jankowski
Use legacy comparison for Tan ref test
01af380
@t-jankowski
Merge remote-tracking branch 'upstream/master' into tj/plugin/templat…
890ae27 
…e/test/compare-tensor
@t-jankowski
Merge remote-tracking branch 'upstream/master' into tj/plugin/templat…
dd10733 
…e/test/compare-tensor
@t-jankowski
Use leagcy comparison for MaxPool and Mish
e4d8f06
@t-jankowski
Merge remote-tracking branch 'upstream/master' into tj/plugin/templat…
719398e 
…e/test/compare-tensor
@mlukasze
Merge branch 'master' into tj/plugin/template/test/compare-tensor
e05e0b8
@t-jankowski
Merge branch 'master' into tj/plugin/template/test/compare-tensor
077af8a
@t-jankowski
Merge branch 'master' into tj/plugin/template/test/compare-tensor
5561073
@t-jankowski
Merge branch 'master' into tj/plugin/template/test/compare-tensor
6df849d
@t-jankowski
Merge remote-tracking branch 'upstream/master' into tj/plugin/templat…
7abbbd1 
…e/test/compare-tensor
@mlukasze
Merge branch 'master' into tj/plugin/template/test/compare-tensor
0a39f97
@t-jankowski
Merge branch 'master' into tj/plugin/template/test/compare-tensor
46c4a3d
@t-jankowski
Enable whole Tensor comparison
94cb70d
@t-jankowski
Merge remote-tracking branch 'upstream/master' into tj/plugin/templat…
12667fe 
…e/test/compare-tensor-cumsum
@t-jankowski t-jankowski requested a review from a team as a code owner April 4, 2024 09:27
@github-actions github-actions bot added the category: TEMPLATE OpenVINO Template plugin label Apr 4, 2024
@t-jankowski t-jankowski requested review from olpipi, vurusovs and praasz and removed request for a team April 4, 2024 09:28
@olpipi
Copy link
Contributor

olpipi commented Apr 5, 2024

Don't we plan to test dynamic scenarios here?

@t-jankowski
Copy link
Contributor Author

Don't we plan to test dynamic scenarios here?

Template Plugin test bases on CommonReferenceTest which uses ov::Tensor and PartialShape brings no usage. All changed shapes were static.
To have dynamic test a new dedicate base class should be created, but I'm not aware of any plan for such test.

@t-jankowski t-jankowski added this pull request to the merge queue Apr 10, 2024
Merged via the queue into openvinotoolkit:master with commit 7a31382 Apr 10, 2024
108 checks passed
bbielawx pushed a commit to bbielawx/openvino that referenced this pull request Apr 12, 2024
@t-jankowski @mlukasze @bbielawx
[Tmpl test] CumSum: enable whole Tensor comparison (openvinotoolkit#2…
a9d4ab5 
…3862)

### Details:
 - Used actual shapes for Tensors creation.

### Tickets:
 - CVS-137153

---------

Co-authored-by: Michal Lukaszewski <michal.lukaszewski@intel.com>
alvoron pushed a commit to alvoron/openvino that referenced this pull request Apr 29, 2024
@t-jankowski @mlukasze @alvoron
[Tmpl test] CumSum: enable whole Tensor comparison (openvinotoolkit#2…
4789cae 
…3862)

### Details:
 - Used actual shapes for Tensors creation.

### Tickets:
 - CVS-137153

---------

Co-authored-by: Michal Lukaszewski <michal.lukaszewski@intel.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@olpipi olpipi olpipi approved these changes

@vurusovs vurusovs Awaiting requested review from vurusovs

@praasz praasz Awaiting requested review from praasz

Assignees
No one assigned
Labels
category: TEMPLATE OpenVINO Template plugin
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@t-jankowski @olpipi @mlukasze