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(vehicle_cmd_gate): change param to relax pedal rate limit when the vehicle velocity is slow enough #1077

Conversation

yuki-takagi-66
Copy link
Contributor

@yuki-takagi-66 yuki-takagi-66 commented Jul 18, 2024

Description

This PR relax the pedal rate limit value when the vehicle velocity is slow enough to achieve stop/go on the slopes.

universe PR: autowarefoundation/autoware.universe#7720

Tests performed

I have tested this change by two type of real vehicles.

Effects on system behavior

Not applicable.

Interface changes

Pre-review checklist for the PR author

The PR author must check the checkboxes below when creating the PR.

In-review checklist for the PR reviewers

The PR reviewers must check the checkboxes below before approval.

Post-review checklist for the PR author

The PR author must check the checkboxes below before merging.

  • There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

@yuki-takagi-66 yuki-takagi-66 marked this pull request as ready for review July 18, 2024 03:06
@yuki-takagi-66 yuki-takagi-66 force-pushed the vehicle_cmd_gate/relaxate-pedal-rate-limit-for-low-speed branch from 75ad2f7 to 2df8a0a Compare July 18, 2024 12:11
@github-actions github-actions bot added component:system System design and integration. (auto-assigned) component:control Vehicle control algorithms and mechanisms. (auto-assigned) labels Jul 18, 2024
@yuki-takagi-66 yuki-takagi-66 force-pushed the vehicle_cmd_gate/relaxate-pedal-rate-limit-for-low-speed branch from 2df8a0a to 4c57618 Compare July 18, 2024 13:38
@github-actions github-actions bot removed component:system System design and integration. (auto-assigned) component:control Vehicle control algorithms and mechanisms. (auto-assigned) labels Jul 18, 2024
@yuki-takagi-66 yuki-takagi-66 force-pushed the vehicle_cmd_gate/relaxate-pedal-rate-limit-for-low-speed branch from 4c57618 to 62f3adb Compare July 19, 2024 02:44
@github-actions github-actions bot added component:system System design and integration. (auto-assigned) component:control Vehicle control algorithms and mechanisms. (auto-assigned) labels Jul 19, 2024
Signed-off-by: Yuki Takagi <[email protected]>
Signed-off-by: Yuki Takagi <[email protected]>
@yuki-takagi-66 yuki-takagi-66 force-pushed the vehicle_cmd_gate/relaxate-pedal-rate-limit-for-low-speed branch from 62f3adb to 18f62bf Compare July 19, 2024 02:46
@github-actions github-actions bot removed component:system System design and integration. (auto-assigned) component:control Vehicle control algorithms and mechanisms. (auto-assigned) labels Jul 19, 2024
Signed-off-by: Yuki Takagi <[email protected]>
@yuki-takagi-66
Copy link
Contributor Author

@TakaHoribe
Here is launch side PR.

Copy link
Contributor

@TakaHoribe TakaHoribe left a comment

Choose a reason for hiding this comment

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

LGTM

@yuki-takagi-66 yuki-takagi-66 merged commit b12fdff into autowarefoundation:main Jul 22, 2024
11 checks passed
@yuki-takagi-66 yuki-takagi-66 deleted the vehicle_cmd_gate/relaxate-pedal-rate-limit-for-low-speed branch July 22, 2024 05:21
yuki-takagi-66 added a commit to tier4/autoware_launch that referenced this pull request Jul 22, 2024
…he vehicle velocity is slow enough (autowarefoundation#1077)

* change param

Signed-off-by: Yuki Takagi <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants