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

cmd/jupyter流式代码简化优化;cmd/jupyter stream play example simplify #549

Merged
merged 11 commits into from
Jul 10, 2024

Conversation

ZaymeShaw
Copy link
Contributor

@ZaymeShaw ZaymeShaw commented Jul 9, 2024

简化cmd/jupyter模式下的流式代码样例,做了更合理的抽象,支持直接流式获取音频编码字节流,去除了难以修改调整的多线程,只保留了必要的文章内容流式获取类,现在可以比较简单的接入服务端-客户端音频开发需求,也能支持本地音频开发测试
Simplified cmd/jupyter mode under the stream code sample, do a more reasonable abstraction, remove the difficult to modify the adjustment of multi-threading, only retain the necessary article content stream access class, now can be relatively simple access to server-side - client audio development needs, Local audio development testing is also supported
#529

@github-actions github-actions bot changed the base branch from main to dev July 9, 2024 02:01
Copy link
Member

@fumiama fumiama left a comment

Choose a reason for hiding this comment

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

Thanks!

examples/cmd/stream.py Outdated Show resolved Hide resolved
examples/cmd/stream.py Outdated Show resolved Hide resolved
examples/cmd/stream.py Outdated Show resolved Hide resolved
examples/cmd/stream.py Outdated Show resolved Hide resolved
examples/cmd/stream.py Outdated Show resolved Hide resolved
examples/cmd/stream.py Outdated Show resolved Hide resolved
examples/cmd/stream.py Outdated Show resolved Hide resolved
examples/cmd/stream.py Outdated Show resolved Hide resolved
examples/cmd/stream.py Outdated Show resolved Hide resolved
@ZaymeShaw ZaymeShaw requested a review from fumiama July 10, 2024 03:47
Copy link
Member

@fumiama fumiama left a comment

Choose a reason for hiding this comment

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

Thanks!

@fumiama fumiama merged commit c0644a5 into 2noise:dev Jul 10, 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.

2 participants