Skip to content

Tools: Test: Audio: Fixes and updates to process_test.m#9448

Merged
kv2019i merged 2 commits intothesofproject:mainfrom
singalsu:process_test_path_prefix_fixes
Sep 9, 2024
Merged

Tools: Test: Audio: Fixes and updates to process_test.m#9448
kv2019i merged 2 commits intothesofproject:mainfrom
singalsu:process_test_path_prefix_fixes

Conversation

@singalsu
Copy link
Collaborator

@singalsu singalsu commented Sep 6, 2024

No description provided.

The full process test fails due to changed paths of Octave
module setup scripts.

The path EQ blob decoder has changed after the EQ tool was moved
under the module. Also the SOF ABI version retrieve function
path need to be set to tune/common to use the generic function
instead of the first version that was made for EQ export.

Signed-off-by: Seppo Ingalsuo <seppo.ingalsuo@linux.intel.com>
This change replaces 6th argument with show_plots. If it is
set to 1, the plots are not hidden. If set to 2, also the
temporary sound files are not deleted that is useful for
debugging.

Signed-off-by: Seppo Ingalsuo <seppo.ingalsuo@linux.intel.com>
@singalsu
Copy link
Collaborator Author

singalsu commented Sep 6, 2024

The plots were earlier hidden by default to speed up batch tests in CI, now in addition to fix to run full test, an option is added to show the plots without need to edit the script.

E.g. process_test('drc', 32, 32, 48000, 1, 1);

Screenshot from 2024-09-06 17-35-04

@singalsu singalsu marked this pull request as ready for review September 6, 2024 15:26
@kv2019i kv2019i merged commit 2950246 into thesofproject:main Sep 9, 2024
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.

3 participants