SleipnirGroup / Choreo

A graphical tool for planning time-optimized trajectories for autonomous mobile robots in the FIRST Robotics Competition.
BSD 3-Clause "New" or "Revised" License
129 stars 38 forks source link

Infeasible trajectories with non-square module configurations #549

Open Nsl106 opened 1 month ago

Nsl106 commented 1 month ago

When setting the wheelbase and trackwidth to two different numbers it seems a lot of very simple paths fail to generate. As an example, here the wheelbase is 20" and the trackwidth is 17" and it fails to find a solution (same path obviously works with a normal configuration). image

Here's the .chor file I was using for testing this. Github won't upload .chor files which is why it's .txt example.chor.txt

calcmogul commented 1 month ago

I reproduced the issue with 2024.2.3, but it generates fine on main.

calcmogul commented 1 month ago

I finished bisecting this issue. It was fixed by #505.

Nsl106 commented 3 weeks ago

I downloaded the version from here and am having trouble building paths still. Is there something else I need to do to get the version with that fix?

calcmogul commented 3 weeks ago

Could you upload a .chor that reproduces the issue with the build on main?

Nsl106 commented 3 weeks ago

So I downloaded the build from here (it was the latest I could find) and ran the installer. Not sure if there's something else I need to be doing, but none of the paths here will generate. choreo.chor.txt

Nsl106 commented 3 weeks ago

Still using the build above, it appears that the 4AndCenter path in that file doesn't generate with both wheelbase and trackwidth set to 19.75".

shueja commented 2 weeks ago

This is a reported issue exacerbated by non-rectangular module configurations.

bruingineer commented 4 days ago

@Nsl106 are you seeing the same issues with main now?

Nsl106 commented 3 days ago

It doesn't seem to have problems problem generating anymore. Though it still seems to overshoot stop points as mentioned here, but that's probably unrelated to this issue.