Skip to content

Conversation

@interimadd
Copy link

@interimadd interimadd commented Jan 13, 2026

Description

Apply #14 to t4-main-caret branch

Test

Pilot.Auto can be built successfully.

takahisaishikawa@dpc2409508:~/pilot-auto-caret$ yes Y | ~/caret_ecu_system_setup/general/build_autoware.sh

And I confirmed that Pilot.Auto can drive a car in AWSIM with following command.

cd ~
git clone git@github.com:tier4/auto-evaluator.git
cd ~/auto-evaluator/scripts/performance_evaluation/nishishinjuku
git pull
./download_nishishinjuku_awsim_and_map.sh
source ~/caret_ecu_system_setup/general/setenv_caret.bash
./exec_awsim_nishishinjuku.sh ~/pilot-auto-caret ./goals_nishishinjuku_short.yaml ./awsim_config_nishishinjuku.json
image

Related Link

https://tier4.atlassian.net/browse/T4DEV-45284

Takahisa.Ishikawa and others added 2 commits January 13, 2026 20:08
…-qos

feat(rclcpp_components): change load_node service QoS to prevent failure on loading composable node.
@interimadd interimadd marked this pull request as ready for review January 13, 2026 11:30
Copy link

@iwatake2222 iwatake2222 left a comment

Choose a reason for hiding this comment

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

@interimadd
LGTM!
However, please hold off on merging until the Pilot.Auto v0.61 release evaluation is completed. The reason is that v0.61 uses a forked version of launch_ros, which requires the pre-change version of rclcpp. In addition, the rclcpp used for CARET evaluation is specified by branch name, so merging this change would cause an inconsistency.

(To avoid an accidental merge, I’m changing the review status to “Request changes.”)

Note:
After merging tier4/caret#225, we can merge this PR. However, I'd like to wait the Pilot.Auto v0.61 release completion for the safe.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants