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.

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

fix aruco tests #3695

Merged
merged 1 commit into from
Mar 15, 2024
Merged

Conversation

vpisarev
Copy link
Contributor

@vpisarev vpisarev commented Mar 15, 2024

@vpisarev vpisarev requested a review from AleksandrPanov March 15, 2024 06:11
@@ -8,7 +8,7 @@
namespace opencv_test { namespace {


TEST(CV_ArucoTutorial, can_find_singlemarkersoriginal)
TEST(DISABLED_CV_ArucoTutorial, can_find_singlemarkersoriginal)
{
string img_path = cvtest::findDataFile("aruco/singlemarkersoriginal.jpg", false);

Choose a reason for hiding this comment

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

No need to disable, need to use testing API properly.

These files are part of opencv_extra. They can't be "optional" as specified by the second parameter. "optional" is used for files from downloaded shared drive (e.g. DL models).

false should be removed (removed not replaced).

Copy link
Contributor

Choose a reason for hiding this comment

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

removed false

@AleksandrPanov AleksandrPanov force-pushed the disabled_aruco_tutorial_tests branch from b7a86b3 to b9a9931 Compare March 15, 2024 07:29
@opencv-alalek opencv-alalek changed the title temporarily disabled aruco tutorial tests; need to correct the path fix aruco tests Mar 15, 2024
@opencv-alalek opencv-alalek self-assigned this Mar 15, 2024
@opencv-pushbot opencv-pushbot merged commit bf95e79 into opencv:4.x Mar 15, 2024
10 checks passed
@asmorkalov asmorkalov mentioned this pull request Apr 1, 2024
@vpisarev vpisarev deleted the disabled_aruco_tutorial_tests branch September 17, 2024 21:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants