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(rtc_reaplyer): add rtc_replayer (#1993) #178

Merged
merged 1 commit into from
Nov 14, 2022

Conversation

taikitanaka3
Copy link

@taikitanaka3 taikitanaka3 commented Nov 11, 2022

  • feat(rtc_replayer): add rtc_reaplayer

Signed-off-by: taikitanaka3 [email protected]

Description

this can be merged if there is no build error
this will be added to debug tool
https://github.com/tier4/debug_tools/blob/ad465ef3d3e082e6a2bf42165fa6a03f4d7b4c61/analyze_without_planning.tmux#L66

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.

* feat(rtc_replayer): add rtc_reaplayer

Signed-off-by: tanaka3 <[email protected]>

* chore: small fix

Signed-off-by: tanaka3 <[email protected]>

* doc: update

Signed-off-by: taikitanaka3 <[email protected]>

* chore: add debug print

Signed-off-by: tanaka3 <[email protected]>

* fix: fix

* fix: build fail

* style: spell check

Signed-off-by: tanaka3 <[email protected]>

Signed-off-by: tanaka3 <[email protected]>
Signed-off-by: taikitanaka3 <[email protected]>
@taikitanaka3 taikitanaka3 marked this pull request as ready for review November 11, 2022 05:59
@codecov-commenter
Copy link

codecov-commenter commented Nov 11, 2022

Codecov Report

❗ No coverage uploaded for pull request base (beta/v0.5.2-odaiba3@e1a9339). Click here to learn what that means.
Patch has no changes to coverable lines.

Additional details and impacted files
@@                  Coverage Diff                  @@
##             beta/v0.5.2-odaiba3    #178   +/-   ##
=====================================================
  Coverage                       ?   0.00%           
=====================================================
  Files                          ?       1           
  Lines                          ?      51           
  Branches                       ?       0           
=====================================================
  Hits                           ?       0           
  Misses                         ?      51           
  Partials                       ?       0           
Flag Coverage Δ
differential 0.00% <0.00%> (?)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@shmpwk
Copy link
Collaborator

shmpwk commented Nov 11, 2022

Let me confirm to make sure that this PR is only for beta/v0.5.2-odaiba3 ?

@taikitanaka3
Copy link
Author

@shmpwk
This is independent package so this can be for any version, is there any other version to include this PR?

@shmpwk
Copy link
Collaborator

shmpwk commented Nov 11, 2022

Ok, I found this, thank you.
autowarefoundation#1993

I understand this is cherry-pick commit.

@taikitanaka3 taikitanaka3 merged commit ac57471 into beta/v0.5.2-odaiba3 Nov 14, 2022
@taikitanaka3 taikitanaka3 deleted the feature/rtc_reaplay branch November 14, 2022 05:25
Comment on lines +2 to +3
<arg name="param_path" default="$(find-pkg-share rtc_auto_mode_manager)/config/rtc_auto_mode_manager.param.yaml"/>

Choose a reason for hiding this comment

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

this param file does not seem to exist in odaiba3

Choose a reason for hiding this comment

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

this was added with autowarefoundation#1541 to main

maxime-clem pushed a commit to maxime-clem/autoware.universe that referenced this pull request Feb 7, 2024
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.

4 participants