Skip to content

perf(velocity_smoother): decrease the processing time #1453

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

Merged
merged 4 commits into from
Aug 20, 2024

Conversation

takayuki5168
Copy link

…ssing time was wrong (autowarefoundation#8161)

fix(velocity_smoother): float type of processing time was wrong

Signed-off-by: Takayuki Murooka <takayuki5168@gmail.com>
* refactor qp_interface

Signed-off-by: Takayuki Murooka <takayuki5168@gmail.com>

* variable names: m_XXX -> XXX_

Signed-off-by: Takayuki Murooka <takayuki5168@gmail.com>

* update

Signed-off-by: Takayuki Murooka <takayuki5168@gmail.com>

* update

Signed-off-by: Takayuki Murooka <takayuki5168@gmail.com>

---------

Signed-off-by: Takayuki Murooka <takayuki5168@gmail.com>
…foundation#8028)

* perf(velocity_smoother): use ProxQP for faster optimization

Signed-off-by: Takayuki Murooka <takayuki5168@gmail.com>

* consider max_iteration

Signed-off-by: Takayuki Murooka <takayuki5168@gmail.com>

* disable warm start

Signed-off-by: Takayuki Murooka <takayuki5168@gmail.com>

* fix test

Signed-off-by: Takayuki Murooka <takayuki5168@gmail.com>

---------

Signed-off-by: Takayuki Murooka <takayuki5168@gmail.com>
…utowarefoundation#8030)

* not resample debug_trajectories if not published

Signed-off-by: Takayuki Murooka <takayuki5168@gmail.com>

* update dependant packages

Signed-off-by: Takayuki Murooka <takayuki5168@gmail.com>

---------

Signed-off-by: Takayuki Murooka <takayuki5168@gmail.com>
Copy link
Collaborator

@shmpwk shmpwk left a comment

Choose a reason for hiding this comment

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

Thank you so much!

@shmpwk shmpwk marked this pull request as draft August 9, 2024 03:55
@shmpwk
Copy link
Collaborator

shmpwk commented Aug 9, 2024

I will merge this after XX1 v0.46.0 release today.

Copy link

@maxime-clem maxime-clem left a comment

Choose a reason for hiding this comment

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

Tested in Psim with the beta/v3.1.0 branch of X2 and there was no issue.

@shmpwk shmpwk marked this pull request as ready for review August 20, 2024 11:31
@shmpwk shmpwk merged commit 4a0a76a into beta/v0.29.0 Aug 20, 2024
21 checks passed
@shmpwk shmpwk deleted the perf/velocity-smoother branch August 20, 2024 11:31
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