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

nack相关参数可配置化(#3507) #3510

Merged
merged 4 commits into from May 12, 2024
Merged

Conversation

gongluck
Copy link
Contributor

@gongluck gongluck commented May 6, 2024

No description provided.

@xia-chu
Copy link
Member

xia-chu commented May 6, 2024

感谢您的pr 要不这些也做成配置化?
image

@xia-chu
Copy link
Member

xia-chu commented May 6, 2024

你修改的这两个参数 对重传效果的调整有限
那两个参数基本不影响抗丢包效果

@gongluck
Copy link
Contributor Author

gongluck commented May 6, 2024

你修改的这两个参数 对重传效果的调整有限

那两个参数基本不影响抗丢包效果

这两个在我们项目上对延时很有用,默认的配置有时会出现几秒的延时,但是将第一个参数设置成低于1s的话,延时就不会超过1s了。

@gongluck
Copy link
Contributor Author

gongluck commented May 6, 2024

感谢您的pr 要不这些也做成配置化?

image

我晚点再补充下哈

@xia-chu
Copy link
Member

xia-chu commented May 6, 2024

你修改的这两个参数 对重传效果的调整有限
那两个参数基本不影响抗丢包效果

这两个在我们项目上对延时很有用,默认的配置有时会出现几秒的延时,但是将第一个参数设置成低于1s的话,延时就不会超过1s了。

那奇怪啊 这两个参数是zlm发送rtp后保留的缓存长度 在接收端反馈丢包后 再重发给对方,
你缩减后,再对方反馈丢包了就无数据可发 对方就会接收丢包

@xia-chu
Copy link
Member

xia-chu commented May 6, 2024

按道理 最大重传缓存时长 应该是接收端控制的 发送端应该设置的比较大才合理

@gongluck
Copy link
Contributor Author

gongluck commented May 6, 2024

你修改的这两个参数 对重传效果的调整有限

那两个参数基本不影响抗丢包效果

这两个在我们项目上对延时很有用,默认的配置有时会出现几秒的延时,但是将第一个参数设置成低于1s的话,延时就不会超过1s了。

那奇怪啊 这两个参数是zlm发送rtp后保留的缓存长度 在接收端反馈丢包后 再重发给对方,

你缩减后,再对方反馈丢包了就无数据可发 对方就会接收丢包

因为我们加了处理pli等刷i帧的,所以能很快恢复

@gongluck
Copy link
Contributor Author

gongluck commented May 6, 2024

按道理 最大重传缓存时长 应该是接收端控制的 发送端应该设置的比较大才合理

情况是接收端是浏览器,很多东西都不能改,客户又要超低延时,所以只能修改发送端了。

@xia-chu
Copy link
Member

xia-chu commented May 9, 2024

@gongluck 要不再在config.ini文件中添加下这些配置以及相关注释?

@xia-chu xia-chu merged commit 5df2b80 into ZLMediaKit:master May 12, 2024
10 checks passed
@xia-chu
Copy link
Member

xia-chu commented May 12, 2024

感谢大佬 加个微信? xiachuziyue

@gongluck
Copy link
Contributor Author

感谢大佬 加个微信? xiachuziyue

感谢zlm的开源分享👍

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.

None yet

2 participants