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

RRT with obstacles file included. #25

Closed
wants to merge 1 commit into from
Closed

RRT with obstacles file included. #25

wants to merge 1 commit into from

Conversation

addy1997
Copy link

@addy1997 addy1997 commented Dec 6, 2019

Hello,

I wrote code for RRT 2d using pygame. I would like to contribute it to your repository if you find it useful.

Thanks

@SZanlongo
Copy link
Member

Hi @addy1997, appreciate your taking the time to put in a pull request.

It seems that much of the functionality from the original codebase has been duplicated, which might be confusing for someone trying to use this notebook as an example. Also, keep in mind that the duplicated code is restricted to 2 dimensions. This is understandable as Pygame operates in 2 dimensions, however this again returns to the previous issue of potentially confusing someone who might eventually seek to use the underlying code in higher dimensions.

It might be useful to see these examples of the demo code using 2 and 3-dimensional representations, which can be easily scaled to higher dimensions by simply scaling the start and end points, and search space.

The animation would definitely be a useful contribution. I have actually been meaning to do so using Plot.ly.

@addy1997
Copy link
Author

addy1997 commented Dec 7, 2019

Hi @addy1997, appreciate your taking the time to put in a pull request.

It seems that much of the functionality from the original codebase has been duplicated, which might be confusing for someone trying to use this notebook as an example. Also, keep in mind that the duplicated code is restricted to 2 dimensions. This is understandable as Pygame operates in 2 dimensions, however this again returns to the previous issue of potentially confusing someone who might eventually seek to use the underlying code in higher dimensions.

It might be useful to see these examples of the demo code using 2 and 3-dimensional representations, which can be easily scaled to higher dimensions by simply scaling the start and end points, and search space.

The animation would definitely be a useful contribution. I have actually been meaning to do so using Plot.ly.

Hi @SZanlongo, I am glad that you responded to my pull up request.
I apologize for coming too straight forward and asking directly to contribute. Please believe me I had no intention to copy the entire code in undesired manner and contribute something that has previously been made.

Actually, In my robotics lab, as an absolute beginner, I have started working on UR5 robotic arm for which i have to design the RRT code in 3d. So It was just an attempt to design a module for a robotic arm and test it in 2d to examine it.

Thanks for your response.

@SZanlongo
Copy link
Member

Hey @addy1997! No problem at all! I'm simply looking to maintain a library that is usable by many people, and that often entails leaving some things out. Contributions that improve the library for users are always welcome.

If you are looking to use a 3D rrt, then the previously-mentioned examples might be useful.

Best of luck in your studies!

@SZanlongo SZanlongo closed this Dec 9, 2019
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