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

behavior_path_planner can't plan a path with the first given goal command #636

Closed
3 tasks done
mehmetdogru opened this issue Apr 5, 2022 · 2 comments
Closed
3 tasks done
Assignees
Labels
priority:high High urgency and importance.

Comments

@mehmetdogru
Copy link
Contributor

mehmetdogru commented Apr 5, 2022

Checklist

  • I've read the contribution guidelines.
  • I've searched other issues and no duplicate issues were found.
  • I'm convinced that this is not my fault but a bug.

Description

Most of the time, behavior_path_planner sometimes can't plan a path with the first given goal command from rviz2. And the node gives this error:

[component_container_mt-13] [ERROR] [1649165181.816988526] [planning.scenario_planning.lane_driving.behavior_planning.behavior_path_planner]: behavior path output is empty! Stop publish.

And when the second time a goal command is given (even though exactly the same goal pose as the first one) behavior_path_planner is able to plan the path.

Expected behavior

behavior_path_planner should be able to calculate a path with the first goal pose given.

Actual behavior

Most of the time, behavior_path_planner sometimes can't plan a path with the first given goal command from rviz2

Steps to reproduce

  1. Launch planning_simulator.launch.xml with any lanelet2 map and any .pcd file as it is described here.
  2. Then set an initial pose for the ego vehicle.
  3. Set a goal pose for the ego vehicle.
  4. Repeat the 3rd step and observe if the path is created by behavior_path_planner.

Versions

  • Autoware.universe commit hash: 70bc967

Possible causes

No response

Additional context

No response

@mehmetdogru mehmetdogru added the priority:high High urgency and importance. label Apr 5, 2022
@mehmetdogru mehmetdogru added this to the Bus ODD Mar - Apr Milestone milestone Apr 5, 2022
@maxime-clem
Copy link
Contributor

I am able to reproduce the issue but it is inconsistent and setting a goal sometimes works the first time.

Using git bisect I have identified the first bad commit as 982263a.

@maxime-clem
Copy link
Contributor

maxime-clem commented Apr 8, 2022

This issue was fixed by c341d4c but I observed a new issue where behavior_planning crashes after setting a goal. This crash happened only once but prevented the trajectory generation, even after setting the goal another time.

