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

[websocket] 8k and 16k support #2505

Merged
merged 4 commits into from
Apr 30, 2024
Merged

Conversation

Sang-Hoon-Pakr
Copy link
Contributor

Wenet previously only supported 16K.
I want to apply this to the call center and needed support for 8k.

I modified part of the code from websocket_client
The code was changed to allow 8k audio to be decoded using the --sr parameter.

You can decode 8k audio by modifying the code below.

./build/bin/websocket_client_main
--hostname x.x.x.x --port xxxxx
--sr 8000
--wav_path $wav_path 2>&1 | tee client.log

@Mddct could you check this modification ?

@Mddct Mddct requested a review from pengzhendong April 26, 2024 10:31
@@ -32,13 +32,13 @@ namespace wenet {
namespace beast = boost::beast; // from <boost/beast.hpp>
namespace http = beast::http; // from <boost/beast/http.hpp>
namespace websocket = beast::websocket; // from <boost/beast/websocket.hpp>
namespace asio = boost::asio; // from <boost/asio.hpp>
namespace asio = boost::asio; // from <boost/asio.hpp> or
Copy link
Member

Choose a reason for hiding this comment

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

why add or?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

oh it is mistake!
you don't care about that.

Copy link
Member

@pengzhendong pengzhendong Apr 30, 2024

Choose a reason for hiding this comment

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

Please fix the linting issue, such as removing the trailing space.

@pengzhendong pengzhendong merged commit 675c037 into wenet-e2e:main Apr 30, 2024
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.

2 participants