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

Added a Trajectory structure to improve the user utilisation #1

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

Conversation

thibaultbarbie
Copy link

I think it is important for the user to have many different information when computing a trajectory. For example it could be useful to know the length (in cartesian space) of the trajectory, or the cost of a trajectory, or the smoothness.

To allow it we need to have a Trajectory structure that can give all of these information . I just created the structure and put the Vec<Vec> behing the "waypoints" attribute but if you think this is a good idea we can then extend it to something with more information.

By the way, I would be interested to create a crate for motion planning in Rust by first starting to modify your crate and then extending it to allow multiple motion planners, would you be interested ?

@OTL
Copy link
Collaborator

OTL commented Mar 28, 2018

Hi.

Do you know below crate?

https://github.com/samueltardieu/pathfinding

When I implemented this crate, I thought that I should add rrt to the pathfinding crate at first.
But Rust prefer small crate then larger one. So I decided to crate small rrt crate.

Could you tell me your plan?

@thibaultbarbie
Copy link
Author

Yes I know this crate, but I think it will be better to have a specific crate for robotic path planning.

I see. I understand that having small crates is more "rusty". Well, in this case we could have one big crate named "path-planning-rs" or something like this and, inside this crate we could call your crate.

However, I still think that your algorithm should output a trajectory structure instead of the raw trajectory inside a Vec<Vec> because if one day you want to expand it it will be far easier.

Base automatically changed from master to main January 28, 2021 08:45
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