[component_container_mt-13] Inconsistency detected by ld.so: ../elf/dl-tls.c: 481: _dl_allocate_tls_init: Assertion `listp->slotinfo[cnt].gen <= GL(dl_tls_generation)' failed!
[ERROR] [component_container_mt-13]: process has died [pid 9361, exit code 127, cmd '/opt/ros/galactic/lib/rclcpp_components/component_container_mt --ros-args -r __node:=behavior_planning_container -r __ns:=/planning/scenario_planning/lane_driving/behavior_planning --params-file /tmp/launch_params_bzfw7nao'].

EDIT: I cannot reproduce the crash and it might have been caused by an unclean build.

13304490790 pushed a commit to 13304490790/autoware.universe that referenced this issue Apr 12, 2022
* release v0.4.0

* Avoid setting CMAKE_BUILD_TYPE=Release in each CMakeLists.txt (autowarefoundation#720)

* remove set CMAKE_BUILD_TYPE Release in each CMakeLists.txt

* remove set CMAKE_BUILD_TYPE Release in ndt_pcl_modified

* set compile options for debug in ndt_omp

* Fix indent

* add warning if -DCMAKE_BUILD_TYPE=Release is not set in ndt_omp

Co-authored-by: Kenji Miyake <31987104+kenji-miyake@users.noreply.github.com>

* remove ROS1 packages temporarily

Signed-off-by: mitsudome-r <ryohsuke.mitsudome@tier4.jp>

* Port autoware_localization_srvs

* Add DEPENDENCIES

* Revert "remove ROS1 packages temporarily"

This reverts commit ee01a755ac5b22c61f4274ae2ee57b0a18150a78.

Signed-off-by: mitsudome-r <ryohsuke.mitsudome@tier4.jp>

* add COLCON_IGNORE to ros1 packages

Signed-off-by: mitsudome-r <ryohsuke.mitsudome@tier4.jp>

* Rename launch files to launch.xml (autowarefoundation#28)

* Port pose_initializer to ROS2 (autowarefoundation#11)

* CMakeLists.txt & package.xml

* Compiles

* Works

* Use callback instead

* Reviewer feedback

* Add sequence number check

* Review feedback

* Fix lint issues (autowarefoundation#128)

* Rename h files to hpp (autowarefoundation#142)

* Change includes

* Rename files

* Adjustments to make things compile

* Other packages

* Use quotes for includes where appropriate (autowarefoundation#144)

* Use quotes for includes where appropriate

* Fix lint tests

* Make tests pass hopefully

* fixing trasient_local in ROS2 packages (autowarefoundation#160)

* [ekf_localizer] [pose_initializer] fix topic message type (autowarefoundation#176)

Co-authored-by: Autoware <autoware@tier4.jp>
Co-authored-by: Takamasa Horibe <horibe.takamasa@gmail.com>

* ading linters to pose_initializer (autowarefoundation#186)

* adding linters to autoware_localization_srvs (autowarefoundation#185)

* Port initial-pose-button-panel (autowarefoundation#125)

* Port

Signed-off-by: Servando German Serrano <servando.german.serrano@linaro.org>

* Initialize button when request received

Signed-off-by: Servando German Serrano <servando.german.serrano@linaro.org>

* Amend buildtool

Signed-off-by: Servando German Serrano <servando.german.serrano@linaro.org>

* Fix license

Signed-off-by: Servando German Serrano <servando.german.serrano@linaro.org>

* apply env_var to  use_sim_time (autowarefoundation#222)

* retry gnss initialize (autowarefoundation#1043) (autowarefoundation#263)

Co-authored-by: YamatoAndo <yamato.ando@gmail.com>

* Ros2 v0.8.0 initial pose buttion panel (autowarefoundation#312)

* restore initial pose button panel files for v0.8.0 update

* fix typos in localization (autowarefoundation#890)

* Fix memory leak and remove unused properties (autowarefoundation#964)

* Revert "restore initial pose button panel files for v0.8.0 update"

This reverts commit 01281a2633636f80f241a788f9d35c517a2977e1.

* delete unused header

* add ament lint test

* apply ament_lint

* fix bug

Co-authored-by: Kazuki Miyahara <kmiya@outlook.com>

* modify pose_initialize for foa  (autowarefoundation#382)

* pose_initialize for foa (autowarefoundation#1171)

* pose_initialize for foa

Signed-off-by: Yamato ANDO <yamato.ando@gmail.com>

* rename topic

Signed-off-by: Yamato ANDO <yamato.ando@gmail.com>

* fix topic name

Signed-off-by: mitsudome-r <ryohsuke.mitsudome@tier4.jp>

* Fix missing gnss subscriber (autowarefoundation#417)

Signed-off-by: wep21 <border_goldenmarket@yahoo.co.jp>

* Ros2 fix ndt align srv (autowarefoundation#442)

* Add success field to ndt align srv response

Signed-off-by: Takagi, Isamu <isamu.takagi@tier4.jp>

* Apply change of ndt align srv

Signed-off-by: Takagi, Isamu <isamu.takagi@tier4.jp>

* Apply change of ndt align srv

Signed-off-by: Takagi, Isamu <isamu.takagi@tier4.jp>

* Fix lint

Signed-off-by: Takagi, Isamu <isamu.takagi@tier4.jp>

* Unify Apache-2.0 license name (autowarefoundation#1242)

* Remove use_sim_time for set_parameter (autowarefoundation#1260)

Signed-off-by: wep21 <border_goldenmarket@yahoo.co.jp>

* Fix dependency type of rosidl_default_generators (autowarefoundation#1801)

* Fix dependency type of rosidl_default_generators

Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp>

* Remove unnecessary depends

Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp>

* Use ament_cmake_auto

Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp>

* Fix -Wunused-parameter (autowarefoundation#1836)

* Fix -Wunused-parameter

Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp>

* Fix mistake

Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp>

* fix spell

* Fix lint issues

Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp>

* Ignore flake8 warnings

Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp>

Co-authored-by: Hiroki OTA <hiroki.ota@tier4.jp>

* Add autoware api (autowarefoundation#1979)

* add sort-package-xml hook in pre-commit (autowarefoundation#1881)

* add sort xml hook in pre-commit

* change retval to exit_status

* rename

* add prettier plugin-xml

* use early return

* add license note

* add tier4 license

* restore prettier

* change license order

* move local hooks to public repo

* move prettier-xml to pre-commit-hooks-ros

* update version for bug-fix

* apply pre-commit

* Change formatter to clang-format and black (autowarefoundation#2332)

* Revert "Temporarily comment out pre-commit hooks"

This reverts commit 748e9cdb145ce12f8b520bcbd97f5ff899fc28a3.

* Replace ament_lint_common with autoware_lint_common

Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp>

* Remove ament_cmake_uncrustify and ament_clang_format

Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp>

* Apply Black

Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp>

* Apply clang-format

Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp>

* Fix build errors

Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp>

* Fix for cpplint

* Fix include double quotes to angle brackets

Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp>

* Apply clang-format

Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp>

* Fix build errors

Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp>

* Add COLCON_IGNORE (autowarefoundation#500)

Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp>

* Remove COLCON_IGNORE under localization (autowarefoundation#553)

* Remove 'pose_initializer/COLCON_IGNORE' (autowarefoundation#556)

* add README.md (autowarefoundation#623)

* add README.md

* fix typo

* minor fix

* Update localization/util/autoware_localization_rviz_plugin/initial_pose_button_panel/README.md

Co-authored-by: Kazuki Miyahara <kmiya@outlook.com>

* Update localization/util/autoware_localization_rviz_plugin/initial_pose_button_panel/README.md

Co-authored-by: Kazuki Miyahara <kmiya@outlook.com>

* Update localization/util/autoware_localization_rviz_plugin/initial_pose_button_panel/README.md

Co-authored-by: Takayuki Murooka <takayuki.murooka@tier4.jp>
Co-authored-by: Kazuki Miyahara <kmiya@outlook.com>
Co-authored-by: tkimura4 <tomoya.kimura@tier4.jp>

* add readme in pose initializer (autowarefoundation#636)

* add readme in pose initializer

Signed-off-by: Kosuke Murakami <kosuke.murakami@tier4.jp>

* modify doc

Signed-off-by: Kosuke Murakami <kosuke.murakami@tier4.jp>

* move autoware_localization_srvs package (autowarefoundation#689)

* move autoware_localization_srvs package

* apply pre-commit

* rename pose_with_cov to pose_with_covariance

* apply pre-commit

* fix: move initial_pose_button_panel directory

Co-authored-by: mitsudome-r <ryohsuke.mitsudome@tier4.jp>
Co-authored-by: Daichi Murakami <harihitode@gmail.com>
Co-authored-by: Kenji Miyake <31987104+kenji-miyake@users.noreply.github.com>
Co-authored-by: Nikolai Morin <nikolai.morin@apex.ai>
Co-authored-by: Nikolai Morin <nnmmgit@gmail.com>
Co-authored-by: nik-tier4 <71747268+nik-tier4@users.noreply.github.com>
Co-authored-by: Ryohsuke Mitsudome <43976834+mitsudome-r@users.noreply.github.com>
Co-authored-by: Autoware <autoware@tier4.jp>
Co-authored-by: Takamasa Horibe <horibe.takamasa@gmail.com>
Co-authored-by: Servando <43142004+sgermanserrano@users.noreply.github.com>
Co-authored-by: Kosuke Murakami <kosuke.murakami@tier4.jp>
Co-authored-by: Daisuke Nishimatsu <42202095+wep21@users.noreply.github.com>
Co-authored-by: YamatoAndo <yamato.ando@gmail.com>
Co-authored-by: tkimura4 <tomoya.kimura@tier4.jp>
Co-authored-by: Kazuki Miyahara <kmiya@outlook.com>
Co-authored-by: Takagi, Isamu <43976882+isamu-takagi@users.noreply.github.com>
Co-authored-by: Hiroki OTA <hiroki.ota@tier4.jp>
Co-authored-by: Keisuke Shima <19993104+KeisukeShima@users.noreply.github.com>
Co-authored-by: Takeshi Ishita <ishitah.takeshi@gmail.com>
Co-authored-by: Takayuki Murooka <takayuki5168@gmail.com>
Co-authored-by: Takayuki Murooka <takayuki.murooka@tier4.jp>
@BonoloAWF BonoloAWF removed this from the Bus ODD Mar - Apr Milestone milestone Jul 12, 2022
1222-takeshi pushed a commit to 1222-takeshi/autoware.universe that referenced this issue Jul 26, 2023
kyoichi-sugahara pushed a commit that referenced this issue Sep 16, 2023
chore(repos): update repos

Signed-off-by: satoshi-ota <satoshi.ota928@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority:high High urgency and importance.
Projects
No open projects
Status: Done
Development

No branches or pull requests

3 participants