Skip to content

Conversation

@jeevantelukula
Copy link
Collaborator

Add the documentation for the new Webserver Demo added showcasing the audio analytics and performance stats.

Copy link
Member

@cshilwant cshilwant left a comment

Choose a reason for hiding this comment

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

Some warnings from vales could also be looked into

@github-actions
Copy link

New warnings found with rstcheck:

source/linux/Demo_User_Guides/Webserver_Demo_User_Guide.rst:112: (ERROR/3) (bash) syntax error near unexpected token `newline'
source/linux/Demo_User_Guides/Webserver_Demo_User_Guide.rst:112: (ERROR/3) (bash) `    filesink buffer-mode=2 location=<fifo_path>'

Copy link
Member

@cshilwant cshilwant left a comment

Choose a reason for hiding this comment

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

#542 (comment) needs to be addressed

Copy link
Member

@cshilwant cshilwant left a comment

Choose a reason for hiding this comment

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

Thanks for handling the suggestions. The v2 is in much better shape.
There are couple of minor fixes needed, post which the PR is good to go

@jeevantelukula
Copy link
Collaborator Author

Updated the PR according to the review comments

Copy link
Member

@StaticRocket StaticRocket left a comment

Choose a reason for hiding this comment

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

I'm mostly fine with this. The image sizes, while being rather large are still under the current mean size. I just fear that the scaled versions will still be hard to see considering the amount of dead space. Feel free to update this if you agree.

Add the documentation for the new Webserver Demo added
showcasing the audio analytics and performance stats.

Signed-off-by: Telukula Jeevan Kumar Sahu <j-sahu@ti.com>
@jeevantelukula
Copy link
Collaborator Author

I'm mostly fine with this. The image sizes, while being rather large are still under the current mean size. I just fear that the scaled versions will still be hard to see considering the amount of dead space. Feel free to update this if you agree.

@StaticRocket I wanted to show the complete view of the webpage. However, I have reduced the size of the images by around 50 percent, keeping the view same.

@jeevantelukula jeevantelukula merged commit d13feb6 into TexasInstruments:master Dec 17, 2025
5 of 6 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.

8 participants