[Design] Update record parameters #1483
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Signed-off-by: Barry Xu <[email protected]>
Barry-Xu-2018
requested review from
gbiggs and
hidmic
and removed request for
a team
|
@MichaelOrlov Please help to review this PR. |
fujitatomoya
approved these changes
Oct 17, 2023
Signed-off-by: Barry Xu <[email protected]>
MichaelOrlov
approved these changes
Oct 18, 2023
There was a problem hiding this comment.
@Barry-Xu-2018 Thanks for the changes in the design document.
Looks good to me.
Barry-Xu-2018
added a commit
to Barry-Xu-2018/rosbag2
that referenced
this pull request
Oct 18, 2023
Refer to ros2#1483 Signed-off-by: Barry Xu <[email protected]>
roncapat
pushed a commit
to roncapat/rosbag2
that referenced
this pull request
Nov 4, 2023
* [Design] Update record parameters Signed-off-by: Barry Xu <[email protected]> * Revise document regarding the misunderstood part. Signed-off-by: Barry Xu <[email protected]> --------- Signed-off-by: Barry Xu <[email protected]> Signed-off-by: roncapat <[email protected]>
roncapat
pushed a commit
to roncapat/rosbag2
that referenced
this pull request
Nov 4, 2023
* [Design] Update record parameters Signed-off-by: Barry Xu <[email protected]> * Revise document regarding the misunderstood part. Signed-off-by: Barry Xu <[email protected]> --------- Signed-off-by: Barry Xu <[email protected]>
Barry-Xu-2018
added a commit
to Barry-Xu-2018/rosbag2
that referenced
this pull request
Nov 14, 2023
Refer to ros2#1483 Signed-off-by: Barry Xu <[email protected]>
Barry-Xu-2018
added a commit
to Barry-Xu-2018/rosbag2
that referenced
this pull request
Dec 13, 2023
Refer to ros2#1483 Signed-off-by: Barry Xu <[email protected]>
MichaelOrlov
pushed a commit
that referenced
this pull request
Dec 21, 2023
…1480) * Implement service record and recorded service info display Signed-off-by: Barry Xu <[email protected]> * Address MichaelOrlov's review comments Signed-off-by: Barry Xu <[email protected]> * Add tests for rosbag2_cpp::Info::read_service_info() Signed-off-by: Barry Xu <[email protected]> * Updated code on exclude parameters Refer to #1483 Signed-off-by: Barry Xu <[email protected]> * Fix flake8 errors Signed-off-by: Barry Xu <[email protected]> * Update code to remove hard-coded values to make the code more readable Signed-off-by: Barry Xu <[email protected]> * Fix a bug on exclude parameter Signed-off-by: Barry Xu <[email protected]> * Update codes on rosbag2 QoS for service event topic Signed-off-by: Barry Xu <[email protected]> * Address the second round of review comments Signed-off-by: Barry Xu <[email protected]> * Updated info test on getting service info Signed-off-by: Barry Xu <[email protected]> * Rename exclude_services to exclude_service_event Signed-off-by: Barry Xu <[email protected]> * Address flakiness in newly added rosbag2_transport tests - Got rid form ambient sleep for waiting for messages to be recorded - Exclude "/rosout" topic from recording Signed-off-by: Michael Orlov <[email protected]> * Use wait_for_srvice_to_be_ready() instead check_service_ready() - Also add default timeout parameter for ClientManager::send_request(..) Signed-off-by: Michael Orlov <[email protected]> * Add rosbag2_test_common::wait_until_condition(..) Signed-off-by: Michael Orlov <[email protected]> * Rewrite get_service_info tests to be more deterministic - Also parameterize tests to use default supported storage plugins Signed-off-by: Michael Orlov <[email protected]> * Renames in the get_service_info tests Signed-off-by: Michael Orlov <[email protected]> * Use `std::filesystem` instead of `rcpputils::fs` functions - Rationale: Deprecation of the rcpputils::fs in future See ros2/rcpputils#164 for details Signed-off-by: Michael Orlov <[email protected]> * Avoid extra metadata readout in `ros2 bag info` verb with `--verbose` Signed-off-by: Michael Orlov <[email protected]> * Add info_end_to_end test with `--verbose` parameter Signed-off-by: Michael Orlov <[email protected]> * Fix bugs in info_with_verbose_option_end_to_end_test Signed-off-by: Barry Xu <[email protected]> * Fix segment fault issue in test_rosbag2_cpp_get_service_info.cpp Signed-off-by: Barry Xu <[email protected]> * Update a fix for info_with_verbose_option_end_to_end_test Signed-off-by: Barry Xu <[email protected]> * Output a warning while setting both '-t' and 'v' for info command Signed-off-by: Barry Xu <[email protected]> * Address comments from Fujita-san Signed-off-by: Barry Xu <[email protected]> * Use topic_in_list instead of service_in_list Signed-off-by: Michael Orlov <[email protected]> * Handle cases when dependent service msg definitions could be in IDL file Signed-off-by: Michael Orlov <[email protected]> * Fix rebase errors - Delete "rosbag2_transport/qos.cpp" which was moved to the rosbag2_storage package during prior PR before rebase Signed-off-by: Michael Orlov <[email protected]> * Fix the path of a library Signed-off-by: Barry Xu <[email protected]> * Resolve the conflicts caused by the rebase Signed-off-by: Barry Xu <[email protected]> * Replace sizeof() with strlen() Signed-off-by: Barry Xu <[email protected]> * Use consistent variable name Signed-off-by: Barry Xu <[email protected]> * Fix a link error on Windows Signed-off-by: Barry Xu <[email protected]> * Use visibility control for format library Signed-off-by: Barry Xu <[email protected]> * Replace rosbag2_format_output by direct source files linkage - Also put `format_service_info(..)` and `format_bag_meta_data(..)` under the rosbag2_py namespace Signed-off-by: Michael Orlov <[email protected]> * Remove header files from pybind modules and delete visibility_control.hpp Signed-off-by: Michael Orlov <[email protected]> --------- Signed-off-by: Barry Xu <[email protected]> Signed-off-by: Michael Orlov <[email protected]> Co-authored-by: Michael Orlov <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Address the proposal.
Item 2 in #1480 (comment)