Skip to content

AI update based on proto changes #448

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Nov 13, 2025
Merged

AI update based on proto changes #448

merged 1 commit into from
Nov 13, 2025

Conversation

@viambot
Copy link
Member

@viambot viambot commented Nov 6, 2025

This is an AI-generated PR to update the SDK based on proto changes. The AI may make mistakes so review carefully.

Summary of Changes:
This change introduces the capability to retrieve 3D models associated with an Arm component.

  • New Protobuf messages (Get3DModelsRequest, Get3DModelsResponse) have been defined.
  • The Arm service now supports a Get3DModels RPC method.
  • Implementations in the Arm interface, client, and service layers have been added to handle this new functionality.
  • Unit tests have been included to verify the correct operation of get3DModels.

@njooma njooma marked this pull request as ready for review November 13, 2025 22:02
@njooma njooma requested a review from a team as a code owner November 13, 2025 22:02
@njooma njooma requested review from njooma and stuqdog November 13, 2025 22:02
@njooma njooma merged commit 7bfe91f into main Nov 13, 2025
2 of 4 checks passed
@njooma njooma deleted the workflow/ai-updater branch November 13, 2025 22:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@njooma njooma Awaiting requested review from njooma njooma is a code owner automatically assigned from viamrobotics/sdk-netcode

@stuqdog stuqdog Awaiting requested review from stuqdog stuqdog is a code owner automatically assigned from viamrobotics/sdk-netcode

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@viambot @njooma