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

Design proposal for snapshot feature #837

Merged
merged 2 commits into from
Aug 5, 2021

Conversation

camm73
Copy link
Contributor

@camm73 camm73 commented Aug 4, 2021

See #663 for details on the original feature request.

@camm73 camm73 requested a review from a team as a code owner August 4, 2021 23:43
@camm73 camm73 requested review from lihui815 and Karsten1987 and removed request for a team August 4, 2021 23:43
Signed-off-by: Cameron Miller <cammlle@amazon.com>
@camm73 camm73 force-pushed the cameron/snapshot-design-proposal branch from d538c2f to daac940 Compare August 4, 2021 23:44
Copy link
Collaborator

@emersonknapp emersonknapp left a comment

Choose a reason for hiding this comment

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

This looks good! I think it's clear enough to start implementation based on this. Let's merge after the small tweak on the invocation section

docs/design/rosbag2_snapshot_mode.md Outdated Show resolved Hide resolved
Signed-off-by: Cameron Miller <cammlle@amazon.com>
@emersonknapp emersonknapp merged commit a5d0111 into ros2:master Aug 5, 2021
camm73 added a commit to camm73/rosbag2 that referenced this pull request Aug 6, 2021
* Create design proposal for snapshot feature

Co-authored-by: Cameron Miller <cammlle@amazon.com>
Signed-off-by: Cameron Miller <cammlle@amazon.com>
WJaworskiRobotec pushed a commit to RobotecAI/rosbag2 that referenced this pull request Sep 12, 2021
* Create design proposal for snapshot feature

Co-authored-by: Cameron Miller <cammlle@amazon.com>
Signed-off-by: Wojciech Jaworski <wojciech.jaworski@robotec.ai>

Starting `rosbag2` in snapshot mode with a specified cache size:
```
$ ros2 bag record --max-cache-size 100000 --snapshot-mode [topics [topics ...]]
Copy link

Choose a reason for hiding this comment

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

Next to --max-cache-size 100000, --max-cache-duration 30 would be nice.
Omitting one would mean no limit.

YXL76 pushed a commit to YXL76/rosbag2 that referenced this pull request Nov 18, 2022
* Create design proposal for snapshot feature

Co-authored-by: Cameron Miller <cammlle@amazon.com>
YXL76 pushed a commit to YXL76/rosbag2 that referenced this pull request Nov 18, 2022
* Create design proposal for snapshot feature

Co-authored-by: Cameron Miller <cammlle@amazon.com>
YXL76 added a commit to YXL76/rosbag2 that referenced this pull request Nov 18, 2022
* Create design proposal for snapshot feature

Co-authored-by: Cameron Miller <cammlle@amazon.com>
Co-authored-by: YXL <chenxin.lan.76@gmail.com>
YXL76 pushed a commit to YXL76/rosbag2 that referenced this pull request Nov 18, 2022
* Create design proposal for snapshot feature

Co-authored-by: Cameron Miller <cammlle@amazon.com>
YXL76 pushed a commit to YXL76/rosbag2 that referenced this pull request Nov 18, 2022
* Create design proposal for snapshot feature

Co-authored-by: Cameron Miller <cammlle@amazon.com>
YXL76 pushed a commit to YXL76/rosbag2 that referenced this pull request Nov 18, 2022
* Create design proposal for snapshot feature

Co-authored-by: Cameron Miller <cammlle@amazon.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.

3 participants