Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
Bounding box fixes for GOTURN tracker #2620
Bounding box fixes for GOTURN tracker #2620
Changes from 17 commits
735511b
63c3cdf
5ab85c1
2dac64a
0d5b40a
229653a
81b3c20
8146256
115a52e
3707382
840992d
b066fd8
ca9e474
7c1148c
743b5e0
e54e573
6303fb9
447cf39
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
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.
Avoid that. We don't testing parameters serialization here.
Use existed straightforward approach:
opencv_contrib/modules/tracking/test/test_trackerParametersIO.cpp
Lines 11 to 19 in e502f30
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.
But I implemented usage of the paths via cv::FileNode. So, this is gonna be no used for the 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.
Test must be simple and straightforward.
At least not in this test with the name "memory_usage".
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.
Please, check last changes:
I was forced to remove cv::FileNode, but instead used
cv::TrackerGOTURN::Params params;
, as @alalek said.Win32/win64 builds are passed, so I think this test is ready.