Skip to content

Update arm docs for Python SDK move_through_joint_positions and get_3d_models - #5284

Open
Shannon Bradshaw (shannonbradshaw) wants to merge 1 commit into
mainfrom
claude/update-arm-python-sdk-methods
Open

Update arm docs for Python SDK move_through_joint_positions and get_3d_models#5284
Shannon Bradshaw (shannonbradshaw) wants to merge 1 commit into
mainfrom
claude/update-arm-python-sdk-methods

Conversation

@shannonbradshaw

@shannonbradshaw Shannon Bradshaw (shannonbradshaw) commented Aug 26, 2026

Copy link
Copy Markdown
Collaborator

Source changes

Docs changes

  • .github/workflows/sdk_protos_map.csv: Added Python method names (move_through_joint_positions, get_3d_models) so the next codegen run adds Python tabs to the auto-generated arm API reference pages (arm.md, arm-table.md).
  • docs/motion-planning/move-an-arm/move-by-joint-positions.md: Replaced the "SDK availability" caution alert that said MoveThroughJointPositions is "not currently exposed by the Python SDK" with a real Python code example using MoveOptions. Removed the "(Go)" qualifier from the comparison table. Updated the troubleshooting section to reference MoveOptions generically instead of "Go's MoveOptions".

How I found these

  • Diff of viam-python-sdk from 060e393..4395a87 showed two new public methods added to src/viam/components/arm/arm.py and client.py
  • Grep for move_through_joint_positions and MoveThroughJointPositions across the docs repo found three files referencing these methods
  • The guide page explicitly stated Python does not expose this method, which is now incorrect

Note on generated docs

The auto-generated files (static/include/components/apis/generated/arm.md and arm-table.md) are not edited in this PR. They are produced by update_sdk_methods.py from sdk_protos_map.csv. The CSV update in this PR ensures the next codegen run will add Python tabs for MoveThroughJointPositions and Get3DModels.


Generated by daily docs change agent


Generated by Claude Code


Generated by Claude Code

…d_models

The Python SDK (viam-python-sdk#1250) added move_through_joint_positions
and get_3d_models methods to the Arm component client. The docs
explicitly stated these methods were not available in Python.

- sdk_protos_map.csv: Add Python method names so the next codegen run
  adds Python tabs to the auto-generated arm API reference.
- move-by-joint-positions.md: Replace "not available in Python" warning
  with a real Python example using MoveOptions, and remove "(Go)" from
  the comparison table.
@netlify

netlify Bot commented Aug 26, 2026

Copy link
Copy Markdown

Deploy Preview for viam-docs ready!

Name Link
🔨 Latest commit 5794c56
🔍 Latest deploy log https://app.netlify.com/projects/viam-docs/deploys/6a8eaec8a2a6c80008cb5ad4
😎 Deploy Preview https://deploy-preview-5284--viam-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 44 (🟢 up 6 from production)
Accessibility: 99 (no change from production)
Best Practices: 100 (no change from production)
SEO: 92 (no change from production)
PWA: 60 (no change from production)
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify project configuration.

@viambot viambot added the safe to build This pull request is marked safe to build from a trusted zone label Aug 26, 2026
@jeremyrose-viam

Copy link
Copy Markdown
Contributor

This merge needs to be held until the next Python SDK release, but otherwise it looks good. Confirmed with Nick Hehr (@HipsterBrown) that the BREAKING note on the SDK PR is technically correct but doesn't necessitate any extra notation in the documentation.

@viam-overwatch

Copy link
Copy Markdown

Hey Shannon Bradshaw (@shannonbradshaw) — this PR has been approved and CI has been green for 3+ business days. Ready to merge?

Auto-comment from overwatch. Will not re-nudge for 7 days.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting release safe to build This pull request is marked safe to build from a trusted zone

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants