Skip to content

Add max_tcp_speed to arm MoveOptions docs#5043

Merged
shannonbradshaw merged 1 commit into
mainfrom
claude/add-max-tcp-speed-to-moveoptions
May 28, 2026
Merged

Add max_tcp_speed to arm MoveOptions docs#5043
shannonbradshaw merged 1 commit into
mainfrom
claude/add-max-tcp-speed-to-moveoptions

Conversation

@shannonbradshaw
Copy link
Copy Markdown
Collaborator

Source changes

Docs changes

  • docs/motion-planning/move-an-arm/move-by-joint-positions.md: Added max_tcp_speed row to the MoveOptions field table. Added MaxTCPSpeedMPerSec to the Go SDK field name list. Added a sentence clarifying that all fields are optional ceilings that can be combined.

How I found these

  • Grep matches: grep -rn "MoveOptions" docs/ found the field table at move-by-joint-positions.md:173
  • Proto diff showed new field 5 (max_tcp_speed) added to MoveOptions message

Generated by daily docs change agent


Generated by Claude Code

The arm proto gained a new optional max_tcp_speed field (field 5) in
MoveOptions, allowing users to cap the tool center point speed in meters
per second. Also adds MaxTCPSpeedMPerSec to the Go SDK field name list.

Source: viamrobotics/api#839, viamrobotics/rdk#5986

https://claude.ai/code/session_01Xd7GAtzjRzypiJK1amk4Uu
@netlify
Copy link
Copy Markdown

netlify Bot commented May 6, 2026

Deploy Preview for viam-docs ready!

Name Link
🔨 Latest commit 428d131
🔍 Latest deploy log https://app.netlify.com/projects/viam-docs/deploys/69fb0734a92c33000879706b
😎 Deploy Preview https://deploy-preview-5043--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: 42 (🔴 down 7 from production)
Accessibility: 99 (no change from production)
Best Practices: 83 (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 May 6, 2026
@viamrobotics-overwatch
Copy link
Copy Markdown

Hey @shannonbradshaw — CI is green and no reviewer is assigned yet. Could you request one when you have a chance?

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

2 similar comments
@viamrobotics-overwatch
Copy link
Copy Markdown

Hey @shannonbradshaw — CI is green and no reviewer is assigned yet. Could you request one when you have a chance?

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

@viamrobotics-overwatch
Copy link
Copy Markdown

Hey @shannonbradshaw — CI is green and no reviewer is assigned yet. Could you request one when you have a chance?

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

@shannonbradshaw shannonbradshaw requested a review from nfranczak May 26, 2026 18:43
@viamrobotics-overwatch
Copy link
Copy Markdown

Hey @shannonbradshaw — this PR has been approved and CI has been green for 15+ business days. Ready to merge?

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

@shannonbradshaw shannonbradshaw merged commit a2b26e9 into main May 28, 2026
14 checks passed
@shannonbradshaw shannonbradshaw deleted the claude/add-max-tcp-speed-to-moveoptions branch May 28, 2026 12:58
@github-actions
Copy link
Copy Markdown

🔎💬 Inkeep AI search and chat service is syncing content for source 'Viam Docs'

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

Labels

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