Skip to content

Video: update download link for NanoTrack#22951

Merged
opencv-pushbot merged 1 commit intoopencv:4.xfrom
zihaomu:update_nanotrack_comment
Dec 14, 2022
Merged

Video: update download link for NanoTrack#22951
opencv-pushbot merged 1 commit intoopencv:4.xfrom
zihaomu:update_nanotrack_comment

Conversation

@zihaomu
Copy link
Copy Markdown
Member

@zihaomu zihaomu commented Dec 13, 2022

Merge this PR with: opencv/opencv_extra#1025.

Update model download link without code change.
NanotrackV2 is faster than NanoTrackV1. At the same time, the accuracy rate is improved.

Test at Apple M1:

Model Version Speed VOT2018 EAO (higher is better)
NanoTrackV1 150 FPS 0.311
NanoTrackV2 200 FPS 0.352

Pull Request Readiness Checklist

See details at https://github.com/opencv/opencv/wiki/How_to_contribute#making-a-good-pull-request

  • I agree to contribute to the project under Apache 2 License.
  • To the best of my knowledge, the proposed patch is not based on a code under GPL or another license that is incompatible with OpenCV
  • The PR is proposed to the proper branch
  • There is a reference to the original bug report and related work
  • There is accuracy test, performance test and test data in opencv_extra repository, if applicable
    Patch to opencv_extra has the same branch name.
  • The feature is well documented and sample code can be built with the project CMake

@asmorkalov
Copy link
Copy Markdown
Contributor

It makes sense to update the model in tests too.

@zihaomu
Copy link
Copy Markdown
Member Author

zihaomu commented Dec 13, 2022

Hi @asmorkalov, V1 and V2 have similar structures. And totally the same pre-process and post-process. I'm not sure if it is necessary.

@asmorkalov
Copy link
Copy Markdown
Contributor

Performance improvement says opposite. It's not just a retrained model. There are some changes in architecture or at least channels.

@zihaomu zihaomu force-pushed the update_nanotrack_comment branch 2 times, most recently from b75cb88 to d57f6d0 Compare December 13, 2022 13:11
@zihaomu
Copy link
Copy Markdown
Member Author

zihaomu commented Dec 14, 2022

The final decision, we keep both Nanotrack v1 and v2 models at regression test. Since the model is quite lightweight, it will not bring too much burden to CI.

Copy link
Copy Markdown
Contributor

@asmorkalov asmorkalov left a comment

Choose a reason for hiding this comment

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

👍

@asmorkalov asmorkalov self-assigned this Dec 14, 2022
@asmorkalov asmorkalov added this to the 4.7.0 milestone Dec 14, 2022
Copy link
Copy Markdown
Member

@alalek alalek left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@asmorkalov
Copy link
Copy Markdown
Contributor

@zihaomu please squash the commits. Waiting for OpenCV Extra PR approve.

@zihaomu zihaomu force-pushed the update_nanotrack_comment branch from b1b2b83 to 7dbb125 Compare December 14, 2022 06:42
@alalek
Copy link
Copy Markdown
Member

alalek commented Dec 14, 2022

BTW, it makes sense to add performance tests of dnn for these (and other) models: https://github.com/opencv/opencv/blob/4.x/modules/dnn/perf/perf_net.cpp

(the last model was added there in 2020)

@zihaomu
Copy link
Copy Markdown
Member Author

zihaomu commented Dec 14, 2022

Hi @alalek, the tracking model is different. It consists of two models, the first model for extracting first frame features, and the second model for extracting and locating the object.
The first mode expected input is an image, and the second model expected two muti-dimension Mat as inputs. If we want to add the performance test, we need to test these two models at single performance test.

BTW, because Nanotrack V2 is superior to V1 in both speed and accuracy a lot, we do not expect users to choose between using Nanotrack V1 or V2. We only want users to use the V2 version of tracking.

@zihaomu zihaomu changed the title Video: update model download link for NanoTrackV2 Video: update download link for NanoTrack Dec 14, 2022
@opencv-pushbot opencv-pushbot merged commit db4a557 into opencv:4.x Dec 14, 2022
@alalek alalek mentioned this pull request Jan 8, 2023
@ycxia
Copy link
Copy Markdown

ycxia commented Sep 14, 2023

@zihaomu, support nanotrackv3?
Somthing wrong when i change model to nanotrackv3.
image

How to deal? thank you

@zihaomu
Copy link
Copy Markdown
Member Author

zihaomu commented Sep 14, 2023

Hi @ycxia, v3 has some changes, but not much. Will try to add nanotrack v3 next week.

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.

5 participants