-
Notifications
You must be signed in to change notification settings - Fork 260
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
[foxy backport] Latest rosbag2 #625
Closed
Closed
Changes from all commits
Commits
Show all changes
77 commits
Select commit
Hold shift + click to select a range
5d34d61
export shared_queues_vendor (#434)
Karsten1987 921b579
Add split by time to recording (#409)
jhdcs e2fce57
More reliable topic remapping test (#456)
Karsten1987 c67d8c3
minimal c++ API test (#451)
Karsten1987 1d39a5e
Add per-message ZSTD compression (#418)
pjreed fa3dc5a
comment out unused variable (#460)
Karsten1987 8c49924
use a single temp dir for the test class (#462)
dirk-thomas 1e25d22
reenable cppcheck for rosbag2_transport (#461)
Karsten1987 b229399
Use foxy testing apt repos to install linters for Actions (#463)
emersonknapp 33a3209
Reenable flake8 linter (#467)
emersonknapp 15c41a7
Consolidate ZSTD utility functions (#459)
pjreed a33c7c1
Adding db directory creation to rosbag2_cpp (#450)
Marwan99 58c5697
move wait_for_shutdown() call out of the context manager (#466)
dirk-thomas cc36fc8
introduce ros2 bag list <plugins> (#468)
Karsten1987 72eb14d
introduce defaults for the C++ API (#452)
Karsten1987 fc4dd59
add wait for closed file handles on Windows (#470)
dirk-thomas 71948dd
catch parent exception (#472)
dirk-thomas 8894248
Fix typo in error message (#475)
jacobperron a1cd778
performance testing packages (#442)
adamdbrw f5eaf4e
Add pytest.ini back to ros2bag. (#492)
clalancette 8290d7b
rosbag2_py reader and writer (#308)
mabelzhang 1228813
AMENT_IGNORE rosbag2_py for now (#509)
mabelzhang 0ef46b3
Add compression and batch size to the README (#510)
dabonnie e1689cb
Document QoS Policy overrides (#508)
380b708
disable sanitizer by default (#517)
dirk-thomas 6dfa891
Mark flaky tests as xfail for now (#520)
emersonknapp cfa377a
Add CODEOWNERS file (#519)
emersonknapp a2c0976
[ros2bag test_record] Gets rid of time.sleep and move to using comman…
jaisontj 4af4676
Remove some code duplication between SequentialWriter and SequentialC…
jaisontj 312713d
Fix rosbag2_py bug when using libc++ (#529)
ivanpauno 4548f23
Fix rosbag2_py on Windows debug and stop ignoring the package (#531)
ivanpauno be9c52e
Do not expect empty StorageOptions URI to work in *CompressionWriterT…
christophebedard a7514b3
Update the package.xml files with the latest Open Robotics maintainer…
b124188
SequentialWriter to cache by message size instead of message count (#…
jaisontj 39c2f21
Change default cache size for sequential_writer to a non zero value (…
jaisontj 211429a
Removed duplicated code in record (#534)
jaisontj f5664b1
Workaround pybind11 bug on Windows when CMAKE_BUILD_TYPE=RelWithDebI…
ivanpauno 1413139
Add record test for ros2bag (#523)
jacobperron dee8ca1
if cache data exists, it needs to flush the data into the storage bef…
fujitatomoya 63a1776
Update deprecated qos policy value names (#548)
ivanpauno bf93d7a
add storage_config_uri (#493)
Karsten1987 e22be96
correct master build (#552)
Karsten1987 f4db9a8
List all storage plugins in plugin xml file (#554)
Barry-Xu-2018 b233316
read yaml config file (#497)
Karsten1987 56df799
Prepare 0.4.0 (#566)
c1cbb67
Bump action-ros-ci to 0.1.0 (#564)
emersonknapp 350ce0e
Sqlite storage double buffering (#546)
adamdbrw 536b27e
Add back rosbag2_cpp::StorageOptions as deprecated (#563)
jacobperron a6cc439
Update codes since rcutils_calculate_directory_size() is changed (#567)
Barry-Xu-2018 3b567d5
Changelog.
clalancette 11ee716
0.5.0
clalancette 2710bdb
Stabilize test_record by reducing copies of executors and messages (#…
emersonknapp 7314479
Compress bag files in separate threads (#506)
pjreed 063632c
Only dereference the data pointer if it is valid. (#581)
clalancette 25ce937
Re-enable Action CI for PRs (#465)
emersonknapp 41afa0e
Fix a bug on parsing wrong description in plugin xml file (#578)
Barry-Xu-2018 4c2e74c
Updating performance writer (#570)
adamdbrw 1549af4
Fixed playing if unknown message types exist (#592)
84472f3
Use std::filesystem compliant non-member `exists` function for path o…
jdlangs 216e40f
SQLite storage optimized by default (#568)
adamdbrw b7da49e
include what you use (#600)
ivanpauno 3927519
Add mjeronimo to codeowners to be auto-assigned in the review rotatio…
emersonknapp b26a9a0
Fix the tests on cyclonedds by translating qos duration values (#606)
emersonknapp 1dfb13f
Mutex protection for db writing and stl collections in writer & stora…
adamdbrw 87e68f0
Remove mjeronimo from CODEOWNERS in favor of having him on the review…
emersonknapp f9c54ab
Recorder --regex and --exclude options (#604)
adamdbrw f0dedb0
Deduplicate SequentialCompressionReader business logic, add fallback …
emersonknapp 9d38957
Performance benchmarking refactor (#594)
adamdbrw 067ad4b
Fix deadlock race condition on compression shutdown (#616)
emersonknapp 320659a
Fix relative metadata paths in SequentialCompressionWriter (#613)
emersonknapp 2f5cbb7
Regex and exclude fix for rosbag recorder (#620)
pijaro 9baa79e
Fix build issues when rosbag2_storage is binary installed (#585)
pjreed ba35046
Use ZSTD's streaming interface for [de]compressing files (#543)
pjreed 3ef615a
Make compressor implementations into a plugin via pluginlib (#624)
emersonknapp 2ab6da4
Prepare bloom release 0.6.0 (#628)
emersonknapp 6007920
Fix the CHANGELOGs
e5c7951
Minimal changes to build+test for Foxy
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,3 @@ | ||
# Reviewers (and by extension approvers) will be the default owners for everything in the repo. | ||
# Unless a later match takes precedence, @reviewers will be requested for review when someone opens a pull request. | ||
* @ros2/tooling-reviewers |
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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,8 +1,4 @@ | ||
# Run linters automatically on pull requests. | ||
# | ||
# ros-tooling/actino-ros-lint is relying on the latest APT binary packages. | ||
# As of 2020-05-01, Eloquent is the latest release for which APT binary | ||
# packages are available, which is why the Docker image is based on Eloquent. | ||
name: Lint rosbag2 | ||
on: | ||
pull_request: | ||
|
@@ -12,14 +8,13 @@ jobs: | |
name: ament_copyright | ||
runs-on: ubuntu-latest | ||
container: | ||
image: rostooling/setup-ros-docker:ubuntu-bionic-ros-eloquent-ros-base-latest | ||
image: rostooling/setup-ros-docker:ubuntu-focal-ros-foxy-ros-base-testing-latest | ||
steps: | ||
# TODO(setup-ros-docker#7): calling chown is necessary for now | ||
- run: sudo chown -R rosbuild:rosbuild "$HOME" . | ||
- uses: actions/checkout@v2 | ||
- uses: ros-tooling/[email protected] | ||
with: | ||
linter: copyright | ||
distribution: foxy | ||
package-name: | | ||
ros2bag | ||
rosbag2_compression | ||
|
@@ -34,14 +29,13 @@ jobs: | |
name: ament_xmllint | ||
runs-on: ubuntu-latest | ||
container: | ||
image: rostooling/setup-ros-docker:ubuntu-bionic-ros-eloquent-ros-base-latest | ||
image: rostooling/setup-ros-docker:ubuntu-focal-ros-foxy-ros-base-testing-latest | ||
steps: | ||
# TODO(setup-ros-docker#7): calling chown is necessary for now | ||
- run: sudo chown -R rosbuild:rosbuild "$HOME" . | ||
- uses: actions/checkout@v2 | ||
- uses: ros-tooling/[email protected] | ||
with: | ||
linter: xmllint | ||
distribution: foxy | ||
package-name: | | ||
ros2bag | ||
rosbag2 | ||
|
@@ -57,19 +51,18 @@ jobs: | |
ament_lint_cpp: # Linters applicable to C++ packages | ||
name: ament_${{ matrix.linter }} | ||
runs-on: ubuntu-latest | ||
container: | ||
image: rostooling/setup-ros-docker:ubuntu-bionic-ros-eloquent-ros-base-latest | ||
strategy: | ||
fail-fast: false | ||
matrix: | ||
linter: [cppcheck, cpplint, uncrustify] | ||
container: | ||
image: rostooling/setup-ros-docker:ubuntu-focal-ros-foxy-ros-base-testing-latest | ||
steps: | ||
# TODO(setup-ros-docker#7): calling chown is necessary for now | ||
- run: sudo chown -R rosbuild:rosbuild "$HOME" . | ||
- uses: actions/checkout@v2 | ||
- uses: ros-tooling/[email protected] | ||
with: | ||
linter: ${{ matrix.linter }} | ||
distribution: foxy | ||
package-name: | | ||
rosbag2_compression | ||
rosbag2_converter_default_plugins | ||
|
@@ -83,17 +76,16 @@ jobs: | |
name: ament_${{ matrix.linter }} | ||
runs-on: ubuntu-latest | ||
container: | ||
image: rostooling/setup-ros-docker:ubuntu-bionic-ros-eloquent-ros-base-latest | ||
image: rostooling/setup-ros-docker:ubuntu-focal-ros-foxy-ros-base-testing-latest | ||
strategy: | ||
fail-fast: false | ||
matrix: | ||
linter: [pep257] # TODO re-enable flake8 when it is fixed | ||
linter: [pep257, flake8] | ||
steps: | ||
# TODO(setup-ros-docker#7): calling chown is necessary for now | ||
- run: sudo chown -R rosbuild:rosbuild "$HOME" . | ||
- uses: actions/checkout@v2 | ||
- uses: ros-tooling/[email protected] | ||
with: | ||
linter: ${{ matrix.linter }} | ||
distribution: foxy | ||
package-name: | | ||
ros2bag |
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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,6 +1,6 @@ | ||
name: Test rosbag2 | ||
on: | ||
# pull_request: | ||
pull_request: | ||
push: | ||
branches: | ||
- master | ||
|
@@ -13,11 +13,9 @@ jobs: | |
build_and_test: | ||
runs-on: ubuntu-latest | ||
container: | ||
image: rostooling/setup-ros-docker:ubuntu-focal-ros-foxy-ros-base-testing-latest | ||
image: rostooling/setup-ros-docker:ubuntu-focal-latest | ||
steps: | ||
# TODO(setup-ros-docker#7): calling chown is necessary for now | ||
- run: sudo chown -R rosbuild:rosbuild "$HOME" . | ||
- uses: ros-tooling/[email protected] | ||
- uses: ros-tooling/[email protected] | ||
with: | ||
package-name: | | ||
ros2bag | ||
|
@@ -31,6 +29,8 @@ jobs: | |
sqlite3_vendor | ||
rosbag2_test_common | ||
rosbag2_tests | ||
target-ros2-distro: foxy | ||
vcs-repo-file-url: "" | ||
- uses: actions/upload-artifact@v1 | ||
with: | ||
name: colcon-logs | ||
|
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
This file was deleted.
Oops, something went wrong.
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
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -2,8 +2,90 @@ | |
Changelog for package ros2bag | ||
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ | ||
|
||
0.3.7 (2021-02-15) | ||
------------------ | ||
0.6.0 (2021-02-01) | ||
------------------ | ||
* Recorder --regex and --exclude options (`#604 <https://github.com/ros2/rosbag2/issues/604>`_) | ||
* Fix the tests on cyclonedds by translating qos duration values (`#606 <https://github.com/ros2/rosbag2/issues/606>`_) | ||
* SQLite storage optimized by default (`#568 <https://github.com/ros2/rosbag2/issues/568>`_) | ||
* Fix a bug on parsing wrong description in plugin xml file (`#578 <https://github.com/ros2/rosbag2/issues/578>`_) | ||
* Compress bag files in separate threads (`#506 <https://github.com/ros2/rosbag2/issues/506>`_) | ||
* Contributors: Adam Dąbrowski, Barry Xu, Emerson Knapp, P. J. Reed | ||
|
||
0.5.0 (2020-12-02) | ||
------------------ | ||
* Sqlite storage double buffering (`#546 <https://github.com/ros2/rosbag2/issues/546>`_) | ||
* Double buffers | ||
* Circular queue and FLUSH option as define | ||
* Minor naming and lexical fixes. | ||
* Removed FLUSH_BUFFERS define check. | ||
* Sqlite3 storage logging fixes. | ||
* Sqlite3 storage circular buffer with pre allocated memory. | ||
* Sqlite3 storage buffers moved to shared_ptrs. | ||
* Uncrustify | ||
* Moved double buffers to writer | ||
* Buffer layer reset in seq compression writer in rosbag2 cpp | ||
* Buffer layer for rosbag2 writer refactor | ||
* Changed buffers in BufferLayer to std vectors. | ||
* BufferLayer uncrustify | ||
* Removed non-applicable test for writer cache. | ||
* BufferLayer review fixes | ||
* Rosbag metadata msgs count fixed for BufferLayer | ||
* Condition variable for buffer layer sync. | ||
* Fixed buffer locks | ||
* Buffers in BufferLayer refactored, moved into new class | ||
* Buffer layer split bags fixed. | ||
* Storage options include fix in buffer layer header. | ||
* Mutex around swapping buffers in buffer layer. | ||
* Fixed cache 0 bug in buffer layer. | ||
* Minor buffer layer refactor. | ||
* Counting messages in writer refactored. | ||
* Changed default cache size to 100Mb and updated parameter description | ||
* 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 | ||
* Applied review comments | ||
- cache consumer now takes a callback and is independent of storage | ||
- namespace changes, renaming, cleaning | ||
- counting and logging messages by topic | ||
* linter | ||
* Changes after review: fixing flushing, topic counts, and more | ||
* Fix for splitting - flushing state now correctly turns off | ||
* cache classes documentation | ||
* simplified signature | ||
* a couple of tests for cache | ||
* address review: explicit constructor and doxygen styling | ||
* Windows warnings fix | ||
* fixed type mismatch warning on Windows | ||
* added minor comment | ||
Co-authored-by: Piotr Jaroszek <[email protected]> | ||
* Contributors: Adam Dąbrowski | ||
|
||
0.4.0 (2020-11-19) | ||
------------------ | ||
* read yaml config file (`#497 <https://github.com/ros2/rosbag2/issues/497>`_) | ||
* List all storage plugins in plugin xml file (`#554 <https://github.com/ros2/rosbag2/issues/554>`_) | ||
* add storage_config_uri (`#493 <https://github.com/ros2/rosbag2/issues/493>`_) | ||
* Update deprecated qos policy value names (`#548 <https://github.com/ros2/rosbag2/issues/548>`_) | ||
* Add record test for ros2bag (`#523 <https://github.com/ros2/rosbag2/issues/523>`_) | ||
* Removed duplicated code in record (`#534 <https://github.com/ros2/rosbag2/issues/534>`_) | ||
* Change default cache size for sequential_writer to a non zero value (`#533 <https://github.com/ros2/rosbag2/issues/533>`_) | ||
* Update the package.xml files with the latest Open Robotics maintainers (`#535 <https://github.com/ros2/rosbag2/issues/535>`_) | ||
* [ros2bag test_record] Gets rid of time.sleep and move to using command.wait_for_output (`#525 <https://github.com/ros2/rosbag2/issues/525>`_) | ||
* Add pytest.ini back to ros2bag. (`#492 <https://github.com/ros2/rosbag2/issues/492>`_) | ||
* performance testing packages (`#442 <https://github.com/ros2/rosbag2/issues/442>`_) | ||
* Validate QoS profile values are not negative. (`#483 <https://github.com/ros2/rosbag2/issues/483>`_) | ||
* catch parent exception (`#472 <https://github.com/ros2/rosbag2/issues/472>`_) | ||
* add wait for closed file handles on Windows (`#470 <https://github.com/ros2/rosbag2/issues/470>`_) | ||
* introduce ros2 bag list <plugins> (`#468 <https://github.com/ros2/rosbag2/issues/468>`_) | ||
* move wait_for_shutdown() call out of the context manager (`#466 <https://github.com/ros2/rosbag2/issues/466>`_) | ||
* Adding db directory creation to rosbag2_cpp (`#450 <https://github.com/ros2/rosbag2/issues/450>`_) | ||
* use a single temp dir for the test class (`#462 <https://github.com/ros2/rosbag2/issues/462>`_) | ||
* Add per-message ZSTD compression (`#418 <https://github.com/ros2/rosbag2/issues/418>`_) | ||
* Add split by time to recording (`#409 <https://github.com/ros2/rosbag2/issues/409>`_) | ||
* Add pytest.ini so local tests don't display warning (`#446 <https://github.com/ros2/rosbag2/issues/446>`_) | ||
* Contributors: Adam Dąbrowski, Barry Xu, Chris Lalancette, Dirk Thomas, Ivan Santiago Paunovic, Jacob Perron, Jaison Titus, Jesse Ikawa, Karsten Knese, Marwan Taher, Michael Jeronimo, P. J. Reed, jhdcs | ||
|
||
0.3.6 (2021-01-05) | ||
------------------ | ||
|
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
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Allows action CI to run on the release branch correctly