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: add Learning-Based Accel/Brake Map Calibrator #231

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

CristianGariboldi
Copy link

Description

This is the PR regarding the learning-based vehicle calibration for longitudinal dynamics already discussed and presented in the discussion thread and in the Planning and Control meeting.

Tests performed

This calibrator has been tested in simulation and real vehicles. We've been using the calibrator for a different range of vehicles in Pix Moving with high accuracy.

Additional materials here:
Pix_Longitudinal_Calibration.pdf
Calibration.pptx

Effects on system behavior

This new calibrator improves the previous one in different aspects:

  1. real data are interpolated with a neural network, which is able to provide a very accurate and smooth map, and can capture the non-linearities of the system;

  2. the whole calibration process is very user-friendly, fast and efficient;

  3. you don't need a huge amount of data for training the neural network, making the data collection process faster;

  4. the learning-based approach can be easily adapted to different kind of scenarios, and not solely on the longitudinal dynamic. In my previous pull request for example, I also included the calibration for steering/parking scenarios, just by adding 1 input to the network;

  5. It can be used with a diverse range of vehicles, with different dynamics, kinematics, engines and transmission systems;

  6. It has been tested in simulation and with real vehicles. We now use it for calibrating our vehicles in Pix Moving.

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.

Copy link

github-actions bot commented Mar 28, 2025

Thank you for contributing to the Autoware project!

🚧 If your pull request is in progress, switch it to draft mode.

Please ensure:

takayuki5168 and others added 3 commits March 28, 2025 17:23
…log (autowarefoundation#228)

Signed-off-by: Takayuki Murooka <takayuki5168@gmail.com>
Signed-off-by: CristianGariboldi <gariboldicristian@gmail.com>
Signed-off-by: CristianGariboldi <gariboldicristian@gmail.com>
Signed-off-by: CristianGariboldi <gariboldicristian@gmail.com>
pre-commit-ci bot and others added 3 commits March 28, 2025 08:25
Signed-off-by: CristianGariboldi <gariboldicristian@gmail.com>
Signed-off-by: CristianGariboldi <gariboldicristian@gmail.com>
@CristianGariboldi CristianGariboldi force-pushed the learning_based_calibrator branch from 389a945 to a171e56 Compare March 28, 2025 09:29
@CristianGariboldi
Copy link
Author

@xmfcx Hi, just for your information, I opened the previous pull request in Autoware Tools, as you suggested.

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.

2 participants