Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Merged
merged 2 commits into from
Sep 9, 2024

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
46 of 47 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.

3 participants