Skip to content

Conversation

@martha-johnston
Copy link
Contributor

This PR does 3 things:

  1. Adds getKinematics to the arm component. Not doing anything with it right now but want to iterate on this implementation in the future
  2. make some small improvements to the arm card to make sure disposed objects aren't being used and that the sliders align with the actual value of the pose
  3. bump the flutter sdk version so I can import this to the viam app <- let me know if I should do this in it's own PR

@martha-johnston martha-johnston requested a review from a team as a code owner November 25, 2025 22:09
Copy link
Member

@stuqdog stuqdog left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think the pubspec.yaml change should be reverted but this otherwise looks reasonable to me!

pubspec.yaml Outdated
name: viam_sdk
description: Viam Robotics Flutter SDK. Connect to your Viam robots using WebRTC or gRPC, and control your robots remotely.
version: 0.12.0
version: 0.12.1
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This file gets updated automatically when we do a new release, I don't think we should be modifying this value by hand.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

oh good to know! will revert

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

Successfully merging this pull request may close these issues.

3 participants