-
Notifications
You must be signed in to change notification settings - Fork 248
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
Ability to split already existing ros2 bags by time #599
Comments
@Kaju-Bubanja Can you please provide the example of the syntax the end user does to perform this in ROS1? I checked this wiki and could not identify this feature. |
As a workaround, in the mean time, I would recommend ros2bag_tools. |
This feature (as worded) is available in rosbag2 without external packages - seehttps://github.com/ros2/rosbag2#converting-bags for use of the However in verifying that this was so, I noticed that (@Gilaadb |
I would love to help implement convert-start-offset (similar to play offset). Currently trying to understand how the .py and .cpp work. Any help would be greatly appreciated? |
resolves ros2#599 Signed-off-by: Peter Favrholdt <ros2github@how.dk>
…1455) * ros2 bag convert now excludes messages not in [start_time;end_time] resolves #599 Signed-off-by: Peter Favrholdt <ros2github@how.dk> * fix linelength and whitespace Signed-off-by: Peter Favrholdt <ros2github@how.dk> * minor fix warning %lld versus long int Signed-off-by: Peter Favrholdt <ros2github@how.dk> * Minor whitespace uncrustify fix Signed-off-by: Peter Favrholdt <ros2github@how.dk> * minor cpplint whitespace/indent change Signed-off-by: Peter Favrholdt <ros2github@how.dk> * minor whitespace fix Signed-off-by: Peter Favrholdt <ros2github@how.dk> * Remove dependency on rclcpp for sequential_writer by using uint64_t. Moved timestamp check above Get TopicInformation Added parsing test. Signed-off-by: Peter Favrholdt <ros2github@how.dk> * Remove superfluous dependency on rclcpp. Signed-off-by: Peter Favrholdt <ros2github@how.dk> * Add note that start_time_ns and end_time_ns are from storage_options. Signed-off-by: Peter Favrholdt <ros2github@how.dk> * Simplified message_within_accepted_time_range now uses int64_t. Signed-off-by: Peter Favrholdt <ros2github@how.dk> * whitespace fixes only Signed-off-by: Peter Favrholdt <ros2github@how.dk> * minor whitespace change Signed-off-by: Peter Favrholdt <ros2github@how.dk> * remove superfluous dependency on rclcpp from storage_sqlite3 Signed-off-by: Peter Favrholdt <ros2github@how.dk> --------- Signed-off-by: Peter Favrholdt <ros2github@how.dk>
This issue has been mentioned on ROS Discourse. There might be relevant details there: https://discourse.ros.org/t/ros-2-tsc-meeting-minutes-2023-10-12/34178/1 |
Description
In rosbag there is the ability to split already existing bags by time. That functionality would be useful for ros2 as well.
Related Issues
None
Completion Criteria
Implementation Notes / Suggestions
Check how it was done in rosbag and how it is currently done for bags which get split by time which are recorded live.
Testing Notes / Suggestions
Have a bag of 10s, split it after 5s to get 2 bags of 5s each.
The text was updated successfully, but these errors were encountered: