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

fix: interpolation interval #111

Merged
merged 1 commit into from
Sep 7, 2022
Merged

Conversation

taikitanaka3
Copy link

Description

  • fix interpolation interval not to insert bag point at high curvature path
    image

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.

@TomohitoAndo
Copy link

TomohitoAndo commented Sep 6, 2022

@taikitanaka3
Just to confirm, is this a workaround and appropriate fixes will be done in awf/universe later?

@taikitanaka3
Copy link
Author

yes will be confirmed.by interval of 2m in behavior path

@TomohitoAndo
Copy link

TomohitoAndo commented Sep 7, 2022

@taikitanaka3

by interval of 2m in behavior path

Is this change already in beta/v0.5.1 branch?

@taikitanaka3
Copy link
Author

@TomohitoAndo
No that change tier4/autoware_launch#444 is not in beta/v0.5.1

@TomohitoAndo
Copy link

I see, thanks 👍

@taikitanaka3 taikitanaka3 merged commit fc36cba into beta/v0.5.1 Sep 7, 2022
@taikitanaka3 taikitanaka3 deleted the fix/interpolation_interval branch September 7, 2022 06:07
tkimura4 pushed a commit that referenced this pull request Oct 24, 2022
maxime-clem pushed a commit to maxime-clem/autoware.universe that referenced this pull request Feb 7, 2024
* ci: add stale.yml for probot/stale

Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp>

* add a comment

Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp>

* Update .github/stale.yml

Co-authored-by: M. Fatih Cırıt <xmfcx@users.noreply.github.com>

* Update stale.yml

Co-authored-by: M. Fatih Cırıt <xmfcx@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants