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
Additional fixes to 0/1D tests #25487
base: 5.x
Are you sure you want to change the base?
Conversation
modules/dnn/test/test_layers_1d.cpp
Outdated
|
||
ASSERT_EQ(shape(output_ref), shape(outputs[0])); | ||
ASSERT_EQ(1, outputs.size()); | ||
ASSERT_EQ(shape(outputs[0]), shape(output_ref)); |
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.
Reference should be put in the first parameter and real output goes to the second.
See #25231 (comment).
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.
Done
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, I mean it should be
ASSERT_EQ(1, outputs.size());
ASSERT_EQ(shape(output_ref), shape(outputs[0]));
std::string opr = get<1>(GetParam()); | ||
|
||
LayerParams lp; | ||
lp.type = "Scatter"; | ||
lp.name = "addLayer"; | ||
lp.set("axis", 0); | ||
lp.set("axis", axis); |
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.
axis = input_shape.size() - 1
should also work.
3f05044
to
a4a606f
Compare
a7eefb5
to
a818f25
Compare
Why do you create a new milestone? |
accidentally, sorry |
101dfd6
to
7c59b39
Compare
111d992
to
2e64f43
Compare
std::vector<int>({1, 4}), | ||
std::vector<int>({4, 4}) | ||
), | ||
/*operation*/ testing::Values(0, 1) |
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.
/*operation*/ testing::Values(0, 1) | |
/*axis*/ testing::Values(0, 1) |
This has additional fixes requited for 0/1D tests.
Pull Request Readiness Checklist
See details at https://github.com/opencv/opencv/wiki/How_to_contribute#making-a-good-pull-request
Patch to opencv_extra has the same branch name.