Skip to content
This repository has been archived by the owner on Sep 13, 2023. It is now read-only.

Fix/some bugs #66

Merged
merged 11 commits into from
May 15, 2023
Merged

Fix/some bugs #66

merged 11 commits into from
May 15, 2023

Conversation

koki0624
Copy link
Contributor

@koki0624 koki0624 commented May 8, 2023

概要

  • 細かなバグの修正、加筆など

なぜこのタスクを行うのか

  • 去年のアルゴリズムをシミュレーション上で使えるようにする過程で、いろいろと修正すべき点が見つかったので

やったこと

  • robot_localizationのパラメータのタイポ修正と、一部パラメータの変更
  1. ekf.yaml: センサデータに謎のディレイがあったので、transform_time_offsetを0.0にすることで改善
  2. navsatekf.yaml: robot_localizationのパラメータ調整 #34 で挙げられている問題を修正
  • ナビゲーション用ノードの一部修正
  • カメラの向きの修正
  • line_detectionのノードを一部修正
  • gmappingの余計な出力をコメントアウト

など

その他

  • 基本的には動作に影響するものはないかと思います。
  • @kengohozumi @bvbnatsu
    robot_localizationのパラメータは先日の問題と競合するかもなので確認お願いします。

@koki0624 koki0624 added the bug Something isn't working label May 8, 2023
@Alpaca-zip Alpaca-zip linked an issue May 9, 2023 that may be closed by this pull request
@koki0624 koki0624 requested a review from Alpaca-zip May 9, 2023 07:10
Alpaca-zip
Alpaca-zip previously approved these changes May 9, 2023
Copy link
Contributor

@Alpaca-zip Alpaca-zip left a comment

Choose a reason for hiding this comment

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

@kengohozumi @bvbnatsu
robot_localizationのパラメータは先日の問題と競合するかもなので確認お願いします。

確認して問題なければマージしてください。

Alpaca-zip
Alpaca-zip previously approved these changes May 10, 2023
Alpaca-zip
Alpaca-zip previously approved these changes May 12, 2023
Copy link
Contributor

@Alpaca-zip Alpaca-zip left a comment

Choose a reason for hiding this comment

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

@kengohozumi @bvbnatsu
こちらのPRはバグを直すことを目的としているので、その他バグ等が見つからなければマージしてしまってください。

@bvbnatsu bvbnatsu merged commit f9646c2 into main May 15, 2023
@Alpaca-zip Alpaca-zip deleted the fix/some_bugs branch May 16, 2023 04:22
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

robot_localizationのパラメータ調整
4 participants