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

LiDAR Performance #128

Open
SijmenHuizenga opened this issue Jun 30, 2020 · 4 comments
Open

LiDAR Performance #128

SijmenHuizenga opened this issue Jun 30, 2020 · 4 comments
Labels
enhancement New feature or request

Comments

@SijmenHuizenga
Copy link
Member

Apparently raycasting simulations can do 600,000 points per second. This simulator can only do 100,000 points per second. Does anyone have an idea on how we can improve lidar performance?

Currently lidar is using unreal's LineTraceSingleByChannel. Here is the code for the lidar sensor in unreal.

@SijmenHuizenga SijmenHuizenga added enhancement New feature or request low-prio This task has a low priority. and removed Unreal Engine 4 labels Jul 11, 2020
@llueder
Copy link
Contributor

llueder commented Aug 25, 2020

Observations:

  • Our impression was that the event simulator had still trouble with 100,000 points/s, as our AS performed significantly better when we further reduced the lidars to 60,000 points/s.
  • Our ubuntu instance running on a cloud VM with medium Power (6 vCPU, 2,6 GHz iirc) performed better than the event's Windows-based instance.
  • (Though I don't know if our system is very sensitive or the other teams just used smaller pointclouds)

Did not investigate the nature of the problem further than to check that lidar messages come at a good rate and seem to be different, judging by the number of points in every message.
When I find some time for it after exams and stuff (certainly not within the next 40 days) and we set up a simulator again, I might try to look into it further.

BTW nice to see activity here even after the event :)

@SijmenHuizenga SijmenHuizenga removed the low-prio This task has a low priority. label Aug 26, 2020
@chentairan
Copy link

This issue may improve its performance: microsoft/AirSim#4505

@wouter-heerwegh
Copy link
Member

Looks great, currently it's not in an official version of Airsim yet, let's wait until it is merged and an official version is released. Changing the airsim version however is not easy, as it has been heavily cut down and modified from the original repo, but I will open a draft PR as soon as it is merged so I can start to work on getting this improvement into the sim.

There was also an issue for the camera performance microsoft/AirSim#4101, this one has not progressed as much though.

@wouter-heerwegh
Copy link
Member

The PR microsoft/AirSim#4505 has been merged in the Colosseum fork. I'll try and start a draft pull request to update the simulator to the newest version of Airsim (aka Colosseum).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

4 participants