Skip to content

Platform audio for Unity#293

Open
MaxHeimbrock wants to merge 6 commits into
mainfrom
max/final-platform-audio
Open

Platform audio for Unity#293
MaxHeimbrock wants to merge 6 commits into
mainfrom
max/final-platform-audio

Conversation

@MaxHeimbrock

@MaxHeimbrock MaxHeimbrock commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

Background

Integrates the Rust Platform Audio Module. The old Unity audio input and output is still supported, the platform audio is now a second path to get audio input and output.

Changes

  • Adds platform audio APIs
  • Adds choice in Meet sample to either use platform audio or Unity audio

@MaxHeimbrock MaxHeimbrock force-pushed the max/final-platform-audio branch from 149fad2 to e88ff05 Compare June 8, 2026 13:49
@MaxHeimbrock MaxHeimbrock changed the title All platform audio changes before vacation Platform audio for Unity Jun 9, 2026
@MaxHeimbrock MaxHeimbrock marked this pull request as ready for review June 9, 2026 15:28
@MaxHeimbrock MaxHeimbrock requested a review from ladvoc June 9, 2026 15:28
@MaxHeimbrock MaxHeimbrock force-pushed the max/final-platform-audio branch from 9bd2f8c to 0f2ee14 Compare June 9, 2026 15:39
Comment thread Runtime/Scripts/PlatformAudio.cs
Comment thread Runtime/Scripts/PlatformAudio.cs
Comment thread Samples~/Meet/Assets/Runtime/MeetManager.cs
@MaxHeimbrock

Copy link
Copy Markdown
Contributor Author

Add tests to init platform audio.

@MaxHeimbrock

Copy link
Copy Markdown
Contributor Author

MaxHeimbrock and others added 2 commits June 10, 2026 11:07
Mirror the C++ PlatformAudio test suites in the Unity Test Framework:

- EditMode (pure-managed, always run): AudioProcessingOptions defaults,
  AudioDevice struct, PlatformAudioSource null-arg guard.
- PlayMode unit (ADM-backed, no server): create source/track, custom
  options, enumerate + select-by-GUID, out-of-range index, start/stop
  recording. Skips via Assert.Ignore when no platform ADM is available.
- PlayMode E2E (Category=E2E): publish/unpublish round-trip, multiple
  sources from one manager, and media flow verified via an InboundRtp
  audio stat (the batch-mode-friendly substitute for the C++ frame
  callback).

Shared PlatformAudioTestHelper.TryCreateOrIgnore mirrors GTEST_SKIP for
environments without an ADM.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@MaxHeimbrock

Copy link
Copy Markdown
Contributor Author

Add tests to init platform audio.

Look at the cpp side: https://github.com/livekit/client-sdk-cpp/pull/140/changes#diff-bdb98936b19d95cb34bc02aa2a30db943ed77f57b168441ff6fb0b05bca03f70

Added tests, but on CI there is no platform audio available. Still useful for manual testing on local machines.

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.

1 participant