Skip to content

[Option B] Update SimulationLoop to IAudioStrategy #34

Description

@danieljsinclair

Issue

Update SimulationLoop to use IAudioStrategy instead of IAudioMode.

Background

Current SimulationLoop uses IAudioMode + IAudioRenderer pair. After consolidation to IAudioStrategy, SimulationLoop needs to be updated.

Goal

Update SimulationLoop to use IAudioStrategy interface:

  • Replace IAudioMode references with IAudioStrategy
  • Update method calls to use IAudioStrategy API
  • Maintain all existing functionality
  • Keep tests GREEN

Implementation Plan

  1. Update SimulationLoop.h to use IAudioStrategy* instead of IAudioMode*
  2. Update SimulationLoop.cpp to call IAudioStrategy methods
  3. Remove IAudioRenderer references (now part of IAudioStrategy)
  4. Update createStrategy() factory calls
  5. Ensure all telemetry/diagnostics still work
  6. Run all tests and verify GREEN

Acceptance Criteria

  1. SimulationLoop uses IAudioStrategy instead of IAudioMode
  2. All IAudioMode references removed from SimulationLoop
  3. All existing SimulationLoop tests pass
  4. All smoke tests pass (--sine, --threaded, --sync-pull)
  5. Clean build with no warnings
  6. No behavioral changes (audio sounds identical)

Testing

  • All existing SimulationLoop unit tests pass
  • All smoke tests pass
  • Simulator-level audio tests produce identical output
  • Integration tests pass

Dependencies

Related Issues

Activity

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

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions