-
Notifications
You must be signed in to change notification settings - Fork 20
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
Fix perfect_odometry
rosbag
#238
Conversation
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.
The topics LGTM, but when I tried to run the perfect odometry example rosbag does not start because the MCAP plugin is not installed.
After installing manually ros-humble-rosbag2-storage-mcap it worked just fine. We are probably missing a dependency declaration.
I had started the docker with ./run --build
.
This patch fixes message timestamps in the rosbag to match the header timestamps that were recorded using simulation time. The storage backend was changed to mcap, the contents of the messages were not changed. It also removes the last messages where there was a collision against one of the walls. Signed-off-by: Nahuel Espinosa <nespinosa@ekumenlabs.com>
fe5cbbe
to
04b4eb9
Compare
Signed-off-by: Nahuel Espinosa <nespinosa@ekumenlabs.com>
04b4eb9
to
5a6157d
Compare
@glpuga Good catch! I originally tested on Rolling. I reverted to using sqlite3 instead of mcap so no need to add extra dependencies. |
I looked into the results of the This can be duplicated by running:
Nothing is left on the logs to indicate why |
@glpuga I can reproduce the error in Humble, but everything works fine in Rolling. More investigation is needed. |
@glpuga I think the failure is most likely related to what @ivanpauno found here: #117 (comment) I just checked and we have adaptive recovery disabled in those benchmarks, but I don't know if it's worth digging too hard if we know that the Humble version may crash unexpectedly. We can benchmark using Rolling where the |
I see. I think it's ok we benchmark in rolling for the time being. Let's merge this then. |
Proposed changes
Fixes #232.
This patch fixes message timestamps and starting time in the rosbag to match the header stamps recorded with sim time.
The storage backend was changed to mcap.Type of change
Checklist