team178 / 2021RomiBot

Created to face the NE FIRST BAE Systems Mini-Bot Challenges during the year of 2021
Other
0 stars 1 forks source link

PathWeaver project using wrong values for Max Velocity, Max Acceleration, and Wheel Base (in inches should be in meters) #1

Closed 22lejo closed 3 years ago

22lejo commented 3 years ago

Needs to be fixed soon as group leaders need to test them. Should be pretty easy, constants.java should have everything there.

image

AM2i9 commented 3 years ago

Have you tried: image

22lejo commented 3 years ago

Yeah I know I can just move to meters, but then it just messes PathWeaver up for whatever reason (https://github.com/team178/2021RomiBot/issues/4). Also I want characterize the Romi Bot instead of using what's in the constants (probably gonna do that today)

22lejo commented 3 years ago

Everything in meters now, still want to play around with Max Velocity and Max Acceleration. Check latest commit and https://github.com/team178/2021RomiBot/issues/4 which is now closed.

AM2i9 commented 3 years ago

It may be messed up because it doesn't convert the values that are already in there as inches into meters?

22lejo commented 3 years ago

Well from what I can interpret from my tests. It seems to be because of the opposite of what you said, and maybe something with how PathWeaver creates a default path when you create a new one.