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

Feature/multiple crosswalk #780

Merged
merged 3 commits into from
Aug 25, 2017
Merged

Feature/multiple crosswalk #780

merged 3 commits into from
Aug 25, 2017

Conversation

yk-fujii
Copy link

Status

DEVELOPMENT

Description

This aims to add a new feature described in autowarefoundation/autoware_ai#966

Todos

  • Tests
  • Documentation

Steps to Test or Reproduce

- Use astar_planner's velocity_set to run on a route with multiple crosswalks.

Future works

  • Smartly to detect the distance
  • Refactoring

@yk-fujii yk-fujii requested a review from aohsato August 21, 2017 05:00
Copy link

@aohsato aohsato left a comment

Choose a reason for hiding this comment

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

This code and result (at anjo map) looks good.
But simply, it's better that this have multiple detection mode only.

@@ -8,6 +8,7 @@

<node pkg="astar_planner" type="velocity_set" name="velocity_set" output="log">
<param name="use_crosswalk_detection" value="$(arg use_crosswalk_detection)" />
<param name="enable_multiple_crosswalk_detection" value="$(arg enable_multiple_crosswalk_detection)" />
Copy link

Choose a reason for hiding this comment

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

add arg 'enable_multiple...' to launch file

@yk-fujii yk-fujii merged commit b7e3e87 into develop Aug 25, 2017
@yk-fujii yk-fujii deleted the feature/multiple_crosswalk branch August 25, 2017 01:43
@dejanpan
Copy link

@yk-fujii how does this work? Does it check if any crosswalks intersect with the planned path?

@yk-fujii
Copy link
Author

It is applied by checking [Use Crosswalk Detection]and [Enable Multiple Crosswalk Detection] in the append window of velocity_set.

Conventionally, only [Use Crosswalk Detection] was existed.
[Use Crosswalk Detection] function control a velocity by detection pedestrians on the closest crosswalk with the planned path.

This feature add [Enable Multiple Crosswalk Detection] function.
[Enable Multiple Crosswalk Detection] function detect pedestrians on the multiple crosswalks with the planned path.

@mitsudome-r mitsudome-r added the version:autoware-ai Autoware.AI label Jun 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants