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

Adds general lidar sensor and adds it also to the foot bot. #71

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

deeplearningrobotics
Copy link

I needed a lidar scanner for my deep learning work and therefore added it. It is basically just a slightly modified proximity sensor.

If any improvements need to be done to get the pull request approved just tell me and I will implement them.

This lidar sensor is just a slightly modified proximity sensor.
@ilpincy
Copy link
Owner

ilpincy commented Apr 27, 2018

Thanks for your work!

One issue I have with adding a LIDAR to the foot-bot is that the foot-bot, in reality, does not have a LIDAR.

If you need a similar robot with a LIDAR, check the Khepera IV plugin. The plugin offers a configurable LIDAR that allows you to add noise and choose how many readings you get out of it.

If you end up using the Khepera, make sure to install ARGoS from the sources because the current release is behind the latest development. I'll make a new release soon.

@deeplearningrobotics
Copy link
Author

Thanks for your reply!
While I did an extensive google search for lidar support for ARGOS 3 it did not list the Khepera robot.

If you want I can remove the lidar scanner from the foot bot. The general changes could make it easier for other people to add a lidar scanner to their custom robot.

I think your tool is great for reinforcement learning, especially for multi-agent environments. It would be great for those people to have a robot where a lidar scanner can be easily added as some interesting papers work with robots with lidar scanners.

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

Successfully merging this pull request may close these issues.

2 participants