-
Notifications
You must be signed in to change notification settings - Fork 240
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
Only pass one regular expression to filter topics #1053
Conversation
@esteve CI failing in multiple tests related to the topics filtering https://build.ros2.org/job/Rpr__rosbag2__ubuntu_jammy_amd64/175/testReport/ |
Signed-off-by: Esteve Fernandez <esteve.fernandez@tier4.jp>
7a15764
to
9948633
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@esteve LGTM. Thank you for your contribution!
Running CI: |
Signed-off-by: Michael Orlov <michael.orlov@apex.ai>
Signed-off-by: Michael Orlov <michael.orlov@apex.ai> Generate changelogs for rosbag2 0.17.0 Signed-off-by: Michael Orlov <michael.orlov@apex.ai> Update rosbag2_compression_zstd/CHANGELOG.rst Signed-off-by: Michael Orlov <michael.orlov@apex.ai> Co-authored-by: jhdcs <48914066+jhdcs@users.noreply.github.com> Update rosbag2_compression/CHANGELOG.rst Signed-off-by: Michael Orlov <michael.orlov@apex.ai> Co-authored-by: jhdcs <48914066+jhdcs@users.noreply.github.com> Update rosbag2_cpp/CHANGELOG.rst Signed-off-by: Michael Orlov <michael.orlov@apex.ai> Co-authored-by: jhdcs <48914066+jhdcs@users.noreply.github.com> Update rosbag2_storage/CHANGELOG.rst Signed-off-by: Michael Orlov <michael.orlov@apex.ai> Co-authored-by: jhdcs <48914066+jhdcs@users.noreply.github.com> Update rosbag2_transport/CHANGELOG.rst Signed-off-by: Michael Orlov <michael.orlov@apex.ai> Co-authored-by: jhdcs <48914066+jhdcs@users.noreply.github.com> Remove `instead` word in description of the #1010 in changelogs Signed-off-by: Michael Orlov <michael.orlov@apex.ai> Update changelogs with #1053 after rebase Signed-off-by: Michael Orlov <michael.orlov@apex.ai> Update changelogs details in regard to the optimization in writer Signed-off-by: Michael Orlov <michael.orlov@apex.ai>
Signed-off-by: Michael Orlov <michael.orlov@apex.ai> Generate changelogs for rosbag2 0.17.0 Signed-off-by: Michael Orlov <michael.orlov@apex.ai> Update rosbag2_compression_zstd/CHANGELOG.rst Signed-off-by: Michael Orlov <michael.orlov@apex.ai> Co-authored-by: jhdcs <48914066+jhdcs@users.noreply.github.com> Update rosbag2_compression/CHANGELOG.rst Signed-off-by: Michael Orlov <michael.orlov@apex.ai> Co-authored-by: jhdcs <48914066+jhdcs@users.noreply.github.com> Update rosbag2_cpp/CHANGELOG.rst Signed-off-by: Michael Orlov <michael.orlov@apex.ai> Co-authored-by: jhdcs <48914066+jhdcs@users.noreply.github.com> Update rosbag2_storage/CHANGELOG.rst Signed-off-by: Michael Orlov <michael.orlov@apex.ai> Co-authored-by: jhdcs <48914066+jhdcs@users.noreply.github.com> Update rosbag2_transport/CHANGELOG.rst Signed-off-by: Michael Orlov <michael.orlov@apex.ai> Co-authored-by: jhdcs <48914066+jhdcs@users.noreply.github.com> Remove `instead` word in description of the #1010 in changelogs Signed-off-by: Michael Orlov <michael.orlov@apex.ai> Update changelogs with #1053 after rebase Signed-off-by: Michael Orlov <michael.orlov@apex.ai> Update changelogs details in regard to the optimization in writer Signed-off-by: Michael Orlov <michael.orlov@apex.ai>
This PR changes the way the
--regex
option works so that only one regular expression is passed to filter topics