-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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
chore!: upgrade to Nebula v0.2.1 #5275
chore!: upgrade to Nebula v0.2.1 #5275
Conversation
43830cb
to
105dc0f
Compare
test with pr branchesIn the following failing test I did:
2024-09-25.19-00-35.mp4I've checked out to all the relevant branches and did a clean build, autoware is not working correctly in the rosbag replay sim demo. test with main branchNow I will re-do the same test in the main branch to make sure. 2024-09-25.19-12-31.mp4I've also ran it in 1.0 rate and it still works in main branch. Something is wrong with the new configuration. I've double checked the pr branches and new config behaved as shown in the video again. |
The main issue was this bug introduced with the latest Nebula version, which cause the cloud timestamps to be rounded down to the nearest second: After fixing it, the simulation looks much better, but due to the way scan cutting and the |
Signed-off-by: Max SCHMELLER <max.schmeller@tier4.jp>
105dc0f
to
d2cef91
Compare
@xmfcx has been merged: timestamp-fix.mp4 |
@mojomex Thanks for updating. But from your video, I see slight jittering in the point cloud and subsequently the point cloud localization. I will test this again regardless to see and compare it before&after the PR. |
Hi @xmfcx, comparing it to the main branch I don't really see a difference. As I see it, main also has jitter. Could you go into more detail of what you think is not right? Thank you for your time 🙇 |
@mojomex Sorry for the delay, yesterday I was testing the latest AWSIM Labs - Autoware latest main incompabilities and fixes. I didn't have time to test this. 🙇 Today I will test it again.
That is good to hear 👍 |
@xmfcx Sorry if you already started evaluating: I implemented the above-mentioned fix (tier4/nebula#201) but did not update the version tag referenced by Autoware. Will fix in a minute 🙇 |
Signed-off-by: Max SCHMELLER <max.schmeller@tier4.jp>
Test 2I will keep updating this message as I make the tests. Latest mainautoware_2024-10-03_10-38-56_UTC.repos 2024-10-03.13-34-55.mp4Nebula 0.2.1
2024-10-03.14-09-00.mp4It works! |
Description
This PR changes the Nebula version to v0.2.0.
This is a breaking change for the way Nebula is launched, and thus, this PR has to be merged in tandem with
Similar PRs have been made to TIER IV's internal repos, for example:
I will merge all PRs in tandem once all of them get approved and have passing CI.
Please do not merge them independently.
Tests performed
Logging simulator with sample_sensor_kit succeeded, all pointclouds were being published:

The logs are looking fine too:
sample_sensor_kit.log
As for
single_lidar_sensor_kit
, it failed with an unrelated error:If anyone knows how to properly launch it (I didn't find any documentation on that), please let me know.
Effects on system behavior
Nebula pointclouds will experience less packet loss and Nebula should be more stable.
Interface changes
/diagnostics
whenlaunch_driver
is enabled./<vendor>_packets
whenlaunch_driver
is disabled (= no real sensor is connected)Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.
In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.
Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.
After all checkboxes are checked, anyone who has write access can merge the PR.