The planning simulator does not work as per the tutorial. #641

yukkysaito · 2022-04-06T06:55:56Z

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

The planning simulator does not work as per the tutorial.
It seems to the pull over function of the behavior path planner does not publish.

Expected behavior

https://autowarefoundation.github.io/autoware-documentation/main/tutorials/ad-hoc-simulation/planning-simulation/

Actual behavior

The planning simulator does not work as per the tutorial.

Steps to reproduce

https://autowarefoundation.github.io/autoware-documentation/main/tutorials/ad-hoc-simulation/planning-simulation/

Versions

Possible causes

No response

Additional context

No response

rej55 · 2022-04-07T08:25:11Z

It seems to be happen because the state of scene modules is not initialized.

autoware.universe/planning/behavior_path_planner/include/behavior_path_planner/scene_module/scene_module_interface.hpp

Line 202 in 7ee98f4

BT::NodeStatus current_state_;

The initial state should be BT::NodeStatus::IDLE.
I'll fix it.

yukkysaito · 2022-04-07T09:05:02Z

@rej55 Thank you!

yukkysaito · 2022-04-07T11:46:01Z

New issues :

Poor trajectory follower performance and no parking state.
If a goal is set quickly at startup, it cannot move.

* release v0.4.0 * add use_object_recognition flag in dummy_perception_publisher (autowarefoundation#696) * remove ROS1 packages temporarily Signed-off-by: mitsudome-r <ryohsuke.mitsudome@tier4.jp> * add sample ros2 packages Signed-off-by: mitsudome-r <ryohsuke.mitsudome@tier4.jp> * remove ROS1 packages Signed-off-by: mitsudome-r <ryohsuke.mitsudome@tier4.jp> * Revert "remove ROS1 packages temporarily" This reverts commit 2e9822586a3539a32653e6bcd378715674b519ca. 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 dummy_perception_publisher to ROS2 (autowarefoundation#90) * Port dummy_perception_publisher to ROS2 * Uncrustify * Lint * Copyright * Period * Further ament_cpplint fixes * Convert calls of Duration to Duration::from_seconds where appropriate (autowarefoundation#131) * Use quotes for includes where appropriate (autowarefoundation#144) * Use quotes for includes where appropriate * Fix lint tests * Make tests pass hopefully * adding linters to dummy_perception_publisher (autowarefoundation#220) * [dummy_perception_publisher] fix launch file and installation (autowarefoundation#215) * [dummy_perception_publisher] fix launch file and installation Signed-off-by: mitsudome-r <ryohsuke.mitsudome@tier4.jp> * Apply suggestions from code review Co-authored-by: Takamasa Horibe <horibe.takamasa@gmail.com> Co-authored-by: Takamasa Horibe <horibe.takamasa@gmail.com> * reduce terminal ouput for better error message visibility (autowarefoundation#200) * reduce terminal ouput for better error message visibility Signed-off-by: mitsudome-r <ryohsuke.mitsudome@tier4.jp> * [costmap_generator] fix waiting for first transform Signed-off-by: mitsudome-r <ryohsuke.mitsudome@tier4.jp> * fix tests Signed-off-by: mitsudome-r <ryohsuke.mitsudome@tier4.jp> * fix test Signed-off-by: mitsudome-r <ryohsuke.mitsudome@tier4.jp> * modify launch file for making psim work (autowarefoundation#238) * modify launch file for making psim work Signed-off-by: kosuke murakami <kosuke.murakami@tier4.jp> * remove unnecesary ns Signed-off-by: kosuke murakami <kosuke.murakami@tier4.jp> * Ros2 v0.8.0 dummy perception publisher (autowarefoundation#286) * Remove "/" in frame_id (autowarefoundation#406) Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp> * Fix transform (autowarefoundation#420) * Replace rclcpp::Time(0) by tf2::TimePointZero() in lookupTransform Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp> * Fix canTransform Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp> * Fix test Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp> * add use_sim-time option (autowarefoundation#454) * Remove use_sim_time for set_parameter (autowarefoundation#1260) Signed-off-by: wep21 <border_goldenmarket@yahoo.co.jp> * Diable dummy_perception_publisher if argument 'scenario_simulation' i… (autowarefoundation#1275) * Diable dummy_perception_publisher if argument 'scenario_simulation' is true * Rename argument to 'disable_dummy_perception_publisher_node' from 'scenario_simulation' * change theta step for obj point cloud (autowarefoundation#1280) * Revert changes of PR autowarefoundation#1275 (autowarefoundation#1377) * Add pre-commit (autowarefoundation#1560) * add pre-commit * add pre-commit-config * add additional settings for private repository * use default pre-commit-config * update pre-commit setting * Ignore whitespace for line breaks in markdown * Update .github/workflows/pre-commit.yml Co-authored-by: Kazuki Miyahara <kmiya@outlook.com> * exclude svg * remove pretty-format-json * add double-quote-string-fixer * consider COLCON_IGNORE file when seaching modified package * format file * pre-commit fixes * Update pre-commit.yml * Update .pre-commit-config.yaml Co-authored-by: Kazuki Miyahara <kmiya@outlook.com> Co-authored-by: pre-commit <pre-commit@example.com> Co-authored-by: Kenji Miyake <31987104+kenji-miyake@users.noreply.github.com> * 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> * fix topic namespace (autowarefoundation#2054) Signed-off-by: Azumi Suzuki <azumi.suzuki@tier4.jp> * 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 * Feature/porting occlusion spot (autowarefoundation#1740) * Feature/occlusion_spot safety planner public road (autowarefoundation#1594) * add blind spot safety planner public road * remove duplicated procesing * remove unused private param * renaming fix typo add comments * fix spell check * velocity -> relative velocity * calc2d, To param, simplify search, To original * add the num possible collision gurd * computational cost reduction * Cosmetic change for PossibleCollisionInfo * add interpolation to possible collision value * refactor codes * fix details * invalid point gurd * To Param * refacotor to occlusion spot util * cosmetic change * clean up blindspot * To Occlusion Spot * revise readme * refactor drawing * for better explanation * fix velocity profile * clean up details * cosmetic change for debug marker * use max velocity in obstacle info instead * add gtest for Too Many Possible Collision case * change case * refactor readme * minor fix * add more occlusion spot explanation * replace svg * add gtest build path lanelet * hotfix lateral distance and searching method * update g test for lateral distance * use faster search * set more realistic param * add lanelet subtype highway * cosmetic change of reviews * add occlusion spot module in private area (autowarefoundation#1640) * add occlusion spot in private * For debugging change * add spline interpolation to path * add review changes * adding minor change * cosmetic change * Vector to retval * Blindspot To OcclusionSpot1 * To Occlusion Spot 2 * To Occlusions spot3 * update gtest with unified anchor * remove anchor * add test slice * simplify interpolation * Too Occlusion spot4 * add test buffer * To Occlusion spot * namespace gurd * correct slice and add interpolation first * handle self crossing with check for invation * to ros debug stream * removed unused interpolation * add readme to plant uml * cosmetic change * minor change * update readme * review change * change occlusion spot text color * To Offset no Throw * better debug marker * catch only inversion error * skip return in case of inversion * for better grid * simplify path lanelet at first * remove std::cout * for better path * limit ego path inside target lanelet location * remove last points * cosmetic change for markers * apply module for limited scene * fix interpolation gurd * for better params * do not includes path behind * remove dummy perception publisher * Revert "remove dummy perception publisher" This reverts commit 4acad985fe31dd9befaa21a16631495de6c3a117. * replace hard coded occupancy grid option in psim * remove respawn * add arg to params and remove redundunt launch * fix spell check * fix default value Co-authored-by: tkimura4 <tomoya.kimura@tier4.jp> * Feature/occlusion spot private slice size param (autowarefoundation#1703) * add min slice size * for a bit narrow lateral distance * Update planning/scenario_planning/lane_driving/behavior_planning/behavior_velocity_planner/config/occlusion_spot_param.yaml Co-authored-by: Maxime CLEMENT <78338830+maxime-clem@users.noreply.github.com> Co-authored-by: Maxime CLEMENT <78338830+maxime-clem@users.noreply.github.com> * Rename files Signed-off-by: wep21 <border_goldenmarket@yahoo.co.jp> * Porting to ros2 Signed-off-by: wep21 <border_goldenmarket@yahoo.co.jp> * pre-commit fixes Signed-off-by: wep21 <border_goldenmarket@yahoo.co.jp> * Fix typo Signed-off-by: wep21 <border_goldenmarket@yahoo.co.jp> * Fix launch namespace Co-authored-by: tkimura4 <tomoya.kimura@tier4.jp> * Fix parameter type Co-authored-by: tkimura4 <tomoya.kimura@tier4.jp> * Update planning/scenario_planning/lane_driving/behavior_planning/behavior_velocity_planner/src/scene_module/occlusion_spot/scene_occlusion_spot_in_private_road.cpp Co-authored-by: tkimura4 <tomoya.kimura@tier4.jp> Co-authored-by: taikitanaka3 <65527974+taikitanaka3@users.noreply.github.com> Co-authored-by: tkimura4 <tomoya.kimura@tier4.jp> Co-authored-by: Maxime CLEMENT <78338830+maxime-clem@users.noreply.github.com> * 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> * port dummy perception publisher to auto (autowarefoundation#562) * port dummy perception publisher to auto * autoware_perception_msgs/DynamicObjectWithFeatureArray convert to autoware_perception_msgs/DetectedObjectsWithFeature * change tracked objects to PREDICTED objects * separate pub type with real * Add README.md to dummy perception publisher (autowarefoundation#641) * Added readme for dummy_perception_pub * README update * README update * Fix pre-commit * fix typo * Update README.md * Update README.md * Update README.md * Modified node.cpp * Modified README.md * change parameter name * Update README.md * [shape_estimation]change type (autowarefoundation#663) * change output type of shape_estimation * remove unused function * add dynamic_object_converter * rename * fix typo * fix dummy_perception_publisher * update readme * fix copyright * rename package * add readme * fix launch name * remove unused variable * fix readme * fix convert function * change topic name of DynamicObjectsWithFeature * Fix no ground pointcloud topic name (autowarefoundation#733) Signed-off-by: j4tfwm6z <proj-jpntaxi@tier4.jp> Co-authored-by: j4tfwm6z <proj-jpntaxi@tier4.jp> * auto/fix occupancy grid name space in dummy perception publisher (autowarefoundation#739) * fix name space * change namespace: object_segmentation -> obstacle_segmentation * feat: add use_traffic_light status Co-authored-by: mitsudome-r <ryohsuke.mitsudome@tier4.jp> Co-authored-by: Taichi Higashide <taichi.higashide@tier4.jp> 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: Takamasa Horibe <horibe.takamasa@gmail.com> Co-authored-by: Kosuke Murakami <kosuke.murakami@tier4.jp> Co-authored-by: Daisuke Nishimatsu <42202095+wep21@users.noreply.github.com> Co-authored-by: Kenji Miyake <31987104+kenji-miyake@users.noreply.github.com> Co-authored-by: tkimura4 <tomoya.kimura@tier4.jp> Co-authored-by: Tatsuya Yamasaki <httperror@404-notfound.jp> Co-authored-by: taikitanaka3 <65527974+taikitanaka3@users.noreply.github.com> Co-authored-by: Keisuke Shima <19993104+KeisukeShima@users.noreply.github.com> Co-authored-by: Kazuki Miyahara <kmiya@outlook.com> Co-authored-by: pre-commit <pre-commit@example.com> Co-authored-by: Hiroki OTA <hiroki.ota@tier4.jp> Co-authored-by: Azumi Suzuki <38061530+s-azumi@users.noreply.github.com> Co-authored-by: Maxime CLEMENT <78338830+maxime-clem@users.noreply.github.com> Co-authored-by: Yohei Mishina <66298900+YoheiMishina@users.noreply.github.com> Co-authored-by: j4tfwm6z <proj-jpntaxi@tier4.jp> Co-authored-by: Satoshi OTA <44889564+satoshi-ota@users.noreply.github.com>

mitsudome-r · 2022-06-21T08:46:22Z

@yukkysaito Do we still have issues with the tutorial? I believe they are working with the newest code.

xmfcx · 2022-07-19T08:45:59Z

We'll close this issue and it can be re-opened if it occurs again.

chore: sync upstream

* feat(autoware.repos): remove velodyne_vls, add nebula for LiDAR driver (autowarefoundation#632) * chore(autoware.repos): change target branch for transport_drivers from tcp to boost (autowarefoundation#637) chore(repos): update repos Signed-off-by: satoshi-ota <satoshi.ota928@gmail.com> Co-authored-by: satoshi-ota <satoshi.ota928@gmail.com> * feat(webauto-ci): add vehicle info params (autowarefoundation#635) * feat(webauto-ci): add vehicle info params Signed-off-by: kosuke55 <kosuke.tnp@gmail.com> * fixup! feat(webauto-ci): add vehicle info params * Update .webauto-ci.yml * add comments Signed-off-by: kosuke55 <kosuke.tnp@gmail.com> * add space Signed-off-by: kosuke55 <kosuke.tnp@gmail.com> * style(pre-commit): autofix --------- Signed-off-by: kosuke55 <kosuke.tnp@gmail.com> Co-authored-by: kosuke55 <kosuke55@users.noreply.github.com> * feat(webauto-ci): traffic_light_stop_line_distance (autowarefoundation#639) feat(webauto-ci) traffic_light_stop_line_distance Signed-off-by: kosuke55 <kosuke.tnp@gmail.com> * feat(webauto-ci): add walkway_stop_line_distance (autowarefoundation#641) * fix(rclcpp): change version (autowarefoundation#642) * fix(rclcpp): change version Signed-off-by: kosuke55 <kosuke.tnp@gmail.com> * fix branch Signed-off-by: kosuke55 <kosuke.tnp@gmail.com> --------- Signed-off-by: kosuke55 <kosuke.tnp@gmail.com> --------- Signed-off-by: satoshi-ota <satoshi.ota928@gmail.com> Signed-off-by: kosuke55 <kosuke.tnp@gmail.com> Signed-off-by: Tomohito Ando <tomohito.ando@tier4.jp> Co-authored-by: David Wong <33114676+drwnz@users.noreply.github.com> Co-authored-by: Shumpei Wakabayashi <42209144+shmpwk@users.noreply.github.com> Co-authored-by: satoshi-ota <satoshi.ota928@gmail.com> Co-authored-by: Kosuke Takeuchi <kosuke.tnp@gmail.com> Co-authored-by: kosuke55 <kosuke55@users.noreply.github.com> Co-authored-by: Tomohito Ando <tomohito.ando@tier4.jp>

rej55 mentioned this issue Apr 7, 2022

fix(behavior_path_planner): initialize scene module status #650

Merged

4 tasks

yukkysaito closed this as completed Apr 7, 2022

yukkysaito reopened this Apr 7, 2022

TakaHoribe mentioned this issue Apr 11, 2022

fix(control_launch): change default mpc param to improve performance #667

Merged

4 tasks

xmfcx mentioned this issue Apr 12, 2022

Some errors are occurred in the tutorial #630

Closed

3 tasks

BonoloAWF added the type:bug Software flaws or errors. label Apr 19, 2022

takayuki5168 mentioned this issue Apr 23, 2022

feat(behavior_path_planner): make initial ego pose in drivable area #769

Merged

7 tasks

mitsudome-r assigned yukkysaito May 10, 2022

xmfcx closed this as completed Jul 19, 2022

1222-takeshi pushed a commit to 1222-takeshi/autoware.universe that referenced this issue Jul 26, 2023

Merge pull request autowarefoundation#641 from tier4/sync-upstream

9319d04

chore: sync upstream

kyoichi-sugahara pushed a commit that referenced this issue Sep 16, 2023

feat(webauto-ci): add walkway_stop_line_distance (#641)

783ab46

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

The planning simulator does not work as per the tutorial. #641

The planning simulator does not work as per the tutorial. #641

yukkysaito commented Apr 6, 2022 •

edited

Loading

rej55 commented Apr 7, 2022 •

edited

Loading

yukkysaito commented Apr 7, 2022

yukkysaito commented Apr 7, 2022

mitsudome-r commented Jun 21, 2022

xmfcx commented Jul 19, 2022

The planning simulator does not work as per the tutorial. #641

The planning simulator does not work as per the tutorial. #641

Comments

yukkysaito commented Apr 6, 2022 • edited Loading

Checklist

Description

Expected behavior

Actual behavior

Steps to reproduce

Versions

Possible causes

Additional context

rej55 commented Apr 7, 2022 • edited Loading

yukkysaito commented Apr 7, 2022

yukkysaito commented Apr 7, 2022

mitsudome-r commented Jun 21, 2022

xmfcx commented Jul 19, 2022

yukkysaito commented Apr 6, 2022 •

edited

Loading

rej55 commented Apr 7, 2022 •

edited

Loading