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

Coordinate System is Incorrect #155

Closed
prateekma opened this issue Dec 22, 2019 · 5 comments
Closed

Coordinate System is Incorrect #155

prateekma opened this issue Dec 22, 2019 · 5 comments

Comments

@prateekma
Copy link
Member

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.

@prateekma
Copy link
Member Author

The bottom left corner should be (0, 0)

@gftabor
Copy link
Contributor

gftabor commented Dec 23, 2019

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.

@prateekma
Copy link
Member Author

prateekma commented Dec 23, 2019

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.

@carbotaniuman
Copy link
Contributor

@prateekma Can you test on latest coord-fix branch? Or if you did say so cause I do not remember.

@prateekma
Copy link
Member Author

I will test it once I get back home to the U.S. on Monday.

@prateekma prateekma mentioned this issue Dec 30, 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

No branches or pull requests

3 participants