-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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.
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
Range
, RDFT
, RandormUniform
, ReduceOps
layer tests to API2.0
#20692
Range
, RDFT
, RandormUniform
, ReduceOps
layer tests to API2.0
#20692
Conversation
32cc69c
to
34b1621
Compare
34b1621
to
fc72b67
Compare
@@ -117,7 +117,7 @@ static std::map<ov::NodeTypeInfo, std::vector<std::vector<InputGenerateData>>> i | |||
{ ov::op::v1::FloorMod::get_type_info_static(), {{{2, 4}}, {{2, 2, 128}}} }, | |||
{ ov::op::v1::Mod::get_type_info_static(), {{{2, 4}}, {{2, 2, 128}}} }, | |||
{ ov::op::v1::ReduceMax::get_type_info_static(), {{{0, 5}}, {{-5, 5, 1000}}} }, | |||
{ ov::op::v1::ReduceMean::get_type_info_static(), {{{0, 5}}, {{0, 5, 1000}}} }, | |||
{ ov::op::v1::ReduceMean::get_type_info_static(), {{{0, 5, 1000}}, {{0, 5, 1000}}} }, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@iefode without the change ReduceOpsLayerTest
with Mean
op type fails on integral types
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hmm, what is the values were in original test?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@iefode 0, 5, 1000
for all types (real and integral) were in original test. With 0, 5
only integral cases are failing in new tests
fc72b67
to
9ee28f7
Compare
@@ -117,7 +117,7 @@ static std::map<ov::NodeTypeInfo, std::vector<std::vector<InputGenerateData>>> i | |||
{ ov::op::v1::FloorMod::get_type_info_static(), {{{2, 4}}, {{2, 2, 128}}} }, | |||
{ ov::op::v1::Mod::get_type_info_static(), {{{2, 4}}, {{2, 2, 128}}} }, | |||
{ ov::op::v1::ReduceMax::get_type_info_static(), {{{0, 5}}, {{-5, 5, 1000}}} }, | |||
{ ov::op::v1::ReduceMean::get_type_info_static(), {{{0, 5}}, {{0, 5, 1000}}} }, | |||
{ ov::op::v1::ReduceMean::get_type_info_static(), {{{0, 5, 1000}}, {{0, 5, 1000}}} }, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hmm, what is the values were in original test?
function = std::make_shared<ov::Model>(reduce->outputs(), ov::ParameterVector{param}, "Reduce"); | ||
} | ||
|
||
void ReduceOpsLayerWithSpecificInputTest::generate_inputs(const std::vector<ov::Shape>& targetInputStaticShapes) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
move to basic generator?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No need, there is some specific use case (not for common use in other tests), so I ported it same way
|
…penvinotoolkit#20692) * `Range` to API2.0 * `RDFTLayerTest` to API2.0 * `RandomUniformLayerTest` to API2.0 * `ReduceOpsLayerTest` to API2.0
…penvinotoolkit#20692) * `Range` to API2.0 * `RDFTLayerTest` to API2.0 * `RandomUniformLayerTest` to API2.0 * `ReduceOpsLayerTest` to API2.0
No description provided.