Config
Log for #openttd on 23rd August 2021:
Times are UTC Toggle Colours
00:15:09  *** nielsm has quit IRC
00:58:49  *** Tirili has quit IRC
02:31:00  *** Wormnest has quit IRC
02:44:03  *** D-HUND has joined #openttd
02:47:25  *** debdog has quit IRC
02:59:00  *** glx has quit IRC
03:59:22  *** Flygon has joined #openttd
04:59:00  *** HerzogDeXtEr has joined #openttd
06:03:40  *** Kitrana1 has quit IRC
06:19:39  *** HerzogDeXtEr has quit IRC
06:57:25  *** Gustavo6046 has quit IRC
06:58:15  *** Gustavo6046 has joined #openttd
07:06:20  <DorpsGek> [OpenTTD/OpenTTD] TrueBrain commented on pull request #9502: Fix #9501: [Network] crash when more than one game-info query was pending https://git.io/JEmpc
07:09:13  <TrueBrain> I guess this whole "ZeroedMemoryAllocator" can be removed by now .. but yeah, scoping of PRs, so not in 9502 :D
07:16:58  *** nielsm has joined #openttd
07:27:16  <TrueBrain> https://cdn.discordapp.com/attachments/337701432230805505/879265460166733854/unknown.png <- think that is a bit more userfriendly over the weird red popup :P
07:42:39  <DorpsGek> [OpenTTD/OpenTTD] TrueBrain opened pull request #9506: Fix: [Network] show query errors in the server listing instead of error popup https://git.io/JEYTP
07:46:55  <DorpsGek> [OpenTTD/OpenTTD] TrueBrain opened pull request #9507: Fix: [Network] don't show GameScript " (v0)" for old servers https://git.io/JEYkS
07:51:21  *** andythenorth has joined #openttd
07:55:30  *** WormnestAndroid has quit IRC
07:55:43  *** WormnestAndroid has joined #openttd
07:58:14  <DorpsGek> [OpenTTD/OpenTTD] TrueBrain opened pull request #9508: Fix #9490: [Network] a full server couldn't be queried either https://git.io/JEYtn
07:58:31  <TrueBrain> right, that fixes all open issues I am aware of :)
08:03:42  <LordAro> :)
08:04:57  <TrueBrain> now on to fixing some minor stuff in the GC ..
08:04:58  <DorpsGek> [OpenTTD/OpenTTD] HelmiMUC opened issue #9509: [Bug]: set signals from another railroad system https://git.io/JEYmY
08:06:05  <DorpsGek> [OpenTTD/OpenTTD] TrueBrain commented on issue #9509: [Bug]: set signals from another railroad system https://git.io/JEYmY
08:06:37  <TrueBrain> hmm, I was wondering why I didn't get telemetry about why people switched to TURN instead of any of the other methods
08:06:43  <TrueBrain> but .. OpenTTD waits 3 seconds for a connect
08:06:47  <TrueBrain> the GC gives up after 1 second :D
08:07:08  <TrueBrain> downside of 3 seconds, it means it can take up to 9 seconds before it switches to TURN
08:07:23  <TrueBrain> (Direct-IP, STUN IPv4, STUN IPv6, TURN)
08:08:42  <TrueBrain> makes me wonder if we can stagger the first 3
08:09:00  <andythenorth> moin
08:09:26  *** Donk has joined #openttd
08:18:34  <DorpsGek> [OpenTTD/OpenTTD] HelmiMUC commented on issue #9509: [Bug]: set signals from another railroad system https://git.io/JEYmY
08:23:13  <DorpsGek> [OpenTTD/game-coordinator] TrueBrain opened pull request #47: Fix: track better why a connection (method) failed https://git.io/JEYGH
08:29:45  <TrueBrain> hmm, requires client modification, but that does allow me to stagger connection methods .. that means the wait is not like 6 or 9 seconds, but at most 3.5, before TURN is tried
08:29:59  <LordAro> is that desirable?
08:30:22  <LordAro> i thought we wanted to avoid TURN where possible?
08:30:39  <TrueBrain> yes? Guess we have a miscommunication, let me try again :)
08:30:51  <TrueBrain> say, you are IPv4 only, makes it a bit easier
08:31:00  <LordAro> if the internet connection/dns lookup/whatever is slow or whatever, could that end up using TURN where it wasn't otherwise necessary?
08:31:06  <TrueBrain> we now try: Direct-IP, wait 3 seconds. Did it fail? Right, lets try STUN. Wait 3 seconds. Failed? NOW we try TURN
08:31:15  <TrueBrain> my suggestion is to apply happy-eyes here too:
08:31:33  <TrueBrain> Direct-IP, wait 0.5 seconds, queue STUN .. now wait for 3 more seconds. Nothing? Go for TURN
08:32:03  <TrueBrain> (mind you that the GC now uses "1 second" instead of "3 seconds", but that might be the reason some people use TURN instead of STUN)
08:32:48  <TrueBrain> as in the end, we do not really care if users have a direct-ip connection or STUN connection :D
08:33:42  <TrueBrain> LordAro: I do not know. What I do know, is that the GC now goes to the next methods after a second, while the client waits 3 seconds. So there is a chance 1 second is just too short for some of those connections
08:34:13  <TrueBrain> but having players wait 6 or 9 seconds before TURN is offered, sounds like a horrible situation too :D
08:37:40  <DorpsGek> [OpenTTD/game-coordinator] TrueBrain merged pull request #47: Fix: track better why a connection (method) failed https://git.io/JEYGH
08:40:09  *** jottyfan has joined #openttd
08:42:29  <TrueBrain> did that make any sense? :P
08:51:51  *** andythenorth has left #openttd
09:00:32  *** Wolf01 has joined #openttd
09:10:08  <DorpsGek> [OpenTTD/game-coordinator] TrueBrain opened pull request #48: Fix: conclude server-probe earlier if all results are in https://git.io/JEYz8
09:10:18  <TrueBrain> that at least should improve server-probing ..
09:11:04  <TrueBrain> yeah, I think applying happy eyeballs to connecting makes most sense .. should improve user-experience, while giving more time for connections to setup
09:12:01  <TrueBrain> sadly that really means I need to modify the client, which means I need to do another protocol bump \o/
09:12:21  <DorpsGek> [OpenTTD/game-coordinator] TrueBrain merged pull request #48: Fix: conclude server-probe earlier if all results are in https://git.io/JEYz8
09:20:27  <TrueBrain> okay, some more math and drawings later .. it will only help with a small portion of the connections, so not really worth the complexity ..
09:21:17  <TrueBrain> (servers with direct-ip to which a client cannot connect .. doesn't happen often .. or a server/client with both IPv4 and IPv6 of which on both the STUN attempt times out .. it is more likely it is refused than times out, etc)
09:25:43  *** jottyfan has joined #openttd
09:37:46  *** nielsm has quit IRC
10:31:45  <DorpsGek> [OpenTTD/game-coordinator] TrueBrain opened pull request #49: Codechange: rework how connections are setup https://git.io/JEY5v
10:33:05  <DorpsGek> [OpenTTD/game-coordinator] TrueBrain merged pull request #49: Codechange: rework how connections are setup https://git.io/JEY5v
10:38:04  <TrueBrain> it seems this all scared LordAro away :P
10:38:19  <TrueBrain> either way, this should address all open issues I had with the GC .. we will see how it does in production :)
10:39:45  <DorpsGek> [OpenTTD/OpenTTD] glx22 approved pull request #9507: Fix: [Network] don't show GameScript " (v0)" for old servers https://git.io/JEYFT
10:41:23  <DorpsGek> [OpenTTD/OpenTTD] TrueBrain merged pull request #9507: Fix: [Network] don't show GameScript " (v0)" for old servers https://git.io/JEYkS
10:43:10  <TrueBrain> okay, smoke-test works .. lets put it to production and see what happens
10:43:41  <DorpsGek> [OpenTTD/game-coordinator] TrueBrain created new tag: 1.2.0 https://git.io/JEYbk
11:13:42  *** WormnestAndroid has quit IRC
11:15:32  *** WormnestAndroid has joined #openttd
11:37:40  <DorpsGek> [OpenTTD/OpenTTD] TrueBrain opened pull request #9510: Fix c4b700f1: remove left-over debug statement https://git.io/JEOTA
11:38:25  <DorpsGek> [OpenTTD/OpenTTD] LordAro approved pull request #9510: Fix c4b700f1: remove left-over debug statement https://git.io/JEOkq
11:45:43  <TrueBrain> hmm .. I solved the TURN issue .. it now no longer offer it :P
11:45:49  <TrueBrain> might not have been the goal
11:46:56  *** Gustavo6046 has quit IRC
11:47:13  *** Gustavo6046 has joined #openttd
11:56:45  <DorpsGek> [OpenTTD/game-coordinator] TrueBrain opened pull request #50: Fix: clear the result-signal before starting a new method https://git.io/JEOqR
11:56:47  <TrueBrain> found the boo-boo :)
11:59:37  <DorpsGek> [OpenTTD/game-coordinator] TrueBrain merged pull request #50: Fix: clear the result-signal before starting a new method https://git.io/JEOqR
11:59:53  <DorpsGek> [OpenTTD/game-coordinator] TrueBrain created new tag: 1.2.1 https://git.io/JEOmt
12:05:41  <DorpsGek> [OpenTTD/OpenTTD] TrueBrain merged pull request #9510: Fix c4b700f1: remove left-over debug statement https://git.io/JEOTA
12:19:36  <TrueBrain> okay .. did a bunch of testing .. some servers work 60% of the time
12:19:38  <TrueBrain> and fail the rest
12:19:40  <TrueBrain> which is really weird :P
12:19:46  <TrueBrain> but we knew that already
12:24:48  *** glx has joined #openttd
12:24:48  *** ChanServ sets mode: +v glx
12:27:59  *** Kitrana has joined #openttd
12:32:12  *** Kitrana1 has joined #openttd
12:36:04  *** urdh has quit IRC
12:37:30  *** Kitrana has quit IRC
12:39:24  *** urdh has joined #openttd
12:41:06  <DorpsGek> [OpenTTD/nml] glx22 opened pull request #235: Update: increase number of OTTD_GUI sprites to 191 https://git.io/JEOC3
12:43:09  *** urdh has joined #openttd
12:43:53  <DorpsGek> [OpenTTD/nml] LordAro approved pull request #235: Update: increase number of OTTD_GUI sprites to 191 https://git.io/JEOCp
13:11:57  *** George has joined #openttd
13:13:13  <DorpsGek> [OpenTTD/nml] glx22 merged pull request #235: Update: increase number of OTTD_GUI sprites to 191 https://git.io/JEOC3
13:25:48  *** Samu has joined #openttd
14:08:57  *** tokai has joined #openttd
14:08:57  *** ChanServ sets mode: +v tokai
14:15:54  *** tokai|noir has quit IRC
14:56:27  *** George has quit IRC
15:20:15  *** Speeder has joined #openttd
15:22:43  *** Donk has quit IRC
15:43:17  <DorpsGek> [OpenTTD/OpenTTD] rubidium42 approved pull request #9502: Fix #9501: [Network] crash when more than one game-info query was pending https://git.io/JE3Uy
15:46:16  <DorpsGek> [OpenTTD/OpenTTD] rubidium42 commented on pull request #9506: Fix: [Network] show query errors in the server listing instead of error popup https://git.io/JE3Ti
15:46:56  <DorpsGek> [OpenTTD/OpenTTD] rubidium42 approved pull request #9506: Fix: [Network] show query errors in the server listing instead of error popup https://git.io/JE3Tj
15:47:24  *** HerzogDeXtEr has joined #openttd
15:49:52  <DorpsGek> [OpenTTD/OpenTTD] rubidium42 approved pull request #9508: Fix #9490: [Network] a full server couldn't be queried either https://git.io/JE3kH
15:53:53  <DorpsGek> [OpenTTD/OpenTTD] LordAro commented on pull request #9506: Fix: [Network] show query errors in the server listing instead of error popup https://git.io/JE3L3
16:13:08  *** jottyfan has quit IRC
16:31:51  *** Strom has quit IRC
16:33:33  *** Strom has joined #openttd
16:50:30  *** Tirili has joined #openttd
17:05:21  *** Wormnest has joined #openttd
17:07:31  *** jottyfan has joined #openttd
17:17:01  *** Donk has joined #openttd
17:18:54  *** frosch123 has joined #openttd
17:20:13  *** andythenorth has joined #openttd
17:33:00  *** esselfe has quit IRC
17:37:55  <DorpsGek> [OpenTTD/OpenTTD] TrueBrain merged pull request #9502: Fix #9501: [Networ

Powered by YARRSTE version: svn-trunk