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

Doc fixups #546

Merged
merged 4 commits into from
Sep 1, 2018
Merged

Doc fixups #546

merged 4 commits into from
Sep 1, 2018

Conversation

wjwwood
Copy link
Member

@wjwwood wjwwood commented Aug 31, 2018

These are some issues raised by doxygen that were pointed out to me.

@wjwwood wjwwood added the in review Waiting for review (Kanban column) label Aug 31, 2018
@wjwwood wjwwood self-assigned this Aug 31, 2018
@wjwwood
Copy link
Member Author

wjwwood commented Aug 31, 2018

CI (since I actually touched code):

  • Linux Build Status
  • Linux-aarch64 Build Status
  • macOS Build Status
  • Windows Build Status

@wjwwood wjwwood merged commit 198c6da into master Sep 1, 2018
@wjwwood wjwwood deleted the doc_fixup branch September 1, 2018 01:32
@wjwwood wjwwood removed the in review Waiting for review (Kanban column) label Sep 1, 2018
nnmm pushed a commit to ApexAI/rclcpp that referenced this pull request Jul 9, 2022
Signed-off-by: Borja Outerelo <borjaouterelo@gmail.com>
DensoADAS pushed a commit to DensoADAS/rclcpp that referenced this pull request Aug 5, 2022
* Double buffers

Signed-off-by: Piotr Jaroszek <piotr.jaroszek@robotec.ai>

* Circular queue and FLUSH option as define

Signed-off-by: Piotr Jaroszek <piotr.jaroszek@robotec.ai>

* Minor naming and lexical fixes.

Signed-off-by: Piotr Jaroszek <piotr.jaroszek@robotec.ai>

* Removed FLUSH_BUFFERS define check.

Signed-off-by: Piotr Jaroszek <piotr.jaroszek@robotec.ai>

* Sqlite3 storage logging fixes.

Signed-off-by: Piotr Jaroszek <piotr.jaroszek@robotec.ai>

* Sqlite3 storage circular buffer with pre allocated memory.

Signed-off-by: Piotr Jaroszek <piotr.jaroszek@robotec.ai>

* Sqlite3 storage buffers moved to shared_ptrs.

Signed-off-by: Piotr Jaroszek <piotr.jaroszek@robotec.ai>

* Uncrustify

Signed-off-by: Piotr Jaroszek <piotr.jaroszek@robotec.ai>

* Moved double buffers to writer

Signed-off-by: Piotr Jaroszek <piotr.jaroszek@robotec.ai>

* Buffer layer reset in seq compression writer in rosbag2 cpp

Signed-off-by: Piotr Jaroszek <piotr.jaroszek@robotec.ai>

* Buffer layer for rosbag2 writer refactor

Signed-off-by: Piotr Jaroszek <piotr.jaroszek@robotec.ai>

* Changed buffers in BufferLayer to std vectors.

Signed-off-by: Piotr Jaroszek <piotr.jaroszek@robotec.ai>

* BufferLayer uncrustify

Signed-off-by: Piotr Jaroszek <piotr.jaroszek@robotec.ai>

* Removed non-applicable test for writer cache.

Signed-off-by: Piotr Jaroszek <piotr.jaroszek@robotec.ai>

* BufferLayer review fixes

Signed-off-by: Piotr Jaroszek <piotr.jaroszek@robotec.ai>

* Rosbag metadata msgs count fixed for BufferLayer

Signed-off-by: Piotr Jaroszek <piotr.jaroszek@robotec.ai>

* Condition variable for buffer layer sync.

Signed-off-by: Piotr Jaroszek <piotr.jaroszek@robotec.ai>

* Fixed buffer locks

Signed-off-by: Piotr Jaroszek <piotr.jaroszek@robotec.ai>

* Buffers in BufferLayer refactored, moved into new class

Signed-off-by: Piotr Jaroszek <piotr.jaroszek@robotec.ai>

* Buffer layer split bags fixed.

Signed-off-by: Piotr Jaroszek <piotr.jaroszek@robotec.ai>

* Storage options include fix in buffer layer header.

Signed-off-by: Piotr Jaroszek <piotr.jaroszek@robotec.ai>

* Mutex around swapping buffers in buffer layer.

Signed-off-by: Piotr Jaroszek <piotr.jaroszek@robotec.ai>

* Fixed cache 0 bug in buffer layer.

Signed-off-by: Piotr Jaroszek <piotr.jaroszek@robotec.ai>

* Minor buffer layer refactor.

Signed-off-by: Piotr Jaroszek <piotr.jaroszek@robotec.ai>

* Counting messages in writer refactored.

Signed-off-by: Piotr Jaroszek <piotr.jaroszek@robotec.ai>

* Changed default cache size to 100Mb and updated parameter description

Signed-off-by: Adam Dabrowski <adam.dabrowski@robotec.ai>

* Applied review remarks:
- significant refactoring: separation of cache classes
- applied suggested improvements
- some renaming
- reduce code duplication that would otherwise increase with cache refactor, between compression and plain writers

Signed-off-by: Adam Dabrowski <adam.dabrowski@robotec.ai>

* Applied review comments
- cache consumer now takes a callback and is independent of storage
- namespace changes, renaming, cleaning
- counting and logging messages by topic

Signed-off-by: Adam Dabrowski <adam.dabrowski@robotec.ai>

* linter

Signed-off-by: Adam Dabrowski <adam.dabrowski@robotec.ai>

* Changes after review: fixing flushing, topic counts, and more

Signed-off-by: Adam Dabrowski <adam.dabrowski@robotec.ai>

* Fix for splitting - flushing state now correctly turns off

Signed-off-by: Adam Dabrowski <adam.dabrowski@robotec.ai>

* cache classes documentation

Signed-off-by: Adam Dabrowski <adam.dabrowski@robotec.ai>

* simplified signature

Signed-off-by: Adam Dabrowski <adam.dabrowski@robotec.ai>

* a couple of tests for cache

Signed-off-by: Adam Dabrowski <adam.dabrowski@robotec.ai>

* address review: explicit constructor and doxygen styling

Signed-off-by: Adam Dabrowski <adam.dabrowski@robotec.ai>

* Windows warnings fix

Signed-off-by: Adam Dabrowski <adam.dabrowski@robotec.ai>

* fixed type mismatch warning on Windows

Signed-off-by: Adam Dabrowski <adam.dabrowski@robotec.ai>

* added minor comment

Signed-off-by: Adam Dabrowski <adam.dabrowski@robotec.ai>

Co-authored-by: Piotr Jaroszek <piotr.jaroszek@robotec.ai>
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.

2 participants