Time  Nick      Message
10:37 MTDiscord <herowl> @luatic
11:17 sfan5     @paradust do you plan to open a PR for that irrlicht thing?
11:40 MTDiscord <paradust> sfan5: I was planning to, but I think it is a low priority compared to some other issues I am looking at. I am working on a PR for these, also found by AI: https://luanti.dustlabs.io/netbugs.txt
11:43 sfan5     as long as it won't cause any regressions
11:46 sfan5     working on our broken netcode is kind of a waste of time. it should be replaced entirely
11:49 pgimeno   "Net effect: effectively no congestion control; on a lossy link the window ramps to 2048 and stays there, generating retransmit storms that make lag worse."
11:49 pgimeno   Sounds like an accurate description of what's been wrong with Minetest and later Luanti for... ever?
11:50 MTDiscord <paradust> I was thinking that. Something like kcp (https://github.com/skywind3000/kcp/) or enet (https://github.com/lsalzman/enet) could be a drop-in ARQ replacement, without changing the actual protocol.
11:51 MTDiscord <paradust> But I was thinking a minimal change would be more likely to be accepted than a large PR like that
11:52 sfan5     sure, having those bug fixes is still good
11:53 MTDiscord <sheriff_u3> Could always open an issue to get some discussion on replacing it though.
11:54 sfan5     we had discussions on replacing it already. the main takeway is that we should also use this opportunity to add encryption. see also the QUIC PR
11:58 sfan5     if that wasn't the case then my favorite would be usrsctp and I probably would have made a PoC already
12:04 MTDiscord <paradust> I don't really think its a good idea for me to take on a project like that, especially since the QUIC PR has been sitting for 4 months (I assume it is dead). I was just looking for the minimal fix to get the most benefit right away. Attaching extras to it (like protocol changes, encryption, etc) would most certainly kill that effort
12:06 sfan5     like I said: feel free to send as many bug fixes for netcode as long as it doesn't introduce regressions. having them is still good.
13:30 MTDiscord <luatic> merging #17359 in 10m
13:30 ShadowBot https://github.com/luanti-org/luanti/issues/17359 -- HUD inventory: allow separate background by CodingMantisUA
13:31 MTDiscord <luatic> after that i will fall off the face of the earth until september ends due to exams
13:41 sfan5     good luck
13:42 MTDiscord <luatic> thanks
18:40 MTDiscord <andrey2470t> What about #17523 ? May it be merged now?
18:40 ShadowBot https://github.com/luanti-org/luanti/issues/17523 -- Correct positioning of the player camera base when attached by Andrey2470T
20:03 Krock     This PR needs at least 2 more approvals before it may be merged. Trivial PRs need 1 approval.
20:38 cheapie   That whole mess is why celevator does the whole thing with attaching the player to another entity and then attaching *that* to the thing it actually wanted to attach to