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.

Sign up for GitHub

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

Jump to bottom

fix human_detector.launch #1

Closed
wants to merge 1 commit into from
Closed

fix human_detector.launch #1

wants to merge 1 commit into from

Conversation

knorth55
Copy link

@knorth55 knorth55 commented Dec 2, 2022

this PR fixes human_detector.launch

@knorth55 knorth55 mentioned this pull request Dec 2, 2022
Open
@AyumuIwa
Copy link
Owner

AyumuIwa commented Dec 3, 2022

@knorth55
提案していただいた変更を加えてみると、カメラに対して奥行き方向だけでなく左右方向の移動もきちんと認識し計算されるようになりました。ありがとうございます。
※改良版をlaunchディレクトリにhuman_detector2.launchとして追加しました。

今後、実世界での距離とRvizの座標上での距離の対応関係を確認し新たなプログラムを書いていこうと思います。

@knorth55
Copy link
Author

knorth55 commented Dec 3, 2022

無事に動いたみたいでよかったです！
変更としては、画像のリサイズや座標系変換が間違っていて、それを修正しました

@AyumuIwa
Copy link
Owner

AyumuIwa commented Dec 4, 2022

@knorth55
ありがとうございます。変更点を追って理解してみようと思います。

@knorth55 knorth55 closed this Dec 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@knorth55 @AyumuIwa