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

multifile reader #206

Merged
merged 23 commits into from
Nov 18, 2019
Merged

multifile reader #206

merged 23 commits into from
Nov 18, 2019

Conversation

Karsten1987
Copy link
Collaborator

mainly work of @piraka9011 based on #187

Anas Abou Allaban and others added 22 commits October 22, 2019 10:04
Signed-off-by: Anas Abou Allaban <allabana@amazon.com>

Introduce new SequentialReader interface

Signed-off-by: Anas Abou Allaban <allabana@amazon.com>

Introduce new SequentialReader interface

Signed-off-by: Anas Abou Allaban <allabana@amazon.com>

Introduce new SequentialReader interface

Signed-off-by: Anas Abou Allaban <allabana@amazon.com>
- Rewrite history
- Move sequential reader implementation to header/source
- Change namespaces
- Linting

Signed-off-by: Anas Abou Allaban <allabana@amazon.com>
Signed-off-by: Anas Abou Allaban <allabana@amazon.com>
Signed-off-by: Prajakta Gokhale <prajaktg@amazon.com>
Signed-off-by: Prajakta Gokhale <prajaktg@amazon.com>
* Add new reader interface
* Use the interface in sequential reader

Signed-off-by: Prajakta Gokhale <prajaktg@amazon.com>
Signed-off-by: Prajakta Gokhale <prajaktg@amazon.com>
* final reader class

Signed-off-by: Karsten Knese <karsten@openrobotics.org>

* adaptations for rosbag2_transport

Signed-off-by: Karsten Knese <karsten@openrobotics.org>

* address review comments

Signed-off-by: Karsten Knese <karsten@openrobotics.org>
Signed-off-by: Prajakta Gokhale <prajaktg@amazon.com>
Signed-off-by: Karsten Knese <karsten@openrobotics.org>
Signed-off-by: Prajakta Gokhale <prajaktg@amazon.com>
Signed-off-by: Anas Abou Allaban <allabana@amazon.com>

Introduce new SequentialReader interface

Signed-off-by: Anas Abou Allaban <allabana@amazon.com>

Introduce new SequentialReader interface

Signed-off-by: Anas Abou Allaban <allabana@amazon.com>

Introduce new SequentialReader interface

Signed-off-by: Anas Abou Allaban <allabana@amazon.com>
- Rewrite history
- Move sequential reader implementation to header/source
- Change namespaces
- Linting

Signed-off-by: Anas Abou Allaban <allabana@amazon.com>
* final reader class

Signed-off-by: Karsten Knese <karsten@openrobotics.org>

* adaptations for rosbag2_transport

Signed-off-by: Karsten Knese <karsten@openrobotics.org>

* address review comments

Signed-off-by: Karsten Knese <karsten@openrobotics.org>
Signed-off-by: Karsten Knese <karsten@openrobotics.org>
Signed-off-by: Karsten Knese <karsten@openrobotics.org>
Signed-off-by: Karsten Knese <karsten@openrobotics.org>
Signed-off-by: Karsten Knese <karsten@openrobotics.org>
Signed-off-by: Karsten Knese <karsten@openrobotics.org>
Signed-off-by: Karsten Knese <karsten@openrobotics.org>
Signed-off-by: Karsten Knese <karsten@openrobotics.org>
@Karsten1987
Copy link
Collaborator Author

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

Copy link
Contributor

@piraka9011 piraka9011 left a comment

Choose a reason for hiding this comment

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

This LGTM 👍

@Karsten1987
Copy link
Collaborator Author

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

@Karsten1987 Karsten1987 merged commit 2f50921 into master Nov 18, 2019
@Karsten1987 Karsten1987 deleted the karsten1987-multifile-reader branch November 18, 2019 17:45
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