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

Add some usage examples #3

Closed
chengwei0427 opened this issue Jan 7, 2022 · 10 comments
Closed

Add some usage examples #3

chengwei0427 opened this issue Jan 7, 2022 · 10 comments
Labels
documentation Improvements or additions to documentation duplicate This issue or pull request already exists

Comments

@chengwei0427
Copy link

Hi, Horvath

Great work! I'm very interested in your work. Can you add some usage demo about this work.

Thanks!

@horverno
Copy link
Member

horverno commented Jan 7, 2022

hi, sure, actually issue #2 will be about this

@horverno horverno added the duplicate This issue or pull request already exists label Jan 7, 2022
@horverno
Copy link
Member

@chengwei0427 here is a usage example:

In a new terminal start roscore:

roscore

In a new terminal go to your bag folder (e.g. ~/Downloads):

cd ~/Downloads

Download a sample rosbag:

wget https://laesze-my.sharepoint.com/:u:/g/personal/herno_o365_sze_hu/EYl_ahy5pgBBhNHt5ZkiBikBoy_j_x95E96rDtTsxueB_A?download=1 -O leaf-2021-04-23-campus.bag

Play rosbag:

rosbag play -l ~/Downloads/leaf-2021-04-23-campus.bag

In a new terminal start the urban_road_filter node, rviz and rqt_reconfigure with roslaunch:

roslaunch urban_road_filter demo1.launch

@chengwei0427
Copy link
Author

good, I'll check and update. Thank you!

@chengwei0427
Copy link
Author

chengwei0427 commented Jan 12, 2022

@chengwei0427 here is a usage example:

In a new terminal start roscore:

roscore

In a new terminal go to your bag folder (e.g. ~/Downloads):

cd ~/Downloads

Download a sample rosbag:

wget https://laesze-my.sharepoint.com/:u:/g/personal/herno_o365_sze_hu/EYl_ahy5pgBBhNHt5ZkiBikBoy_j_x95E96rDtTsxueB_A?download=1 -O leaf-2021-04-23-campus.bag

Play rosbag:

rosbag play -l ~/Downloads/leaf-2021-04-23-campus.bag

In a new terminal start the urban_road_filter node, rviz and rqt_reconfigure with roslaunch:

roslaunch urban_road_filter demo1.launch

It's amazing on my test data.
But the code is hard to read,Can you add some English annotation?

2022-01-12 14-42-10屏幕截图

@horverno
Copy link
Member

@chengwei0427 thanks for the feedback
#1 will be about the refactor and english comments in the code @dobaybalazs is assigned to this task

In the meantime you can read the published paper as a pdf. It describes the algorithm in detail.

It's amazing on my test data.
But the code is hard to read,Can you add some English annotation?

@horverno horverno pinned this issue Jan 12, 2022
@horverno horverno added the documentation Improvements or additions to documentation label Jan 12, 2022
@chengwei0427
Copy link
Author

Another question, Does this method support 16-line or 32-line lasers? If yes, what parameters need to be adjusted?
thanks again.

@horverno
Copy link
Member

horverno commented Jan 14, 2022

The original algorithm was developed for 16 channel LIDAR, so probably it will work (unless we have done something what breaks this compatibility). The star_shaped_method has the least limitations, so try to disable x_zero_method and z_zero_method in the rqt_reconfigure window. The star_shaped_method also works on special pointclouds (e.g. provided by a solid state LIDAR).

This issue is about the usage examples, so I will close this now, but feel free to open another issue about the compatibility if needed.

@chengwei0427
Copy link
Author

My questions are well solved. Thank you for your patient reply.
Thanks again.

@juliangaal
Copy link

The original algorithm was developed for 16 channel LIDAR

Can you share some good parameters for, let's say, a VLP16?

@horverno
Copy link
Member

Currently I don't have acees to VLP16, but as mentioned maybe you can try to adjust the interval param which adjusts the LIDAR's vertical resolution. Try it in a launch file, but as far as i remember rqt_reconfigure may also works.

Maybe @csaplaci also can suggest something.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation duplicate This issue or pull request already exists
Projects
None yet
Development

No branches or pull requests

3 participants