-
Notifications
You must be signed in to change notification settings - Fork 17
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
Conversation
Signed-off-by: Yukihiro Saito <[email protected]>
fcb50b6
to
606ec89
Compare
Note that existing aip_xx1 will not work after merging this one. |
Signed-off-by: Yuki Takagi <[email protected]>
add OT128 launch Signed-off-by: Yuki Takagi <[email protected]>
Signed-off-by: Yuki Takagi <[email protected]>
Signed-off-by: Yuki Takagi <[email protected]>
Signed-off-by: Yuki Takagi <[email protected]>
Signed-off-by: Yuki Takagi <[email protected]>
2e0edd2
to
e97400f
Compare
Signed-off-by: yuki.takagi@car_8 <[email protected]>
Signed-off-by: yuki.takagi@car_8 <[email protected]>
121592b
to
dec2668
Compare
Signed-off-by: Kenzo Lobos-Tsunekawa <[email protected]>
… feature/xx1-gen2
Signed-off-by: yuki.takagi@car_8 <[email protected]>
Signed-off-by: yuki.takagi@car_8 <[email protected]>
Signed-off-by: Yukihiro Saito <[email protected]>
@N-Eiki We could manage the usage of different sensors with environment variables or arguments in launches. |
After discussion of the architecture with @shmpwk, I will:
|
489078d
to
0680405
Compare
Signed-off-by: Yuxuan Liu <[email protected]>
Signed-off-by: Yuxuan Liu <[email protected]>
Signed-off-by: Yuxuan Liu <[email protected]>
Signed-off-by: Yuxuan Liu <[email protected]>
Signed-off-by: N-Eiki <[email protected]>
The current arrangement What will be modified in code in detailed
|
Signed-off-by: N-Eiki <[email protected]>
… feature/xx1-gen2
…p_xx1_* Signed-off-by: N-Eiki <[email protected]>
@Owen-Liuyuxuan @N-Eiki |
@Owen-Liuyuxuan @N-Eiki |
This reverts commit d1d423d. Signed-off-by: Sho Iwasawa <[email protected]>
Revert "feat(aip_xx1_description): update for xx1 gen2 (#210)" This reverts commit d1d423d. Signed-off-by: Sho Iwasawa <[email protected]>
* update for xx1 gen2 Signed-off-by: Yukihiro Saito <[email protected]> * set network configs Signed-off-by: Yuki Takagi <[email protected]> * use angle param add OT128 launch Signed-off-by: Yuki Takagi <[email protected]> * set lidar horizontal FOV angle Signed-off-by: Yuki Takagi <[email protected]> * update settings Signed-off-by: Yuki Takagi <[email protected]> * update Signed-off-by: Yuki Takagi <[email protected]> * add OT128 launch Signed-off-by: Yuki Takagi <[email protected]> * add ptp param Signed-off-by: yuki.takagi@car_8 <[email protected]> * complete to write lidar settings Signed-off-by: yuki.takagi@car_8 <[email protected]> * ci(pre-commit): autofix * chore: generalized radar names Signed-off-by: Kenzo Lobos-Tsunekawa <[email protected]> * feat(aip_xx1_description): update side lidar name Signed-off-by: yuki.takagi@car_8 <[email protected]> * feat(aip_xx1_launch): enable imu in adm ecus Signed-off-by: yuki.takagi@car_8 <[email protected]> * 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 <[email protected]> * change max_range * feat: added the radars to the aip launcher and description Signed-off-by: j4tfwm6z <[email protected]> * add nebula argments * fix: fixed naming and enabled the radars Signed-off-by: Kenzo Lobos-Tsunekawa <[email protected]> * fix pointcloud preprocess for multi_lidar_ogm * feat: enable radar launch and concatenation for all lidars Signed-off-by: j4tfwm6z <[email protected]> * fix(hesai_xt32): update ptp_transport_type from l2 to udp for xt32 Signed-off-by: j4tfwm6z <[email protected]> * feat: added an option to select the frame of the radar objects Signed-off-by: Kenzo Lobos-Tsunekawa <[email protected]> * feat: add ptp_switch_type to support latest nebula at 2024/03/15 (#222) Signed-off-by: j4tfwm6z <[email protected]> Co-authored-by: j4tfwm6z <[email protected]> * feat: add radar feature Signed-off-by: yoshiri <[email protected]> * fix: revert commenting out of radar.launch.xml Signed-off-by: kminoda <[email protected]> * add new folders Signed-off-by: Yuxuan Liu <[email protected]> * ci(pre-commit): autofix * add folders for description Signed-off-by: Yuxuan Liu <[email protected]> * ci(pre-commit): autofix * fix Spell-check Signed-off-by: Yuxuan Liu <[email protected]> * fix package name Signed-off-by: Yuxuan Liu <[email protected]> * fix: recover radar.xacro to make gen1 operational Signed-off-by: N-Eiki <[email protected]> * ci(pre-commit): autofix * update aip_xx1 for gen2 Signed-off-by: N-Eiki <[email protected]> * fix: packages name is aip_xx1_gen2_* but in these files written as aip_xx1_* Signed-off-by: N-Eiki <[email protected]> --------- Signed-off-by: Yukihiro Saito <[email protected]> Signed-off-by: Yuki Takagi <[email protected]> Signed-off-by: yuki.takagi@car_8 <[email protected]> Signed-off-by: Kenzo Lobos-Tsunekawa <[email protected]> Signed-off-by: j4tfwm6z <[email protected]> Signed-off-by: yoshiri <[email protected]> Signed-off-by: kminoda <[email protected]> Signed-off-by: Yuxuan Liu <[email protected]> Signed-off-by: N-Eiki <[email protected]> Co-authored-by: Yuki Takagi <[email protected]> Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com> Co-authored-by: Kenzo Lobos-Tsunekawa <[email protected]> Co-authored-by: j4tfwm6z <[email protected]> Co-authored-by: kminoda <[email protected]> Co-authored-by: yoshiri <[email protected]> Co-authored-by: kminoda <[email protected]> Co-authored-by: Yuxuan Liu <[email protected]> Co-authored-by: Yuxuan Liu <[email protected]> Co-authored-by: N-Eiki <[email protected]>
Revert "feat(aip_xx1_description): update for xx1 gen2 (#210)" This reverts commit d1d423d. Signed-off-by: Sho Iwasawa <[email protected]>
* update for xx1 gen2 Signed-off-by: Yukihiro Saito <[email protected]> * set network configs Signed-off-by: Yuki Takagi <[email protected]> * use angle param add OT128 launch Signed-off-by: Yuki Takagi <[email protected]> * set lidar horizontal FOV angle Signed-off-by: Yuki Takagi <[email protected]> * update settings Signed-off-by: Yuki Takagi <[email protected]> * update Signed-off-by: Yuki Takagi <[email protected]> * add OT128 launch Signed-off-by: Yuki Takagi <[email protected]> * add ptp param Signed-off-by: yuki.takagi@car_8 <[email protected]> * complete to write lidar settings Signed-off-by: yuki.takagi@car_8 <[email protected]> * ci(pre-commit): autofix * chore: generalized radar names Signed-off-by: Kenzo Lobos-Tsunekawa <[email protected]> * feat(aip_xx1_description): update side lidar name Signed-off-by: yuki.takagi@car_8 <[email protected]> * feat(aip_xx1_launch): enable imu in adm ecus Signed-off-by: yuki.takagi@car_8 <[email protected]> * 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 <[email protected]> * change max_range * feat: added the radars to the aip launcher and description Signed-off-by: j4tfwm6z <[email protected]> * add nebula argments * fix: fixed naming and enabled the radars Signed-off-by: Kenzo Lobos-Tsunekawa <[email protected]> * fix pointcloud preprocess for multi_lidar_ogm * feat: enable radar launch and concatenation for all lidars Signed-off-by: j4tfwm6z <[email protected]> * fix(hesai_xt32): update ptp_transport_type from l2 to udp for xt32 Signed-off-by: j4tfwm6z <[email protected]> * feat: added an option to select the frame of the radar objects Signed-off-by: Kenzo Lobos-Tsunekawa <[email protected]> * feat: add ptp_switch_type to support latest nebula at 2024/03/15 (#222) Signed-off-by: j4tfwm6z <[email protected]> Co-authored-by: j4tfwm6z <[email protected]> * feat: add radar feature Signed-off-by: yoshiri <[email protected]> * fix: revert commenting out of radar.launch.xml Signed-off-by: kminoda <[email protected]> * add new folders Signed-off-by: Yuxuan Liu <[email protected]> * ci(pre-commit): autofix * add folders for description Signed-off-by: Yuxuan Liu <[email protected]> * ci(pre-commit): autofix * fix Spell-check Signed-off-by: Yuxuan Liu <[email protected]> * fix package name Signed-off-by: Yuxuan Liu <[email protected]> * fix: recover radar.xacro to make gen1 operational Signed-off-by: N-Eiki <[email protected]> * ci(pre-commit): autofix * update aip_xx1 for gen2 Signed-off-by: N-Eiki <[email protected]> * fix: packages name is aip_xx1_gen2_* but in these files written as aip_xx1_* Signed-off-by: N-Eiki <[email protected]> --------- Signed-off-by: Yukihiro Saito <[email protected]> Signed-off-by: Yuki Takagi <[email protected]> Signed-off-by: yuki.takagi@car_8 <[email protected]> Signed-off-by: Kenzo Lobos-Tsunekawa <[email protected]> Signed-off-by: j4tfwm6z <[email protected]> Signed-off-by: yoshiri <[email protected]> Signed-off-by: kminoda <[email protected]> Signed-off-by: Yuxuan Liu <[email protected]> Signed-off-by: N-Eiki <[email protected]> Co-authored-by: Yuki Takagi <[email protected]> Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com> Co-authored-by: Kenzo Lobos-Tsunekawa <[email protected]> Co-authored-by: j4tfwm6z <[email protected]> Co-authored-by: kminoda <[email protected]> Co-authored-by: yoshiri <[email protected]> Co-authored-by: kminoda <[email protected]> Co-authored-by: Yuxuan Liu <[email protected]> Co-authored-by: Yuxuan Liu <[email protected]> Co-authored-by: N-Eiki <[email protected]>
* feat(aip_xx1_description): update for xx1 gen2 (#210) * update for xx1 gen2 Signed-off-by: Yukihiro Saito <[email protected]> * set network configs Signed-off-by: Yuki Takagi <[email protected]> * use angle param add OT128 launch Signed-off-by: Yuki Takagi <[email protected]> * set lidar horizontal FOV angle Signed-off-by: Yuki Takagi <[email protected]> * update settings Signed-off-by: Yuki Takagi <[email protected]> * update Signed-off-by: Yuki Takagi <[email protected]> * add OT128 launch Signed-off-by: Yuki Takagi <[email protected]> * add ptp param Signed-off-by: yuki.takagi@car_8 <[email protected]> * complete to write lidar settings Signed-off-by: yuki.takagi@car_8 <[email protected]> * ci(pre-commit): autofix * chore: generalized radar names Signed-off-by: Kenzo Lobos-Tsunekawa <[email protected]> * feat(aip_xx1_description): update side lidar name Signed-off-by: yuki.takagi@car_8 <[email protected]> * feat(aip_xx1_launch): enable imu in adm ecus Signed-off-by: yuki.takagi@car_8 <[email protected]> * 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 <[email protected]> * change max_range * feat: added the radars to the aip launcher and description Signed-off-by: j4tfwm6z <[email protected]> * add nebula argments * fix: fixed naming and enabled the radars Signed-off-by: Kenzo Lobos-Tsunekawa <[email protected]> * fix pointcloud preprocess for multi_lidar_ogm * feat: enable radar launch and concatenation for all lidars Signed-off-by: j4tfwm6z <[email protected]> * fix(hesai_xt32): update ptp_transport_type from l2 to udp for xt32 Signed-off-by: j4tfwm6z <[email protected]> * feat: added an option to select the frame of the radar objects Signed-off-by: Kenzo Lobos-Tsunekawa <[email protected]> * feat: add ptp_switch_type to support latest nebula at 2024/03/15 (#222) Signed-off-by: j4tfwm6z <[email protected]> Co-authored-by: j4tfwm6z <[email protected]> * feat: add radar feature Signed-off-by: yoshiri <[email protected]> * fix: revert commenting out of radar.launch.xml Signed-off-by: kminoda <[email protected]> * add new folders Signed-off-by: Yuxuan Liu <[email protected]> * ci(pre-commit): autofix * add folders for description Signed-off-by: Yuxuan Liu <[email protected]> * ci(pre-commit): autofix * fix Spell-check Signed-off-by: Yuxuan Liu <[email protected]> * fix package name Signed-off-by: Yuxuan Liu <[email protected]> * fix: recover radar.xacro to make gen1 operational Signed-off-by: N-Eiki <[email protected]> * ci(pre-commit): autofix * update aip_xx1 for gen2 Signed-off-by: N-Eiki <[email protected]> * fix: packages name is aip_xx1_gen2_* but in these files written as aip_xx1_* Signed-off-by: N-Eiki <[email protected]> --------- Signed-off-by: Yukihiro Saito <[email protected]> Signed-off-by: Yuki Takagi <[email protected]> Signed-off-by: yuki.takagi@car_8 <[email protected]> Signed-off-by: Kenzo Lobos-Tsunekawa <[email protected]> Signed-off-by: j4tfwm6z <[email protected]> Signed-off-by: yoshiri <[email protected]> Signed-off-by: kminoda <[email protected]> Signed-off-by: Yuxuan Liu <[email protected]> Signed-off-by: N-Eiki <[email protected]> Co-authored-by: Yuki Takagi <[email protected]> Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com> Co-authored-by: Kenzo Lobos-Tsunekawa <[email protected]> Co-authored-by: j4tfwm6z <[email protected]> Co-authored-by: kminoda <[email protected]> Co-authored-by: yoshiri <[email protected]> Co-authored-by: kminoda <[email protected]> Co-authored-by: Yuxuan Liu <[email protected]> Co-authored-by: Yuxuan Liu <[email protected]> Co-authored-by: N-Eiki <[email protected]> * fix path to config file Signed-off-by: Kento Yabuuchi <[email protected]> * add launch_hw_monitor variable to enable/disable hw_monitor Signed-off-by: Kento Yabuuchi <[email protected]> * scan_phase is integer Signed-off-by: Kento Yabuuchi <[email protected]> * launch hw_monitor only when both launch_driver&launch_monitor are true Signed-off-by: Kento Yabuuchi <[email protected]> * fix Substition using context Signed-off-by: Kento Yabuuchi <[email protected]> * use septentrio driver for gnss * fix OT128 & XT32 PTP profile * ci(pre-commit): autofix * cspell ignore some septentrio term Signed-off-by: Kento Yabuuchi <[email protected]> * remove not used files Signed-off-by: Kento Yabuuchi <[email protected]> * reflect revewer's point Signed-off-by: Kento Yabuuchi <[email protected]> --------- Signed-off-by: Yukihiro Saito <[email protected]> Signed-off-by: Yuki Takagi <[email protected]> Signed-off-by: yuki.takagi@car_8 <[email protected]> Signed-off-by: Kenzo Lobos-Tsunekawa <[email protected]> Signed-off-by: j4tfwm6z <[email protected]> Signed-off-by: yoshiri <[email protected]> Signed-off-by: kminoda <[email protected]> Signed-off-by: Yuxuan Liu <[email protected]> Signed-off-by: N-Eiki <[email protected]> Signed-off-by: Kento Yabuuchi <[email protected]> Co-authored-by: Yukihiro Saito <[email protected]> Co-authored-by: Yuki Takagi <[email protected]> Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com> Co-authored-by: Kenzo Lobos-Tsunekawa <[email protected]> Co-authored-by: j4tfwm6z <[email protected]> Co-authored-by: kminoda <[email protected]> Co-authored-by: yoshiri <[email protected]> Co-authored-by: kminoda <[email protected]> Co-authored-by: Yuxuan Liu <[email protected]> Co-authored-by: Yuxuan Liu <[email protected]> Co-authored-by: N-Eiki <[email protected]>
* feat(aip_xx1_description): update for xx1 gen2 (#210) * update for xx1 gen2 Signed-off-by: Yukihiro Saito <[email protected]> * set network configs Signed-off-by: Yuki Takagi <[email protected]> * use angle param add OT128 launch Signed-off-by: Yuki Takagi <[email protected]> * set lidar horizontal FOV angle Signed-off-by: Yuki Takagi <[email protected]> * update settings Signed-off-by: Yuki Takagi <[email protected]> * update Signed-off-by: Yuki Takagi <[email protected]> * add OT128 launch Signed-off-by: Yuki Takagi <[email protected]> * add ptp param Signed-off-by: yuki.takagi@car_8 <[email protected]> * complete to write lidar settings Signed-off-by: yuki.takagi@car_8 <[email protected]> * ci(pre-commit): autofix * chore: generalized radar names Signed-off-by: Kenzo Lobos-Tsunekawa <[email protected]> * feat(aip_xx1_description): update side lidar name Signed-off-by: yuki.takagi@car_8 <[email protected]> * feat(aip_xx1_launch): enable imu in adm ecus Signed-off-by: yuki.takagi@car_8 <[email protected]> * 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 <[email protected]> * change max_range * feat: added the radars to the aip launcher and description Signed-off-by: j4tfwm6z <[email protected]> * add nebula argments * fix: fixed naming and enabled the radars Signed-off-by: Kenzo Lobos-Tsunekawa <[email protected]> * fix pointcloud preprocess for multi_lidar_ogm * feat: enable radar launch and concatenation for all lidars Signed-off-by: j4tfwm6z <[email protected]> * fix(hesai_xt32): update ptp_transport_type from l2 to udp for xt32 Signed-off-by: j4tfwm6z <[email protected]> * feat: added an option to select the frame of the radar objects Signed-off-by: Kenzo Lobos-Tsunekawa <[email protected]> * feat: add ptp_switch_type to support latest nebula at 2024/03/15 (#222) Signed-off-by: j4tfwm6z <[email protected]> Co-authored-by: j4tfwm6z <[email protected]> * feat: add radar feature Signed-off-by: yoshiri <[email protected]> * fix: revert commenting out of radar.launch.xml Signed-off-by: kminoda <[email protected]> * add new folders Signed-off-by: Yuxuan Liu <[email protected]> * ci(pre-commit): autofix * add folders for description Signed-off-by: Yuxuan Liu <[email protected]> * ci(pre-commit): autofix * fix Spell-check Signed-off-by: Yuxuan Liu <[email protected]> * fix package name Signed-off-by: Yuxuan Liu <[email protected]> * fix: recover radar.xacro to make gen1 operational Signed-off-by: N-Eiki <[email protected]> * ci(pre-commit): autofix * update aip_xx1 for gen2 Signed-off-by: N-Eiki <[email protected]> * fix: packages name is aip_xx1_gen2_* but in these files written as aip_xx1_* Signed-off-by: N-Eiki <[email protected]> --------- Signed-off-by: Yukihiro Saito <[email protected]> Signed-off-by: Yuki Takagi <[email protected]> Signed-off-by: yuki.takagi@car_8 <[email protected]> Signed-off-by: Kenzo Lobos-Tsunekawa <[email protected]> Signed-off-by: j4tfwm6z <[email protected]> Signed-off-by: yoshiri <[email protected]> Signed-off-by: kminoda <[email protected]> Signed-off-by: Yuxuan Liu <[email protected]> Signed-off-by: N-Eiki <[email protected]> Co-authored-by: Yuki Takagi <[email protected]> Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com> Co-authored-by: Kenzo Lobos-Tsunekawa <[email protected]> Co-authored-by: j4tfwm6z <[email protected]> Co-authored-by: kminoda <[email protected]> Co-authored-by: yoshiri <[email protected]> Co-authored-by: kminoda <[email protected]> Co-authored-by: Yuxuan Liu <[email protected]> Co-authored-by: Yuxuan Liu <[email protected]> Co-authored-by: N-Eiki <[email protected]> * fix path to config file Signed-off-by: Kento Yabuuchi <[email protected]> * add launch_hw_monitor variable to enable/disable hw_monitor Signed-off-by: Kento Yabuuchi <[email protected]> * scan_phase is integer Signed-off-by: Kento Yabuuchi <[email protected]> * launch hw_monitor only when both launch_driver&launch_monitor are true Signed-off-by: Kento Yabuuchi <[email protected]> * fix Substition using context Signed-off-by: Kento Yabuuchi <[email protected]> * use septentrio driver for gnss * fix OT128 & XT32 PTP profile * ci(pre-commit): autofix * cspell ignore some septentrio term Signed-off-by: Kento Yabuuchi <[email protected]> * remove not used files Signed-off-by: Kento Yabuuchi <[email protected]> * fix conflict resolution Signed-off-by: Kento Yabuuchi <[email protected]> * reflect revewer's point Signed-off-by: Kento Yabuuchi <[email protected]> --------- Signed-off-by: Yukihiro Saito <[email protected]> Signed-off-by: Yuki Takagi <[email protected]> Signed-off-by: yuki.takagi@car_8 <[email protected]> Signed-off-by: Kenzo Lobos-Tsunekawa <[email protected]> Signed-off-by: j4tfwm6z <[email protected]> Signed-off-by: yoshiri <[email protected]> Signed-off-by: kminoda <[email protected]> Signed-off-by: Yuxuan Liu <[email protected]> Signed-off-by: N-Eiki <[email protected]> Signed-off-by: Kento Yabuuchi <[email protected]> Co-authored-by: Yukihiro Saito <[email protected]> Co-authored-by: Yuki Takagi <[email protected]> Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com> Co-authored-by: Kenzo Lobos-Tsunekawa <[email protected]> Co-authored-by: j4tfwm6z <[email protected]> Co-authored-by: kminoda <[email protected]> Co-authored-by: yoshiri <[email protected]> Co-authored-by: kminoda <[email protected]> Co-authored-by: Yuxuan Liu <[email protected]> Co-authored-by: Yuxuan Liu <[email protected]> Co-authored-by: N-Eiki <[email protected]>
update aip_xx1_description for xx1 gen2
Update(06/20 by Yuxuan Liu)
The current arrangement
We will change the
sensor_model
fromaip_xx1
toaip_xx1_gen2
for Gen2 vehicles.What will be modified in code in detailed
sensor_model
for Gen2 cars.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)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