-
Notifications
You must be signed in to change notification settings - Fork 68
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
Coordinate System is Incorrect #155
Comments
The bottom left corner should be (0, 0) |
I would like to point out that the current system is a standard for images. Upper left pixel is 0,0 extending downwards and rightwards. It really shouldn't matter at all once you pass these coordinates to a trajectory generator, as they have no concept of global coordinate frames only local ones. |
Actually, it does matter. A trajectory that's visualized as turning to the right in PathWeaver will actually cause the robot to turn to the left in real life. This is evidenced by #137. Standard for images is not standard mathematically and this will cause unnecessary confusion for users. Even the local coordinate frame for the trajectory generator uses standard mathematical convention, which is +x going forward and +y going to the left. |
@prateekma Can you test on latest coord-fix branch? Or if you did say so cause I do not remember. |
I will test it once I get back home to the U.S. on Monday. |
WPILib's coordinate system (like standard mathematical convention) requires Y-values to increase as you travel upward. This is not the case with Pathweaver, and it will result in the exported trajectories being incorrect.
The text was updated successfully, but these errors were encountered: