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

Use multi threads in the object tracking example to stabilize the tracking #11

Merged
merged 1 commit into from
Jun 26, 2020

Conversation

ShotaAk
Copy link
Contributor

@ShotaAk ShotaAk commented Jun 26, 2020

Checklists

  • I have read the CONTRIBUTING guidelines.
  • I have checked to ensure there aren't other open Pull Requests for the same change.

What does this implement/fix? Explain your changes.

カメラサンプルでmouseとobject_trackingノードの起動にマルチスレッドを使用します。

これまではシングルスレッドを使っていたため動作が安定しない(動作がフリーズする)ことがありました。

参考:

Does this close any currently open issues?

いいえ。

How has this been tested?

実機で動作確認しました。

@ShotaAk ShotaAk added the Type: Bug Bug or Bug fixes label Jun 26, 2020
@ShotaAk ShotaAk requested a review from Tiryoh June 26, 2020 04:07
@Tiryoh Tiryoh self-assigned this Jun 26, 2020
@ShotaAk ShotaAk added Type: Feature New Feature and removed Type: Bug Bug or Bug fixes labels Jun 26, 2020
@Tiryoh Tiryoh changed the title Use multi threads for object tracking example Use multi threads in the object tracking example to stabilize the tracking Jun 26, 2020
Copy link
Contributor

@Tiryoh Tiryoh left a comment

Choose a reason for hiding this comment

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

lgtm

@Tiryoh
Copy link
Contributor

Tiryoh commented Jun 26, 2020

実機にて動作確認しました。
かなり安定していていいですね!(オレンジのボールがないので適当な画像で代用しています)
Screenshot from 2020-06-26 18-05-52

@Tiryoh Tiryoh merged commit 3f8c15e into master Jun 26, 2020
@Tiryoh Tiryoh deleted the fix_object_tracking branch June 26, 2020 09:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Feature New Feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants