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

feat(aip_xx1_description): update for xx1 gen2 #210

Merged
merged 46 commits into from
Jun 20, 2024

Conversation

yukkysaito
Copy link
Collaborator

@yukkysaito yukkysaito commented Feb 3, 2024

update aip_xx1_description for xx1 gen2

Update(06/20 by Yuxuan Liu)

The current arrangement
We will change the sensor_model from aip_xx1 to aip_xx1_gen2 for Gen2 vehicles.

What will be modified in code in detailed

  1. We will change the argument of sensor_model for Gen2 cars.
  2. individual_model/config/vehicle_id/aip_xx1 -> ..../vehicle_id/aip_xx1_gen2 (https://github.com/tier4/autoware_individual_params.jpntaxi/pull/102 ready to test and merge)
  3. Add the aip_xx1_gen2_description/aip_xx1_gen2_launch (this PR. ready to test and merge)

Test Performed
Planning simulation: internal link
Perception evaluator: Internal link1, internal link2

Signed-off-by: Yukihiro Saito <yukky.saito@gmail.com>
@yukkysaito yukkysaito marked this pull request as draft February 3, 2024 13:25
@yukkysaito
Copy link
Collaborator Author

Note that existing aip_xx1 will not work after merging this one.

Signed-off-by: Yuki Takagi <yuki.takagi@tier4.jp>
add OT128 launch

Signed-off-by: Yuki Takagi <yuki.takagi@tier4.jp>
Signed-off-by: Yuki Takagi <yuki.takagi@tier4.jp>
Signed-off-by: Yuki Takagi <yuki.takagi@tier4.jp>
Signed-off-by: Yuki Takagi <yuki.takagi@tier4.jp>
Signed-off-by: Yuki Takagi <yuki.takagi@tier4.jp>
Signed-off-by: yuki.takagi@car_8 <yuki.takagi@tier4.jp>
Signed-off-by: yuki.takagi@car_8 <yuki.takagi@tier4.jp>
@Owen-Liuyuxuan
Copy link
Contributor

@N-Eiki
For now, the conflicts have been fixed and the python-launch has been completed.

We could manage the usage of different sensors with environment variables or arguments in launches.

@Owen-Liuyuxuan
Copy link
Contributor

Owen-Liuyuxuan commented Jun 17, 2024

After discussion of the architecture with @shmpwk, I will:

  • Set an xx1-gen2-parameterized-lidar branch to store the updates on the parameterized lidar/parameterized launch (the refactoring one with high flexibility)
  • In this PR/branch, revert my own contributions (from b20cfcd to 489078d), and make a new folder of new aip_xx1_gen2_launch (this will also revoke the merge commit, but I have tested on a small branch that it preserves all of the existing 34 commits before)

HinsRyu and others added 7 commits June 17, 2024 16:40
Signed-off-by: Yuxuan Liu <uken.ryu.2@tier4.jp>
Signed-off-by: Yuxuan Liu <uken.ryu.2@tier4.jp>
Signed-off-by: Yuxuan Liu <uken.ryu.2@tier4.jp>
Signed-off-by: Yuxuan Liu <uken.ryu.2@tier4.jp>
@Owen-Liuyuxuan Owen-Liuyuxuan marked this pull request as ready for review June 18, 2024 06:39
N-Eiki and others added 2 commits June 18, 2024 16:25
@Owen-Liuyuxuan
Copy link
Contributor

@yukkysaito @tkimura4

The current arrangement
We will change the sensor_model from aip_xx1 to aip_xx1_gen2 for Gen2 vehicles.

What will be modified in code in detailed

  1. We will change the argument of sensor_model for Gen2 cars.
  2. individual_model/config/vehicle_id/aip_xx1 -> ..../vehicle_id/aip_xx1_gen2 (https://github.com/tier4/autoware_individual_params.jpntaxi/pull/102 ready to test and merge)
  3. Add the aip_xx1_gen2_description/aip_xx1_gen2_launch (this PR. ready to test and merge)

N-Eiki added 3 commits June 19, 2024 14:22
Signed-off-by: N-Eiki <eiki.nagata.2@tier4.jp>
…p_xx1_*

Signed-off-by: N-Eiki <eiki.nagata.2@tier4.jp>
@N-Eiki N-Eiki self-requested a review June 20, 2024 05:56
@shmpwk
Copy link
Contributor

shmpwk commented Jun 20, 2024

@Owen-Liuyuxuan @N-Eiki
Could you reflect this description to the top description?
Also, if you tested something, could you write there?

@Owen-Liuyuxuan Owen-Liuyuxuan merged commit d1d423d into tier4/universe Jun 20, 2024
10 of 11 checks passed
@Owen-Liuyuxuan Owen-Liuyuxuan deleted the feature/xx1-gen2 branch June 20, 2024 07:10
@shmpwk
Copy link
Contributor

shmpwk commented Jun 20, 2024

@Owen-Liuyuxuan @N-Eiki
DId you check perception evaluator?

zusizusi added a commit that referenced this pull request Aug 5, 2024
This reverts commit d1d423d.

Signed-off-by: Sho Iwasawa <sho.iwasawa.2@tier4.jp>
zusizusi added a commit that referenced this pull request Aug 5, 2024
Revert "feat(aip_xx1_description): update for xx1 gen2 (#210)"

This reverts commit d1d423d.

Signed-off-by: Sho Iwasawa <sho.iwasawa.2@tier4.jp>
KYabuuchi pushed a commit that referenced this pull request Aug 23, 2024
* update for xx1 gen2

Signed-off-by: Yukihiro Saito <yukky.saito@gmail.com>

* set network configs

Signed-off-by: Yuki Takagi <yuki.takagi@tier4.jp>

* use angle param
add OT128 launch

Signed-off-by: Yuki Takagi <yuki.takagi@tier4.jp>

* set lidar horizontal FOV angle

Signed-off-by: Yuki Takagi <yuki.takagi@tier4.jp>

* update settings

Signed-off-by: Yuki Takagi <yuki.takagi@tier4.jp>

* update

Signed-off-by: Yuki Takagi <yuki.takagi@tier4.jp>

* add OT128 launch

Signed-off-by: Yuki Takagi <yuki.takagi@tier4.jp>

* add ptp param

Signed-off-by: yuki.takagi@car_8 <yuki.takagi@tier4.jp>

* complete to write lidar settings

Signed-off-by: yuki.takagi@car_8 <yuki.takagi@tier4.jp>

* ci(pre-commit): autofix

* chore: generalized radar names

Signed-off-by: Kenzo Lobos-Tsunekawa <kenzo.lobos@tier4.jp>

* feat(aip_xx1_description): update side lidar name

Signed-off-by: yuki.takagi@car_8 <yuki.takagi@tier4.jp>

* feat(aip_xx1_launch): enable imu in adm ecus

Signed-off-by: yuki.takagi@car_8 <yuki.takagi@tier4.jp>

* change concate lidars

* update for xx1 gen2

* rotate top lidar 180

* change imu_corrector param aip_x2 -> aip_xx1

* enable gnss

* fix wrong urdf model for OT128

* fix bug

Signed-off-by: Yukihiro Saito <yukky.saito@gmail.com>

* change max_range

* feat: added the radars to the aip launcher and description

Signed-off-by: j4tfwm6z <j4tfwm6z@tier4.jp>

* add nebula argments

* fix: fixed naming and enabled the radars

Signed-off-by: Kenzo Lobos-Tsunekawa <kenzo.lobos@tier4.jp>

* fix pointcloud preprocess for multi_lidar_ogm

* feat: enable radar launch and concatenation for all lidars

Signed-off-by: j4tfwm6z <j4tfwm6z@tier4.jp>

* fix(hesai_xt32): update ptp_transport_type from l2 to udp for xt32

Signed-off-by: j4tfwm6z <j4tfwm6z@tier4.jp>

* feat: added an option to select the frame of the radar objects

Signed-off-by: Kenzo Lobos-Tsunekawa <kenzo.lobos@tier4.jp>

* feat: add ptp_switch_type to support latest nebula at 2024/03/15 (#222)

Signed-off-by: j4tfwm6z <j4tfwm6z@tier4.jp>
Co-authored-by: j4tfwm6z <j4tfwm6z@tier4.jp>

* feat: add radar feature

Signed-off-by: yoshiri <yoshiyoshidetteiu@gmail.com>

* fix: revert commenting out of radar.launch.xml

Signed-off-by: kminoda <koji.minoda@tier4.jp>

* add new folders

Signed-off-by: Yuxuan Liu <uken.ryu.2@tier4.jp>

* ci(pre-commit): autofix

* add folders for description

Signed-off-by: Yuxuan Liu <uken.ryu.2@tier4.jp>

* ci(pre-commit): autofix

* fix Spell-check

Signed-off-by: Yuxuan Liu <uken.ryu.2@tier4.jp>

* fix package name

Signed-off-by: Yuxuan Liu <uken.ryu.2@tier4.jp>

* fix: recover radar.xacro to make gen1 operational

Signed-off-by: N-Eiki <eiki.nagata.2@tier4.jp>

* ci(pre-commit): autofix

* update aip_xx1 for gen2

Signed-off-by: N-Eiki <eiki.nagata.2@tier4.jp>

* fix: packages name is aip_xx1_gen2_* but in these files written as aip_xx1_*

Signed-off-by: N-Eiki <eiki.nagata.2@tier4.jp>

---------

Signed-off-by: Yukihiro Saito <yukky.saito@gmail.com>
Signed-off-by: Yuki Takagi <yuki.takagi@tier4.jp>
Signed-off-by: yuki.takagi@car_8 <yuki.takagi@tier4.jp>
Signed-off-by: Kenzo Lobos-Tsunekawa <kenzo.lobos@tier4.jp>
Signed-off-by: j4tfwm6z <j4tfwm6z@tier4.jp>
Signed-off-by: yoshiri <yoshiyoshidetteiu@gmail.com>
Signed-off-by: kminoda <koji.minoda@tier4.jp>
Signed-off-by: Yuxuan Liu <uken.ryu.2@tier4.jp>
Signed-off-by: N-Eiki <eiki.nagata.2@tier4.jp>
Co-authored-by: Yuki Takagi <yuki.takagi@tier4.jp>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Kenzo Lobos-Tsunekawa <kenzo.lobos@tier4.jp>
Co-authored-by: j4tfwm6z <j4tfwm6z@tier4.jp>
Co-authored-by: kminoda <44218668+kminoda@users.noreply.github.com>
Co-authored-by: yoshiri <yoshiyoshidetteiu@gmail.com>
Co-authored-by: kminoda <koji.minoda@tier4.jp>
Co-authored-by: Yuxuan Liu <uken.ryu.2@tier4.jp>
Co-authored-by: Yuxuan Liu <619684051@qq.com>
Co-authored-by: N-Eiki <eiki.nagata.2@tier4.jp>
TomohitoAndo pushed a commit that referenced this pull request Aug 30, 2024
Revert "feat(aip_xx1_description): update for xx1 gen2 (#210)"

This reverts commit d1d423d.

Signed-off-by: Sho Iwasawa <sho.iwasawa.2@tier4.jp>
KYabuuchi pushed a commit that referenced this pull request Oct 7, 2024
* update for xx1 gen2

Signed-off-by: Yukihiro Saito <yukky.saito@gmail.com>

* set network configs

Signed-off-by: Yuki Takagi <yuki.takagi@tier4.jp>

* use angle param
add OT128 launch

Signed-off-by: Yuki Takagi <yuki.takagi@tier4.jp>

* set lidar horizontal FOV angle

Signed-off-by: Yuki Takagi <yuki.takagi@tier4.jp>

* update settings

Signed-off-by: Yuki Takagi <yuki.takagi@tier4.jp>

* update

Signed-off-by: Yuki Takagi <yuki.takagi@tier4.jp>

* add OT128 launch

Signed-off-by: Yuki Takagi <yuki.takagi@tier4.jp>

* add ptp param

Signed-off-by: yuki.takagi@car_8 <yuki.takagi@tier4.jp>

* complete to write lidar settings

Signed-off-by: yuki.takagi@car_8 <yuki.takagi@tier4.jp>

* ci(pre-commit): autofix

* chore: generalized radar names

Signed-off-by: Kenzo Lobos-Tsunekawa <kenzo.lobos@tier4.jp>

* feat(aip_xx1_description): update side lidar name

Signed-off-by: yuki.takagi@car_8 <yuki.takagi@tier4.jp>

* feat(aip_xx1_launch): enable imu in adm ecus

Signed-off-by: yuki.takagi@car_8 <yuki.takagi@tier4.jp>

* change concate lidars

* update for xx1 gen2

* rotate top lidar 180

* change imu_corrector param aip_x2 -> aip_xx1

* enable gnss

* fix wrong urdf model for OT128

* fix bug

Signed-off-by: Yukihiro Saito <yukky.saito@gmail.com>

* change max_range

* feat: added the radars to the aip launcher and description

Signed-off-by: j4tfwm6z <j4tfwm6z@tier4.jp>

* add nebula argments

* fix: fixed naming and enabled the radars

Signed-off-by: Kenzo Lobos-Tsunekawa <kenzo.lobos@tier4.jp>

* fix pointcloud preprocess for multi_lidar_ogm

* feat: enable radar launch and concatenation for all lidars

Signed-off-by: j4tfwm6z <j4tfwm6z@tier4.jp>

* fix(hesai_xt32): update ptp_transport_type from l2 to udp for xt32

Signed-off-by: j4tfwm6z <j4tfwm6z@tier4.jp>

* feat: added an option to select the frame of the radar objects

Signed-off-by: Kenzo Lobos-Tsunekawa <kenzo.lobos@tier4.jp>

* feat: add ptp_switch_type to support latest nebula at 2024/03/15 (#222)

Signed-off-by: j4tfwm6z <j4tfwm6z@tier4.jp>
Co-authored-by: j4tfwm6z <j4tfwm6z@tier4.jp>

* feat: add radar feature

Signed-off-by: yoshiri <yoshiyoshidetteiu@gmail.com>

* fix: revert commenting out of radar.launch.xml

Signed-off-by: kminoda <koji.minoda@tier4.jp>

* add new folders

Signed-off-by: Yuxuan Liu <uken.ryu.2@tier4.jp>

* ci(pre-commit): autofix

* add folders for description

Signed-off-by: Yuxuan Liu <uken.ryu.2@tier4.jp>

* ci(pre-commit): autofix

* fix Spell-check

Signed-off-by: Yuxuan Liu <uken.ryu.2@tier4.jp>

* fix package name

Signed-off-by: Yuxuan Liu <uken.ryu.2@tier4.jp>

* fix: recover radar.xacro to make gen1 operational

Signed-off-by: N-Eiki <eiki.nagata.2@tier4.jp>

* ci(pre-commit): autofix

* update aip_xx1 for gen2

Signed-off-by: N-Eiki <eiki.nagata.2@tier4.jp>

* fix: packages name is aip_xx1_gen2_* but in these files written as aip_xx1_*

Signed-off-by: N-Eiki <eiki.nagata.2@tier4.jp>

---------

Signed-off-by: Yukihiro Saito <yukky.saito@gmail.com>
Signed-off-by: Yuki Takagi <yuki.takagi@tier4.jp>
Signed-off-by: yuki.takagi@car_8 <yuki.takagi@tier4.jp>
Signed-off-by: Kenzo Lobos-Tsunekawa <kenzo.lobos@tier4.jp>
Signed-off-by: j4tfwm6z <j4tfwm6z@tier4.jp>
Signed-off-by: yoshiri <yoshiyoshidetteiu@gmail.com>
Signed-off-by: kminoda <koji.minoda@tier4.jp>
Signed-off-by: Yuxuan Liu <uken.ryu.2@tier4.jp>
Signed-off-by: N-Eiki <eiki.nagata.2@tier4.jp>
Co-authored-by: Yuki Takagi <yuki.takagi@tier4.jp>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Kenzo Lobos-Tsunekawa <kenzo.lobos@tier4.jp>
Co-authored-by: j4tfwm6z <j4tfwm6z@tier4.jp>
Co-authored-by: kminoda <44218668+kminoda@users.noreply.github.com>
Co-authored-by: yoshiri <yoshiyoshidetteiu@gmail.com>
Co-authored-by: kminoda <koji.minoda@tier4.jp>
Co-authored-by: Yuxuan Liu <uken.ryu.2@tier4.jp>
Co-authored-by: Yuxuan Liu <619684051@qq.com>
Co-authored-by: N-Eiki <eiki.nagata.2@tier4.jp>
KYabuuchi added a commit that referenced this pull request Oct 8, 2024
* feat(aip_xx1_description): update for xx1 gen2 (#210)

* update for xx1 gen2

Signed-off-by: Yukihiro Saito <yukky.saito@gmail.com>

* set network configs

Signed-off-by: Yuki Takagi <yuki.takagi@tier4.jp>

* use angle param
add OT128 launch

Signed-off-by: Yuki Takagi <yuki.takagi@tier4.jp>

* set lidar horizontal FOV angle

Signed-off-by: Yuki Takagi <yuki.takagi@tier4.jp>

* update settings

Signed-off-by: Yuki Takagi <yuki.takagi@tier4.jp>

* update

Signed-off-by: Yuki Takagi <yuki.takagi@tier4.jp>

* add OT128 launch

Signed-off-by: Yuki Takagi <yuki.takagi@tier4.jp>

* add ptp param

Signed-off-by: yuki.takagi@car_8 <yuki.takagi@tier4.jp>

* complete to write lidar settings

Signed-off-by: yuki.takagi@car_8 <yuki.takagi@tier4.jp>

* ci(pre-commit): autofix

* chore: generalized radar names

Signed-off-by: Kenzo Lobos-Tsunekawa <kenzo.lobos@tier4.jp>

* feat(aip_xx1_description): update side lidar name

Signed-off-by: yuki.takagi@car_8 <yuki.takagi@tier4.jp>

* feat(aip_xx1_launch): enable imu in adm ecus

Signed-off-by: yuki.takagi@car_8 <yuki.takagi@tier4.jp>

* change concate lidars

* update for xx1 gen2

* rotate top lidar 180

* change imu_corrector param aip_x2 -> aip_xx1

* enable gnss

* fix wrong urdf model for OT128

* fix bug

Signed-off-by: Yukihiro Saito <yukky.saito@gmail.com>

* change max_range

* feat: added the radars to the aip launcher and description

Signed-off-by: j4tfwm6z <j4tfwm6z@tier4.jp>

* add nebula argments

* fix: fixed naming and enabled the radars

Signed-off-by: Kenzo Lobos-Tsunekawa <kenzo.lobos@tier4.jp>

* fix pointcloud preprocess for multi_lidar_ogm

* feat: enable radar launch and concatenation for all lidars

Signed-off-by: j4tfwm6z <j4tfwm6z@tier4.jp>

* fix(hesai_xt32): update ptp_transport_type from l2 to udp for xt32

Signed-off-by: j4tfwm6z <j4tfwm6z@tier4.jp>

* feat: added an option to select the frame of the radar objects

Signed-off-by: Kenzo Lobos-Tsunekawa <kenzo.lobos@tier4.jp>

* feat: add ptp_switch_type to support latest nebula at 2024/03/15 (#222)

Signed-off-by: j4tfwm6z <j4tfwm6z@tier4.jp>
Co-authored-by: j4tfwm6z <j4tfwm6z@tier4.jp>

* feat: add radar feature

Signed-off-by: yoshiri <yoshiyoshidetteiu@gmail.com>

* fix: revert commenting out of radar.launch.xml

Signed-off-by: kminoda <koji.minoda@tier4.jp>

* add new folders

Signed-off-by: Yuxuan Liu <uken.ryu.2@tier4.jp>

* ci(pre-commit): autofix

* add folders for description

Signed-off-by: Yuxuan Liu <uken.ryu.2@tier4.jp>

* ci(pre-commit): autofix

* fix Spell-check

Signed-off-by: Yuxuan Liu <uken.ryu.2@tier4.jp>

* fix package name

Signed-off-by: Yuxuan Liu <uken.ryu.2@tier4.jp>

* fix: recover radar.xacro to make gen1 operational

Signed-off-by: N-Eiki <eiki.nagata.2@tier4.jp>

* ci(pre-commit): autofix

* update aip_xx1 for gen2

Signed-off-by: N-Eiki <eiki.nagata.2@tier4.jp>

* fix: packages name is aip_xx1_gen2_* but in these files written as aip_xx1_*

Signed-off-by: N-Eiki <eiki.nagata.2@tier4.jp>

---------

Signed-off-by: Yukihiro Saito <yukky.saito@gmail.com>
Signed-off-by: Yuki Takagi <yuki.takagi@tier4.jp>
Signed-off-by: yuki.takagi@car_8 <yuki.takagi@tier4.jp>
Signed-off-by: Kenzo Lobos-Tsunekawa <kenzo.lobos@tier4.jp>
Signed-off-by: j4tfwm6z <j4tfwm6z@tier4.jp>
Signed-off-by: yoshiri <yoshiyoshidetteiu@gmail.com>
Signed-off-by: kminoda <koji.minoda@tier4.jp>
Signed-off-by: Yuxuan Liu <uken.ryu.2@tier4.jp>
Signed-off-by: N-Eiki <eiki.nagata.2@tier4.jp>
Co-authored-by: Yuki Takagi <yuki.takagi@tier4.jp>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Kenzo Lobos-Tsunekawa <kenzo.lobos@tier4.jp>
Co-authored-by: j4tfwm6z <j4tfwm6z@tier4.jp>
Co-authored-by: kminoda <44218668+kminoda@users.noreply.github.com>
Co-authored-by: yoshiri <yoshiyoshidetteiu@gmail.com>
Co-authored-by: kminoda <koji.minoda@tier4.jp>
Co-authored-by: Yuxuan Liu <uken.ryu.2@tier4.jp>
Co-authored-by: Yuxuan Liu <619684051@qq.com>
Co-authored-by: N-Eiki <eiki.nagata.2@tier4.jp>

* fix path to config file

Signed-off-by: Kento Yabuuchi <kento.yabuuchi.2@tier4.jp>

* add launch_hw_monitor variable to enable/disable hw_monitor

Signed-off-by: Kento Yabuuchi <kento.yabuuchi.2@tier4.jp>

* scan_phase is integer

Signed-off-by: Kento Yabuuchi <kento.yabuuchi.2@tier4.jp>

* launch hw_monitor only when both launch_driver&launch_monitor are true

Signed-off-by: Kento Yabuuchi <kento.yabuuchi.2@tier4.jp>

* fix Substition using context

Signed-off-by: Kento Yabuuchi <kento.yabuuchi.2@tier4.jp>

* use septentrio driver for gnss

* fix OT128 & XT32 PTP profile

* ci(pre-commit): autofix

* cspell ignore some septentrio term

Signed-off-by: Kento Yabuuchi <kento.yabuuchi.2@tier4.jp>

* remove not used files

Signed-off-by: Kento Yabuuchi <kento.yabuuchi.2@tier4.jp>

* reflect revewer's point

Signed-off-by: Kento Yabuuchi <kento.yabuuchi.2@tier4.jp>

---------

Signed-off-by: Yukihiro Saito <yukky.saito@gmail.com>
Signed-off-by: Yuki Takagi <yuki.takagi@tier4.jp>
Signed-off-by: yuki.takagi@car_8 <yuki.takagi@tier4.jp>
Signed-off-by: Kenzo Lobos-Tsunekawa <kenzo.lobos@tier4.jp>
Signed-off-by: j4tfwm6z <j4tfwm6z@tier4.jp>
Signed-off-by: yoshiri <yoshiyoshidetteiu@gmail.com>
Signed-off-by: kminoda <koji.minoda@tier4.jp>
Signed-off-by: Yuxuan Liu <uken.ryu.2@tier4.jp>
Signed-off-by: N-Eiki <eiki.nagata.2@tier4.jp>
Signed-off-by: Kento Yabuuchi <kento.yabuuchi.2@tier4.jp>
Co-authored-by: Yukihiro Saito <yukky.saito@gmail.com>
Co-authored-by: Yuki Takagi <yuki.takagi@tier4.jp>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Kenzo Lobos-Tsunekawa <kenzo.lobos@tier4.jp>
Co-authored-by: j4tfwm6z <j4tfwm6z@tier4.jp>
Co-authored-by: kminoda <44218668+kminoda@users.noreply.github.com>
Co-authored-by: yoshiri <yoshiyoshidetteiu@gmail.com>
Co-authored-by: kminoda <koji.minoda@tier4.jp>
Co-authored-by: Yuxuan Liu <uken.ryu.2@tier4.jp>
Co-authored-by: Yuxuan Liu <619684051@qq.com>
Co-authored-by: N-Eiki <eiki.nagata.2@tier4.jp>
KYabuuchi added a commit that referenced this pull request Oct 8, 2024
* feat(aip_xx1_description): update for xx1 gen2 (#210)

* update for xx1 gen2

Signed-off-by: Yukihiro Saito <yukky.saito@gmail.com>

* set network configs

Signed-off-by: Yuki Takagi <yuki.takagi@tier4.jp>

* use angle param
add OT128 launch

Signed-off-by: Yuki Takagi <yuki.takagi@tier4.jp>

* set lidar horizontal FOV angle

Signed-off-by: Yuki Takagi <yuki.takagi@tier4.jp>

* update settings

Signed-off-by: Yuki Takagi <yuki.takagi@tier4.jp>

* update

Signed-off-by: Yuki Takagi <yuki.takagi@tier4.jp>

* add OT128 launch

Signed-off-by: Yuki Takagi <yuki.takagi@tier4.jp>

* add ptp param

Signed-off-by: yuki.takagi@car_8 <yuki.takagi@tier4.jp>

* complete to write lidar settings

Signed-off-by: yuki.takagi@car_8 <yuki.takagi@tier4.jp>

* ci(pre-commit): autofix

* chore: generalized radar names

Signed-off-by: Kenzo Lobos-Tsunekawa <kenzo.lobos@tier4.jp>

* feat(aip_xx1_description): update side lidar name

Signed-off-by: yuki.takagi@car_8 <yuki.takagi@tier4.jp>

* feat(aip_xx1_launch): enable imu in adm ecus

Signed-off-by: yuki.takagi@car_8 <yuki.takagi@tier4.jp>

* change concate lidars

* update for xx1 gen2

* rotate top lidar 180

* change imu_corrector param aip_x2 -> aip_xx1

* enable gnss

* fix wrong urdf model for OT128

* fix bug

Signed-off-by: Yukihiro Saito <yukky.saito@gmail.com>

* change max_range

* feat: added the radars to the aip launcher and description

Signed-off-by: j4tfwm6z <j4tfwm6z@tier4.jp>

* add nebula argments

* fix: fixed naming and enabled the radars

Signed-off-by: Kenzo Lobos-Tsunekawa <kenzo.lobos@tier4.jp>

* fix pointcloud preprocess for multi_lidar_ogm

* feat: enable radar launch and concatenation for all lidars

Signed-off-by: j4tfwm6z <j4tfwm6z@tier4.jp>

* fix(hesai_xt32): update ptp_transport_type from l2 to udp for xt32

Signed-off-by: j4tfwm6z <j4tfwm6z@tier4.jp>

* feat: added an option to select the frame of the radar objects

Signed-off-by: Kenzo Lobos-Tsunekawa <kenzo.lobos@tier4.jp>

* feat: add ptp_switch_type to support latest nebula at 2024/03/15 (#222)

Signed-off-by: j4tfwm6z <j4tfwm6z@tier4.jp>
Co-authored-by: j4tfwm6z <j4tfwm6z@tier4.jp>

* feat: add radar feature

Signed-off-by: yoshiri <yoshiyoshidetteiu@gmail.com>

* fix: revert commenting out of radar.launch.xml

Signed-off-by: kminoda <koji.minoda@tier4.jp>

* add new folders

Signed-off-by: Yuxuan Liu <uken.ryu.2@tier4.jp>

* ci(pre-commit): autofix

* add folders for description

Signed-off-by: Yuxuan Liu <uken.ryu.2@tier4.jp>

* ci(pre-commit): autofix

* fix Spell-check

Signed-off-by: Yuxuan Liu <uken.ryu.2@tier4.jp>

* fix package name

Signed-off-by: Yuxuan Liu <uken.ryu.2@tier4.jp>

* fix: recover radar.xacro to make gen1 operational

Signed-off-by: N-Eiki <eiki.nagata.2@tier4.jp>

* ci(pre-commit): autofix

* update aip_xx1 for gen2

Signed-off-by: N-Eiki <eiki.nagata.2@tier4.jp>

* fix: packages name is aip_xx1_gen2_* but in these files written as aip_xx1_*

Signed-off-by: N-Eiki <eiki.nagata.2@tier4.jp>

---------

Signed-off-by: Yukihiro Saito <yukky.saito@gmail.com>
Signed-off-by: Yuki Takagi <yuki.takagi@tier4.jp>
Signed-off-by: yuki.takagi@car_8 <yuki.takagi@tier4.jp>
Signed-off-by: Kenzo Lobos-Tsunekawa <kenzo.lobos@tier4.jp>
Signed-off-by: j4tfwm6z <j4tfwm6z@tier4.jp>
Signed-off-by: yoshiri <yoshiyoshidetteiu@gmail.com>
Signed-off-by: kminoda <koji.minoda@tier4.jp>
Signed-off-by: Yuxuan Liu <uken.ryu.2@tier4.jp>
Signed-off-by: N-Eiki <eiki.nagata.2@tier4.jp>
Co-authored-by: Yuki Takagi <yuki.takagi@tier4.jp>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Kenzo Lobos-Tsunekawa <kenzo.lobos@tier4.jp>
Co-authored-by: j4tfwm6z <j4tfwm6z@tier4.jp>
Co-authored-by: kminoda <44218668+kminoda@users.noreply.github.com>
Co-authored-by: yoshiri <yoshiyoshidetteiu@gmail.com>
Co-authored-by: kminoda <koji.minoda@tier4.jp>
Co-authored-by: Yuxuan Liu <uken.ryu.2@tier4.jp>
Co-authored-by: Yuxuan Liu <619684051@qq.com>
Co-authored-by: N-Eiki <eiki.nagata.2@tier4.jp>

* fix path to config file

Signed-off-by: Kento Yabuuchi <kento.yabuuchi.2@tier4.jp>

* add launch_hw_monitor variable to enable/disable hw_monitor

Signed-off-by: Kento Yabuuchi <kento.yabuuchi.2@tier4.jp>

* scan_phase is integer

Signed-off-by: Kento Yabuuchi <kento.yabuuchi.2@tier4.jp>

* launch hw_monitor only when both launch_driver&launch_monitor are true

Signed-off-by: Kento Yabuuchi <kento.yabuuchi.2@tier4.jp>

* fix Substition using context

Signed-off-by: Kento Yabuuchi <kento.yabuuchi.2@tier4.jp>

* use septentrio driver for gnss

* fix OT128 & XT32 PTP profile

* ci(pre-commit): autofix

* cspell ignore some septentrio term

Signed-off-by: Kento Yabuuchi <kento.yabuuchi.2@tier4.jp>

* remove not used files

Signed-off-by: Kento Yabuuchi <kento.yabuuchi.2@tier4.jp>

* fix conflict resolution

Signed-off-by: Kento Yabuuchi <kento.yabuuchi.2@tier4.jp>

* reflect revewer's point

Signed-off-by: Kento Yabuuchi <kento.yabuuchi.2@tier4.jp>

---------

Signed-off-by: Yukihiro Saito <yukky.saito@gmail.com>
Signed-off-by: Yuki Takagi <yuki.takagi@tier4.jp>
Signed-off-by: yuki.takagi@car_8 <yuki.takagi@tier4.jp>
Signed-off-by: Kenzo Lobos-Tsunekawa <kenzo.lobos@tier4.jp>
Signed-off-by: j4tfwm6z <j4tfwm6z@tier4.jp>
Signed-off-by: yoshiri <yoshiyoshidetteiu@gmail.com>
Signed-off-by: kminoda <koji.minoda@tier4.jp>
Signed-off-by: Yuxuan Liu <uken.ryu.2@tier4.jp>
Signed-off-by: N-Eiki <eiki.nagata.2@tier4.jp>
Signed-off-by: Kento Yabuuchi <kento.yabuuchi.2@tier4.jp>
Co-authored-by: Yukihiro Saito <yukky.saito@gmail.com>
Co-authored-by: Yuki Takagi <yuki.takagi@tier4.jp>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Kenzo Lobos-Tsunekawa <kenzo.lobos@tier4.jp>
Co-authored-by: j4tfwm6z <j4tfwm6z@tier4.jp>
Co-authored-by: kminoda <44218668+kminoda@users.noreply.github.com>
Co-authored-by: yoshiri <yoshiyoshidetteiu@gmail.com>
Co-authored-by: kminoda <koji.minoda@tier4.jp>
Co-authored-by: Yuxuan Liu <uken.ryu.2@tier4.jp>
Co-authored-by: Yuxuan Liu <619684051@qq.com>
Co-authored-by: N-Eiki <eiki.nagata.2@tier4.jp>
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.

9 participants