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

テスト実行時はダミーのパラメータファイルを読み込めるようにdescription_loaderを修正 #73

Merged
merged 3 commits into from
Sep 8, 2023

Conversation

YusukeKato
Copy link
Collaborator

@YusukeKato YusukeKato commented Sep 7, 2023

What does this implement/fix?

パラメータファイルを引数で受け取るようにdescription_loaderを修正します。
理由はROS build firm上(またはcolcon test)におけるエラーを回避するためです。

Does this close any currently open issues?

#71

How has this been tested?

setupファイルを読み込まなくても、colcon testが通ることを確認しました。

Any other comments?

Checklists

@YusukeKato YusukeKato added the Type: Bug Bug or Bug fixes label Sep 7, 2023
@YusukeKato YusukeKato requested a review from ShotaAk September 7, 2023 07:09
@YusukeKato YusukeKato self-assigned this Sep 7, 2023
Copy link

@ShotaAk ShotaAk left a comment

Choose a reason for hiding this comment

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

LGTMです。
gazeboでの動作も確認できました。

ついでに、CIからBEFORE_RUN_TARGET_TEST_EMBED変数を取り除いてもらえませんか?
事前setup読み込みが不要になったので、変数も不要になります。

- { ROS_DISTRO: humble, ROS_REPO: ros, BEFORE_RUN_TARGET_TEST_EMBED: "ici_with_unset_variables source /root/target_ws/install/setup.bash" }

@ShotaAk ShotaAk merged commit 909db12 into master Sep 8, 2023
2 checks passed
@ShotaAk ShotaAk deleted the fix-robot-description-loader branch September 8, 2023 05:55
ShotaAk pushed a commit that referenced this pull request Sep 8, 2023
* READMEにカメラサンプルデモのGIFを追加 (#70)

* カメラサンプルのデモGIFをREADMEに添付

* 水平線の追加

* テスト実行時はダミーのパラメータファイルを読み込めるようにdescription_loaderを修正 (#73)

* テスト時はダミーのパラメータファイルを読み込めるようにdescription_loaderを修正

* CIの設定ファイルから読み込む必要のないsetup.bashの変数を削除

* 余計な文字を削除

* 2.0.1リリースのためにCHANGELOGとpackage.xmlを更新 (#74)

* リリースのためにCHANGELOGを更新

* 2.0.1
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Bug Bug or Bug fixes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants