Skip to content

Conversation

@tmoida
Copy link
Contributor

@tmoida tmoida commented Jan 22, 2026

Optimized AudioPlayback test suite by reducing configurations from 20 to 10 and resolved YAML parameter conflicts causing LAVA execution failures.

Changes:

  • AudioPlayback: Reduced from 20 to 10 test configurations
  • Renamed Config1-20 to playback_config1-10 for naming consistency
  • meta-ar-ci-premerge.yaml: Rebalanced to 5 playback + 5 record tests
  • Fixed YAML parameter conflicts in both AudioPlayback and AudioRecord
  • Updated documentation to reflect new configuration structure

AudioPlayback reduction (20→10 configs):

  • Reduced clip configurations from 20 to 10 to optimize test suite size
  • Updated naming convention: Config1-20 → playback_config1-10
  • Modified audio_common.sh to map new 10-clip configuration
  • Updated sample rate coverage: 8 KHz to 48 KHz
  • Updated Read_me.md with new configuration table and examples

YAML parameter conflict resolution:

  • AudioPlayback.yaml: Removed --formats and --durations from run command
  • AudioRecord.yaml: Removed --durations from run command
  • Fixed "Cannot mix discovery parameters with legacy matrix parameters" error

Pre-merge CI rebalancing:

  • AudioPlayback: 5 tests (playback_config1, 3, 5, 8, 10)
  • AudioRecord: 5 tests (record_config1, 3, 5, 7, 9)
  • Total remains 10 test cases with balanced 5+5 split
  • Uses pre-staged clips at /home/AudioClips/
  • Unique result files prevent overwriting in parallel execution

Test coverage:

  • AudioPlayback: 8-48 KHz, 8-32 bit, 1-8 channels
  • AudioRecord: 8-96 KHz, 1-6 channels
  • 10 diverse configurations ensure comprehensive audio validation

…licts

Optimized AudioPlayback test suite by reducing configurations from 20 to 10
and resolved YAML parameter conflicts causing LAVA execution failures.

Changes:
- AudioPlayback: Reduced from 20 to 10 test configurations
- Renamed Config1-20 to playback_config1-10 for naming consistency
- meta-ar-ci-premerge.yaml: Rebalanced to 5 playback + 5 record tests
- Fixed YAML parameter conflicts in both AudioPlayback and AudioRecord
- Updated documentation to reflect new configuration structure

AudioPlayback reduction (20→10 configs):
* Reduced clip configurations from 20 to 10 to optimize test suite size
* Updated naming convention: Config1-20 → playback_config1-10
* Modified audio_common.sh to map new 10-clip configuration
* Updated sample rate coverage: 8 KHz to 48 KHz
* Updated Read_me.md with new configuration table and examples

YAML parameter conflict resolution:
* AudioPlayback.yaml: Removed --formats and --durations from run command
* AudioRecord.yaml: Removed --durations from run command
* Fixed "Cannot mix discovery parameters with legacy matrix parameters" error

Pre-merge CI rebalancing:
* AudioPlayback: 5 tests (playback_config1, 3, 5, 8, 10)
* AudioRecord: 5 tests (record_config1, 3, 5, 7, 9)
* Total remains 10 test cases with balanced 5+5 split
* Uses pre-staged clips at /home/AudioClips/
* Unique result files prevent overwriting in parallel execution

Test coverage:
- AudioPlayback: 8-48 KHz, 8-32 bit, 1-8 channels
- AudioRecord: 8-96 KHz, 1-6 channels
- 10 diverse configurations ensure comprehensive audio validation

Signed-off-by: Teja Swaroop Moida <tmoida@qti.qualcomm.com>
@vnarapar vnarapar self-requested a review January 23, 2026 03:07
Copy link
Contributor

@vnarapar vnarapar left a comment

Choose a reason for hiding this comment

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

LGTM

@vnarapar vnarapar merged commit 5867b6c into qualcomm-linux:main Jan 23, 2026
8 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