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

Implement new Variables in JS #31

Open
GearsAD opened this issue Mar 29, 2022 · 3 comments
Open

Implement new Variables in JS #31

GearsAD opened this issue Mar 29, 2022 · 3 comments

Comments

@GearsAD
Copy link
Contributor

GearsAD commented Mar 29, 2022

  • ContinuousScalar
  • Maybe Pose1 based on discussion
@dehann
Copy link
Member

dehann commented Mar 31, 2022

I'm linking the SDK.js under Above and Beyond for now. Can reevaluate as we get closer to ICRA and some of the meat and potatoes Py JL are out the way.

@GearsAD
Copy link
Contributor Author

GearsAD commented Mar 31, 2022

Cool yeah I made another board with the JS gaps in it to track this specifically

@dehann
Copy link
Member

dehann commented Apr 7, 2022

oh and clarification, Position1, Point1, or ContinuousScalar. I'm not sure yet about Pose1, but that could just because it sounds strange to me. The main thing tripping me up on that is Position2 == Point2 has no rotation. Then we indicate rotation by using Pose2. So with Pose1 I'm expecting some kind of rotation, yet none can exist.

So my thinking is people might get used to the name Pose1, but tnen get confused by the rotation in Pose2. So my sense is we should not use Pose1, rather use Position1 / ContinuousScalar.

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

2 participants