Have you switched to using driver.launch from sensor.launch?

Have you switched to using driver.launch from sensor.launch?

I am using sensor.launch, but I saw the "scan_ring" param in driver.launch, I will try using driver.launch and update the issue

thanks

Hi @Samahu, I looked through the code, I am looking to reduce the vertical resolution of the Pointcloud from let's say, 128 rings to 32 rings, I was unable to do that with the existing implementation.

thanks

@shubham-shahh please test #191 which shows how to reduce the vertical resolution.

@shubham-shahh please test #191 which shows how to reduce the vertical resolution.

Hi, thanks for quick solution, I'll test and update the thread.

thanks

Hi @Samahu I tried the changes you mentioned, I am getting 32 rings as the output, but in the output cloud, the rings look distorted, below I have attached a screenshot for reference.
thanks

Hi @Samahu I tried the changes you mentioned, I am getting 32 rings as the output, but in the output cloud, the rings look distorted, below I have attached a screenshot for reference.
thanks

Hi @Samahu any update on this?

@shubham-shahh I got side tracked with another request.. I don't have an update but was planning to take another stab at it tomorrow. I have a question for you though. You described the point cloud as distorted. It is hard for me to guess how the point cloud should appear for you but does this distortion only happen with the branch I linked or the point cloud also appear distorted to you in general even when using the mainline? We recently implemented a change in which we publish the point cloud in its destaggered form (for additional details refer to #188 and #199). So maybe what you are observing is the net effect of this change. All I am doing in the linked PR is allocate smaller buffer and skip a certain amount of rings determined by this hard-coded value: