Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

some commits #2169

Closed
wants to merge 31 commits into from
Closed

some commits #2169

wants to merge 31 commits into from

Conversation

duiniuluantanqin
Copy link
Member

add statistic info for webrtc play
for #2149

@winlinvip
Copy link
Member

👍

@duiniuluantanqin
Copy link
Member Author

duiniuluantanqin commented Jan 28, 2021

@winlinvip This commit (add kbps statistic for webrtc play 2b2b6c1) added the kbps statistic feature, only for webrtc play functionality.

TRANS_BY_GPT3

@ghostsf
Copy link
Contributor

ghostsf commented Feb 3, 2021

This webrtc live streaming is not good, right?

TRANS_BY_GPT3

@duiniuluantanqin
Copy link
Member Author

duiniuluantanqin commented Feb 3, 2021

This webrtc live streaming is not good, right?

Streaming hasn't been added, do you need to use it?

TRANS_BY_GPT3

@codecov-io
Copy link

codecov-io commented Feb 4, 2021

Codecov Report

Merging #2169 (7c68834) into develop (10205e8) will increase coverage by 0.04%.
The diff coverage is 15.68%.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop    #2169      +/-   ##
===========================================
+ Coverage    56.25%   56.29%   +0.04%     
===========================================
  Files          125      125              
  Lines        54374    54417      +43     
===========================================
+ Hits         30586    30633      +47     
+ Misses       23788    23784       -4     

| Impacted Files | Coverage Δ | |'

Translated to English while maintaining the markdown structure:

'| Impacted Files | Coverage Δ | |
|---|---|---|
| trunk/src/app/srs_app_listener.cpp | 11.15% <0.00%> (-0.30%) | ⬇️ |
| trunk/src/app/srs_app_rtc_api.cpp | 0.00% <0.00%> (ø) | |'

Translated to English while maintaining the markdown structure:

| trunk/src/app/srs_app_rtc_api.cpp | 0.00% <0.00%> (ø) | |
| trunk/src/app/srs_app_rtc_source.cpp | 11.25% <0.00%> (-0.05%) | ⬇️ |
| trunk/src/app/srs_app_rtc_conn.cpp | 9.25% <44.44%> (+0.35%) | ⬆️ |
| trunk/src/protocol/srs_rtmp_stack.cpp | 89.05% <0.00%> (+0.04%) | ⬆️ |
| trunk/src/protocol/srs_service_utility.cpp | 72.97% <0.00%> (+0.54%) | ⬆️ |
| trunk/src/app/srs_app_statistic.cpp | 8.76% <0.00%> (+8.76%) | ⬆️ |


Continue to review full report at Codecov.

Legend - Click here to learn more
| Δ = absolute <relative> (impact), ø = not affected, ? = missing data |

Translated to English while maintaining the markdown structure:

| Δ = absolute <relative> (impact), ø = not affected, ? = missing data |

Powered by Codecov. Last update 10205e8...7c68834. Read the comment docs.

TRANS_BY_GPT3

@duiniuluantanqin
Copy link
Member Author

duiniuluantanqin commented Feb 4, 2021

This webrtc live streaming is not good, right?

TRANS_BY_GPT3

@@ -134,9 +135,12 @@ class SrsTcpListener : public ISrsCoroutineHandler
};

// TODO: FIXME: Rename it. Refine it for performance issue.
class SrsUdpMuxSocket
class SrsUdpMuxSocket : public ISrsProtocolStatistic
Copy link
Member

@winlinvip winlinvip Apr 15, 2021

Choose a reason for hiding this comment

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

The structure of 'SrsUdpMuxSocket' is unreasonable and too large. It will be split in version 4.0. Therefore, the corresponding statistics may be better implemented after the split.

TRANS_BY_GPT3

string tcUrl;
srs_parse_rtmp_url(streamurl, tcUrl, stream_name);
string tcUrl;
srs_parse_rtmp_url(streamurl, tcUrl, stream_name);
Copy link
Member

@winlinvip winlinvip Apr 15, 2021

Choose a reason for hiding this comment

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

Every time I see the URL, I want to make changes to it. This part also needs to be improved before it is suitable for statistics.
Originally, there was only RTMP URL, but now it would be better to use HTTP URL specification.
There are also many and miscellaneous functions for parsing the URL, which need to be improved for better performance.

TRANS_BY_GPT3

@codecov-commenter
Copy link

codecov-commenter commented Apr 21, 2021

Codecov Report

Merging #2169 (a98d60f) into develop (efdbf37) will decrease coverage by 4.41%.
The diff coverage is 8.00%.

❗ Current head a98d60f differs from pull request most recent head 6355038. Consider uploading reports for the commit 6355038 to get more accurate results
Impacted file tree graph

@@             Coverage Diff             @@
##           develop    #2169      +/-   ##
===========================================
- Coverage    42.74%   38.32%   -4.42%     
===========================================
  Files          101      108       +7     
  Lines        35972    40187    +4215     
===========================================
+ Hits         15376    15402      +26     
- Misses       20596    24785    +4189     

| Impacted Files | Coverage Δ | |'

Translated to English while maintaining the markdown structure:

'| Impacted Files | Coverage Δ | |
|---|---|---|
| trunk/src/app/srs_app_listener.cpp | 10.69% <0.00%> (+0.95%) | ⬆️ |
| trunk/src/app/srs_app_rtc_api.cpp | 0.00% <0.00%> (ø) | |'

Translated to English while maintaining the markdown structure:

| trunk/src/app/srs_app_rtc_api.cpp | 0.00% <0.00%> (ø) | |
| trunk/src/app/srs_app_rtc_source.cpp | 12.15% <0.00%> (+4.17%) | ⬆️ |
| trunk/src/app/srs_app_statistic.cpp | 8.39% <0.00%> (+8.39%) | ⬆️ |
| trunk/src/app/srs_app_rtc_conn.cpp | 9.87% <42.10%> (+0.40%) | ⬆️ |
| trunk/src/kernel/srs_kernel_log.cpp | 75.00% <0.00%> (-25.00%) | ⬇️ |
| trunk/src/app/srs_app_hourglass.cpp | 12.26% <0.00%> (-7.35%) | ⬇️ |
| trunk/src/app/srs_app_log.cpp | 9.02% <0.00%> (-3.76%) | ⬇️ |
| trunk/src/app/srs_app_config.cpp | 64.63% <0.00%> (-2.36%) | ⬇️ |
| trunk/src/kernel/srs_kernel_flv.cpp | 96.42% <0.00%> (-1.63%) | ⬇️ |
| ... and 104 more | |


Continue to review full report at Codecov.

Legend - Click here to learn more
| Δ = absolute <relative> (impact), ø = not affected, ? = missing data |

Translated to English while maintaining the markdown structure:

| Δ = absolute <relative> (impact), ø = not affected, ? = missing data |

Powered by Codecov. Last update efdbf37...6355038. Read the comment docs.

TRANS_BY_GPT3

@duiniuluantanqin duiniuluantanqin changed the title add statistic info for webrtc play some commits May 10, 2021
@winlinvip
Copy link
Member

winlinvip commented May 12, 2021

The client stat bug #2311 fixed by 71dda68

@winlinvip
Copy link
Member

winlinvip commented Aug 27, 2021

This PR should be able to be closed, I see there are other PRs.

TRANS_BY_GPT3

@winlinvip winlinvip closed this Aug 27, 2021
@winlinvip winlinvip added the TransByAI Translated by AI/GPT. label Jul 29, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
TransByAI Translated by AI/GPT.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants