Skip to content

Conversation

@oliviamiller
Copy link
Member

@oliviamiller oliviamiller commented Nov 25, 2025

Add fake audio in and out components, tested functions with go client script.

@viambot viambot added the safe to test This pull request is marked safe to test from a trusted zone label Nov 25, 2025
@viambot viambot added safe to test This pull request is marked safe to test from a trusted zone and removed safe to test This pull request is marked safe to test from a trusted zone labels Nov 25, 2025
@viambot viambot added safe to test This pull request is marked safe to test from a trusted zone and removed safe to test This pull request is marked safe to test from a trusted zone labels Nov 25, 2025
@viambot viambot added safe to test This pull request is marked safe to test from a trusted zone and removed safe to test This pull request is marked safe to test from a trusted zone labels Nov 25, 2025
@viambot viambot added safe to test This pull request is marked safe to test from a trusted zone and removed safe to test This pull request is marked safe to test from a trusted zone labels Nov 25, 2025
@viambot viambot added safe to test This pull request is marked safe to test from a trusted zone and removed safe to test This pull request is marked safe to test from a trusted zone labels Nov 25, 2025
@viambot viambot added safe to test This pull request is marked safe to test from a trusted zone and removed safe to test This pull request is marked safe to test from a trusted zone labels Nov 25, 2025
@oliviamiller oliviamiller marked this pull request as ready for review November 25, 2025 20:41
Copy link
Member

@seanavery seanavery left a comment

Choose a reason for hiding this comment

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

LGTM


a := &AudioIn{
Named: conf.ResourceName().AsNamed(),
Geometry: []spatialmath.Geometry{},
Copy link
Member

Choose a reason for hiding this comment

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

Not related to this PR, but why does AudioIn need a Geometry?

Copy link
Member Author

Choose a reason for hiding this comment

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

every API has a getGeometry function so i added one to audio for consistency, it probably wouldn't ever be used tho

Copy link
Contributor

@SebastianMunozP SebastianMunozP left a comment

Choose a reason for hiding this comment

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

LGTM

@viambot viambot removed the safe to test This pull request is marked safe to test from a trusted zone label Dec 1, 2025
@viambot viambot added the safe to test This pull request is marked safe to test from a trusted zone label Dec 1, 2025
@oliviamiller oliviamiller merged commit 7004216 into viamrobotics:main Dec 1, 2025
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

safe to test This pull request is marked safe to test from a trusted zone

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants