Skip to content

Fix COVER mode failures, audio preview, Insert-into-DAW, and add pass-through#21

Merged
lmangani merged 5 commits into
mainfrom
copilot/fix-acestep-output-and-audio-player
Mar 22, 2026
Merged

Fix COVER mode failures, audio preview, Insert-into-DAW, and add pass-through#21
lmangani merged 5 commits into
mainfrom
copilot/fix-acestep-output-and-audio-player

Conversation

Copilot AI commented Mar 22, 2026

Copy link
Copy Markdown
Contributor
  • Add AudioFormat enum (WAV=default, MP3) to AcestepAudioProcessor
  • Expose setAudioFormat / getAudioFormat on the processor
  • Pass --wav flag to ace-synth when WAV format is selected; MP3 is ace-synth's default so no flag needed
  • Persist format in both global config and DAW project state; old configs without the key default to WAV
  • Add "Output format" combo (WAV 48kHz / MP3) to the Settings tab in the editor
  • Wire combo to load on construction and every Settings tab visit; save via Apply Settings
  • Refactor: parseAudioFormat static helper on processor; audioFormatFromComboId / comboIdFromAudioFormat + named kFmtIdWav/kFmtIdMp3 constants in editor (no magic numbers)
  • All existing audio-related functionality (library scan, import, drag filter, file chooser) already handled both WAV and MP3 — no changes needed
  • Build verified successfully (Linux VST3)

💬 Send tasks to Copilot coding agent from Slack and Teams to turn conversations into code. Copilot posts an update in your thread when it's finished.

Copilot AI changed the title [WIP] Fix ace-synth COVER/REFERENCE mode failures and audio player issues Fix COVER mode failures, audio preview, Insert-into-DAW, and add pass-through Mar 22, 2026
Copilot AI requested a review from lmangani March 22, 2026 15:34
@lmangani
lmangani marked this pull request as ready for review March 22, 2026 16:28
@lmangani
lmangani merged commit a944011 into main Mar 22, 2026
3 checks passed
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.

2 participants