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

SRT: Support Coroutine Native SRT over ST. #3010

Closed
wants to merge 41 commits into from

Conversation

xiaozhihong
Copy link
Collaborator

@xiaozhihong xiaozhihong commented Apr 15, 2022

  1. Rewrite SRT to support Coroutine Native, based on ST.
  2. Add SRT unit tests.
  3. Error based handler.

@codecov-commenter
Copy link

codecov-commenter commented Apr 15, 2022

Codecov Report

Merging #3010 (593747b) into develop (3ed4aed) will increase coverage by 0.70%.
The diff coverage is 37.66%.

❗ Current head 593747b differs from pull request most recent head edc291a. Consider uploading reports for the commit edc291a to get more accurate results

@@             Coverage Diff             @@
##           develop    #3010      +/-   ##
===========================================
+ Coverage    56.10%   56.81%   +0.70%     
===========================================
  Files          151      150       -1     
  Lines        57559    57899     +340     
===========================================
+ Hits         32292    32893     +601     
+ Misses       25267    25006     -261     

| Impacted Files | Coverage Δ | |'

Translated to English while maintaining the markdown structure:

'| Impacted Files | Coverage Δ | |
|---|---|---|
| trunk/src/app/srs_app_config.cpp | 64.92% <0.00%> (-0.63%) | ⬇️ |
| trunk/src/app/srs_app_config.hpp | 100.00% <ø> (ø) | |'

Translated to English while maintaining the markdown structure:

| trunk/src/app/srs_app_config.hpp | 100.00% <ø> (ø) | |
| trunk/src/app/srs_app_pithy_print.cpp | 31.94% <0.00%> (-0.92%) | ⬇️ |
| trunk/src/app/srs_app_rtmp_conn.cpp | 0.00% <ø> (ø) | |
| trunk/src/app/srs_app_source.cpp | 0.64% <0.00%> (ø) | |
| trunk/src/app/srs_app_srt_conn.cpp | 0.00% <0.00%> (ø) | |
| trunk/src/main/srs_main_server.cpp | 0.00% <0.00%> (ø) | |
| trunk/src/app/srs_app_srt_source.cpp | 0.51% <0.51%> (ø) | |
| trunk/src/app/srs_app_srt_server.cpp | 12.02% <12.02%> (ø) | |
| trunk/src/app/srs_app_rtc_source.cpp | 12.36% <50.00%> (ø) | |
| ... and 12 more | |


Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3ed4aed...edc291a. Read the comment docs.

TRANS_BY_GPT3

@winlinvip winlinvip changed the title Refine SRT code, with StateThread adpater SRT: Refine SRT code, with StateThread adapter. Apr 16, 2022
@winlinvip winlinvip changed the title SRT: Refine SRT code, with StateThread adapter. SRT: Support Coroutine Native SRT over ST. Jun 14, 2022
@winlinvip
Copy link
Member

Merged by 1af30de

@winlinvip winlinvip closed this Jun 14, 2022
@winlinvip winlinvip added the TransByAI Translated by AI/GPT. label Jul 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PullRequest Has PR or solution in issue. SRT It's about SRT protocol. TransByAI Translated by AI/GPT.
Projects
None yet
3 participants