search

MirrorNetworking / kcp2k

MIT License
276 stars 47 forks source link

Increased KCP Fragments #35

Closed JesusLuvsYooh closed 1 year ago

JesusLuvsYooh commented 2 years ago

Credits to the top-men / advisors and the big guy himself.

We do get people trying to send large mb maps, or 4k HD camera footage (in Update() ) in hashtag help, so its for those rare few, + future proofing.

miwarnec commented 2 years ago

looks good, just need one 70 MB send test to be sure. needs different window sizes in test settings tho, I can add it later

miwarnec commented 2 years ago

for future reference. 70mb sending did not seem to work atm. worth considering keeping the old limit possibly

JesusLuvsYooh commented 2 years ago

Old limit was 288 kb right? I can get 10,000 kb to work with the above changes, before unity, kcp or my hardware disagrees with it. Its a massive difference, even if we cant reach the 70,000 kb we'd hoped.

I'l ask in the Discord for opinions.

miwarnec commented 2 years ago

not merged yet, sorry. considering NetworkReader/Writer fixed capacity pooling. where we would always pool max sized readers/writers. in which case the 144 KB max is quite nice. int.max nice would not be so ni ce