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

Update camera line follower: Set motor power with switch input. Add area_threthold param. #51

Merged
merged 4 commits into from
Feb 7, 2024

Conversation

ShotaAk
Copy link
Contributor

@ShotaAk ShotaAk commented Feb 7, 2024

What does this implement/fix?

カメラライントレースサンプルに下記の機能を追加します。

  • スイッチ操作でモータパワーをON/OFFする
  • area_thretholdパラメータを用意する

合わせて、下記の変更を実施しています。

  • 走行開始のしきい値を0.01から0.2に変更
    • シミュレータでは、ラインが40%以上のエリアを満たしているので、その半分をしきい値とします
  • READMEのパラメータ設定方法の誤記修正

Does this close any currently open issues?

いいえ

How has this been tested?

モータパワー

  • 実機でスイッチを操作し、モータパワーがON/OFFすることを確認しました
  • deactivate、cleanupと、シャットダウン操作でモータパワーがOFFすることを確認しました。

area_thretholdパラメータ

  • シミュレータでros2 param set camera_follower area_threshold 0.50を実行し、走行が止まることを確認しました。

Any other comments?

Checklists

@ShotaAk ShotaAk added the Type: Feature New Feature label Feb 7, 2024
@ShotaAk ShotaAk requested a review from YusukeKato February 7, 2024 06:23
Copy link
Contributor

@YusukeKato YusukeKato left a comment

Choose a reason for hiding this comment

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

シミュレータ環境で動作確認できました。

良いと思います。

industrial_ciが失敗しているのでそれだけ修正をお願いします。

@YusukeKato YusukeKato merged commit 930dc29 into master Feb 7, 2024
2 checks passed
@YusukeKato YusukeKato deleted the update_camera_line_following branch February 7, 2024 07:15
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.

2 participants