Skip to content

Conversation

@singalsu
Copy link
Collaborator

The interpreter for LaTeX style formatting messes up text strings with underscores (to use subscript font). With disable of formatting interpreter the plot title texts from e.g. process_test.m look sane since we don't use formatted text.

The interpreter for LaTeX style formatting messes up
text strings with underscores (to use subscript font).
With disable of formatting interpreter the plot title
texts from e.g. process_test.m look sane since we don't
use formatted text.

Signed-off-by: Seppo Ingalsuo <seppo.ingalsuo@linux.intel.com>
Copilot AI review requested due to automatic review settings December 12, 2025 16:08
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR fixes the display of plot titles containing underscores in MATLAB/Octave audio test scripts. Previously, underscores were interpreted as LaTeX subscript formatting, making titles with underscores (like filenames or test names) appear incorrectly. The fix disables the LaTeX interpreter for plot titles.

  • Updated title() function calls to disable LaTeX-style text interpretation
  • Applied fix consistently across three test files (tdfb_test.m, src_test.m, process_test.m)

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
tools/test/audio/tdfb_test.m Disabled LaTeX interpreter for plot title to properly display underscores
tools/test/audio/src_test.m Disabled LaTeX interpreter for both conditional plot title branches
tools/test/audio/process_test.m Disabled LaTeX interpreter for plot titles in test result printing loop

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@lgirdwood lgirdwood merged commit 906076a into thesofproject:main Dec 15, 2025
40 of 43 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