Config
Log for #openttd on 2nd July 2021:
Times are UTC Toggle Colours
00:01:49  *** WormnestAndroid has quit IRC
00:02:10  *** WormnestAndroid has joined #openttd
00:14:55  *** Wuzzy has quit IRC
00:44:47  *** yeoldetoast has quit IRC
01:52:39  *** WormnestAndroid has quit IRC
01:53:22  *** WormnestAndroid has joined #openttd
01:55:50  *** tokai|noir has joined #openttd
01:55:50  *** ChanServ sets mode: +v tokai|noir
01:58:19  *** yeoldetoast has joined #openttd
02:01:17  <esselfe> Hi, I'm trying out the quebecois town names newGRF on a dedicated server but I get german names... I followed https://www.tt-forums.net/viewtopic.php?t=76674 so the content is downloaded on the server and the appropriate section is filled, but I think I must set town_name to something else than german...
02:01:51  <esselfe> I don't know what it needs
02:02:21  <esselfe> better vehicle names works fine
02:02:37  *** yeoldetoast has quit IRC
02:02:47  *** tokai has quit IRC
02:03:05  *** yeoldetoast has joined #openttd
02:06:03  <esselfe> it's not "quebecois" because I get english names
02:07:49  <esselfe> well it says 100 most populated towns, and the map is 2048x2048... maybe it's that
02:08:20  <glx> you need to put a number in the config
02:15:50  <glx> newgrf townnames start at 21
02:24:51  <glx> oh and "content select all" doesn't work since 1.11 (it's disabled)
02:28:31  <esselfe> let me try that :)
02:31:02  <esselfe> it works!
02:31:07  <esselfe> thanks alot
02:33:19  *** WormnestAndroid has quit IRC
02:43:32  *** Flygon has joined #openttd
02:44:30  *** WormnestAndroid has joined #openttd
02:50:26  *** D-HUND has joined #openttd
02:53:47  *** debdog has quit IRC
03:11:47  *** glx has quit IRC
04:29:58  *** snail_UES_ has quit IRC
05:36:56  *** tokai has joined #openttd
05:36:56  *** ChanServ sets mode: +v tokai
05:43:43  *** tokai|noir has quit IRC
06:21:58  *** sla_ro|master has joined #openttd
06:25:21  *** andythenorth has joined #openttd
06:45:00  *** whatsthetime has joined #openttd
07:48:59  *** andythenorth has quit IRC
07:50:32  *** nielsm has joined #openttd
07:51:07  <TrueBrain> ugh, nightly failed because github.com rendered a 500 on an URL :P
07:51:08  <TrueBrain> lovely
07:52:46  <LordAro> lol
08:07:48  <DorpsGek> [OpenTTD/bananas-api] TrueBrain updated pull request #94: Scheduled monthly dependency update for July https://git.io/JcWEM
08:08:27  <DorpsGek> [OpenTTD/bananas-frontend-cli] TrueBrain updated pull request #27: Scheduled monthly dependency update for July https://git.io/JcWuU
08:08:51  *** tokai|noir has joined #openttd
08:08:51  *** ChanServ sets mode: +v tokai|noir
08:09:54  <DorpsGek> [OpenTTD/bananas-frontend-web] TrueBrain updated pull request #59: Scheduled monthly dependency update for July https://git.io/JcWu4
08:10:32  <DorpsGek> [OpenTTD/bananas-api] TrueBrain updated pull request #94: Scheduled monthly dependency update for July https://git.io/JcWEM
08:11:07  <DorpsGek> [OpenTTD/bananas-server] TrueBrain updated pull request #51: Scheduled monthly dependency update for July https://git.io/JcWu1
08:11:40  <TrueBrain> it is funny .. this used to be simple .. but because of pip being pedantic about dependencies, and requests not allowing idna 3.N .. it is all of a sudden a lot more work :P
08:12:01  <TrueBrain> dependabot does this nicer, there you can just say: don't suggest this dependency update again for N time
08:12:05  <TrueBrain> pyup .. no such luck :P
08:13:59  <DorpsGek> [OpenTTD/eints] TrueBrain updated pull request #46: Scheduled monthly dependency update for July https://git.io/JcWz5
08:14:21  <DorpsGek> [OpenTTD/master-server] TrueBrain updated pull request #32: Scheduled monthly dependency update for July https://git.io/JcWg3
08:15:49  *** tokai has quit IRC
08:15:49  <DorpsGek> [OpenTTD/master-server-web] TrueBrain updated pull request #26: Scheduled monthly dependency update for July https://git.io/JcWgi
08:17:01  <DorpsGek> [OpenTTD/DorpsGek] TrueBrain updated pull request #56: Scheduled monthly dependency update for July https://git.io/JcWzf
08:17:40  <DorpsGek> [OpenTTD/bananas-frontend-cli] TrueBrain updated pull request #27: Scheduled monthly dependency update for July https://git.io/JcWuU
08:20:00  <DorpsGek> [OpenTTD/bananas-frontend-web] TrueBrain merged pull request #59: Scheduled monthly dependency update for July https://git.io/JcWu4
08:21:37  *** tokai|noir has quit IRC
08:24:28  <DorpsGek> [OpenTTD/bananas-frontend-web] TrueBrain opened pull request #60: Update: dependencies update for npm https://git.io/Jc4WJ
08:24:48  <DorpsGek> [OpenTTD/bananas-server] TrueBrain approved pull request #51: Scheduled monthly dependency update for July https://git.io/Jc4Wm
08:24:55  <DorpsGek> [OpenTTD/bananas-server] TrueBrain merged pull request #51: Scheduled monthly dependency update for July https://git.io/JcWu1
08:25:14  <DorpsGek> [OpenTTD/eints] TrueBrain approved pull request #46: Scheduled monthly dependency update for July https://git.io/Jc4WC
08:25:20  <DorpsGek> [OpenTTD/eints] TrueBrain merged pull request #46: Scheduled monthly dependency update for July https://git.io/JcWz5
08:25:23  *** andythenorth has joined #openttd
08:25:53  <DorpsGek> [OpenTTD/master-server] TrueBrain approved pull request #32: Scheduled monthly dependency update for July https://git.io/Jc4Ww
08:25:59  <DorpsGek> [OpenTTD/master-server] TrueBrain merged pull request #32: Scheduled monthly dependency update for July https://git.io/JcWg3
08:26:09  <DorpsGek> [OpenTTD/master-server-web] TrueBrain approved pull request #26: Scheduled monthly dependency update for July https://git.io/Jc4WM
08:26:13  <DorpsGek> [OpenTTD/master-server-web] TrueBrain merged pull request #26: Scheduled monthly dependency update for July https://git.io/JcWgi
08:27:03  <DorpsGek> [OpenTTD/bananas-frontend-cli] TrueBrain approved pull request #27: Scheduled monthly dependency update for July https://git.io/Jc4lv
08:27:07  <DorpsGek> [OpenTTD/bananas-frontend-cli] TrueBrain merged pull request #27: Scheduled monthly dependency update for July https://git.io/JcWuU
08:27:51  <DorpsGek> [OpenTTD/DorpsGek] TrueBrain approved pull request #56: Scheduled monthly dependency update for July https://git.io/Jc4ln
08:27:55  <DorpsGek> [OpenTTD/DorpsGek] TrueBrain merged pull request #56: Scheduled monthly dependency update for July https://git.io/JcWzf
08:28:10  <DorpsGek> [OpenTTD/bananas-frontend-web] LordAro commented on pull request #60: Update: dependencies update for npm https://git.io/Jc4l0
08:29:08  <DorpsGek> [OpenTTD/bananas-frontend-web] TrueBrain commented on pull request #60: Update: dependencies update for npm https://git.io/Jc4lP
08:29:26  <DorpsGek> [OpenTTD/bananas-api] TrueBrain approved pull request #94: Scheduled monthly dependency update for July https://git.io/Jc4l9
08:29:31  <DorpsGek> [OpenTTD/bananas-api] TrueBrain merged pull request #94: Scheduled monthly dependency update for July https://git.io/JcWEM
08:35:31  <DorpsGek> [OpenTTD/master-server-web] TrueBrain approved pull request #22: Fix #4: Unknown server error message https://git.io/Jc446
08:36:16  <andythenorth> spam!
08:36:18  <andythenorth> "lol"
08:36:38  <DorpsGek> [OpenTTD/master-server-web] TrueBrain merged pull request #22: Fix #4: Unknown server error message https://git.io/JOUGc
08:36:43  <DorpsGek> [OpenTTD/master-server-web] TrueBrain closed issue #4: Unknown server-id gives odd error https://git.io/JUn7q
08:37:18  <TrueBrain> and more spam to come .. but not till after lunch
08:37:30  <TrueBrain> I need to do some decent testing, as some bumps might break stuff .. it shouldn't, but it might
08:37:34  <andythenorth> ban those spammers!
08:50:46  <TrueBrain> LordAro: if you can take a look at #60, would be lovely :)
08:55:04  <peter1138> Boop.
08:55:23  *** andythenorth has quit IRC
08:55:37  <peter1138> Monday: Please do this, with a progress update Thursday.
08:55:39  <peter1138> Thursday: ...
08:55:46  <peter1138> Friday: Is this built and tested?
08:56:57  <TrueBrain> Sounds like a normal business week ;)
08:57:47  <DorpsGek> [OpenTTD/bananas-frontend-web] LordAro approved pull request #60: Update: dependencies update for npm https://git.io/Jc4go
08:58:50  <TrueBrain> Thank you
08:59:38  <peter1138> Now trying to sync data from an external CRM system that doesn't have many data constraints enforced. Duplicate records? Fuck Right Off.
09:07:38  *** felix has quit IRC
09:19:02  *** felix has joined #openttd
09:28:39  *** Samu has joined #openttd
09:41:07  *** WormnestAndroid has quit IRC
09:41:20  *** WormnestAndroid has joined #openttd
09:43:02  *** Wuzzy has joined #openttd
09:43:41  <Xaroth> sounds like you're having fun, peter1138
09:52:04  <DorpsGek> [OpenTTD/bananas-frontend-web] TrueBrain merged pull request #60: Update: dependencies update for npm https://git.io/Jc4WJ
10:01:26  *** andythenorth has joined #openttd
10:02:50  <peter1138> Yeah. Actually it wasn't "Is this ...", it was "Please confirm this is ..."
10:03:13  <DorpsGek> [OpenTTD/master-server] TrueBrain opened pull request #33: Several GitHub workflow related fixes https://git.io/Jc47q
10:10:26  <DorpsGek> [OpenTTD/master-server-web] TrueBrain opened pull request #27: Several GitHub workflow related fixes  https://git.io/Jc45A
10:10:39  *** andythenorth has quit IRC
10:12:07  <DorpsGek> [OpenTTD/bananas-api] TrueBrain opened pull request #95: Several GitHub workflow related fixes  https://git.io/Jc4dn
10:13:13  <DorpsGek> [OpenTTD/bananas-frontend-web] TrueBrain opened pull request #61: Several GitHub workflow related fixes  https://git.io/Jc4do
10:14:25  <DorpsGek> [OpenTTD/bananas-server] TrueBrain opened pull request #52: Several GitHub workflow related fixes https://git.io/Jc4dQ
10:22:46  <DorpsGek> [OpenTTD/eints] TrueBrain opened pull request #47: Several GitHub workflow related fixes  https://git.io/Jc4Nf
10:22:55  <TrueBrain> I think I got all repos now ..
10:23:19  <TrueBrain> ah, no, "website", but that has its own problems :P
10:24:58  <TrueBrain> lies, found one more :)
10:24:59  <DorpsGek> [OpenTTD/py-helpers] TrueBrain opened pull request #8: Several GitHub workflow related fixes  https://git.io/Jc4Nw
10:27:53  <DorpsGek> [OpenTTD/workflows] TrueBrain opened pull request #21: Several GitHub workflow related fixes https://git.io/Jc4Am
10:29:15  <DorpsGek> [OpenTTD/workflows] TrueBrain updated pull request #21: Several GitHub workflow related fixes https://git.io/Jc4Am
10:31:21  <DorpsGek> [OpenTTD/workflows] TrueBrain opened pull request #22: Update: dependencies update https://git.io/Jc4AN
10:31:24  <DorpsGek> [OpenTTD/master-server] glx22 approved pull request #33: Several GitHub workflow related fixes https://git.io/Jc4AA
10:32:08  <DorpsGek> [OpenTTD/master-server-web] glx22 approved pull request #27: Several GitHub workflow related fixes  https://git.io/Jc4xf
10:32:30  <DorpsGek> [OpenTTD/bananas-api] glx22 approved pull request #95: Several GitHub workflow related fixes  https://git.io/Jc4xk
10:32:53  <DorpsGek> [OpenTTD/bananas-frontend-web] glx22 approved pull request #61: Several GitHub workflow related fixes  https://git.io/Jc4xq
10:33:12  <DorpsGek> [OpenTTD/bananas-server] glx22 approved pull request #52: Several GitHub workflow related fixes https://git.io/Jc4xO
10:33:30  <DorpsGek> [OpenTTD/eints] glx22 approved pull request #47: Several GitHub workflow related fixes  https://git.io/Jc4xZ
10:33:33  <DorpsGek> [OpenTTD/website] TrueBrain opened pull request #218: Several GitHub workflow related fixes  https://git.io/Jc4xn
10:33:52  <DorpsGek> [OpenTTD/bananas-api] TrueBrain merged pull request #95: Several GitHub workflow related fixes  https://git.io/Jc4dn
10:34:00  <DorpsGek> [OpenTTD/py-helpers] glx22 approved pull request #8: Several GitHub workflow related fixes  https://git.io/Jc4x4
10:34:03  <DorpsGek> [OpenTTD/bananas-frontend-web] TrueBrain merged pull request #61: Several GitHub workflow related fixes  https://git.io/Jc4do
10:34:11  <DorpsGek> [OpenTTD/bananas-server] TrueBrain merged pull request #52: Several GitHub workflow related fixes https://git.io/Jc4dQ
10:34:19  <DorpsGek> [OpenTTD/eints] TrueBrain merged pull request #47: Several GitHub workflow related fixes  https://git.io/Jc4Nf
10:34:27  <DorpsGek> [OpenTTD/master-server] TrueBrain merged pull request #33: Several GitHub workflow related fixes https://git.io/Jc47q
10:34:37  <DorpsGek> [OpenTTD/master-server-web] TrueBrain merged pull request #27: Several GitHub workflow related fixes  https://git.io/Jc45A
10:34:46  <DorpsGek> [OpenTTD/py-helpers] TrueBrain merged pull request #8: Several GitHub workflow related fixes  https://git.io/Jc4Nw
10:38:09  <TrueBrain> okay .. after lunch, mass renaming master->main for those repos .. that should be the last of the cleanups
10:39:12  <DorpsGek> [OpenTTD/workflows] glx22 approved pull request #21: Several GitHub workflow related fixes https://git.io/Jc4pE
10:39:48  <DorpsGek> [OpenTTD/workflows] TrueBrain merged pull request #21: Several GitHub workflow related fixes https://git.io/Jc4Am
10:43:41  <DorpsGek> [OpenTTD/workflows] glx22 approved pull request #22: Update: dependencies update https://git.io/Jc4hW
10:44:41  <DorpsGek> [OpenTTD/workflows] TrueBrain merged pull request #22: Update: dependencies update https://git.io/Jc4AN
10:47:50  <DorpsGek> [OpenTTD/website] glx22 approved pull request #218: Several GitHub workflow related fixes  https://git.io/Jc4hd
10:48:34  <DorpsGek> [OpenTTD/website] TrueBrain merged pull request #218: Several GitHub workflow related fixes  https://git.io/Jc4xn
10:48:35  <TrueBrain> and that is a full set :D
10:48:48  <LordAro> yay
11:20:35  <Xaroth> holy spam batman.
11:22:37  <TrueBrain> and more to come, don't you worry!
11:23:35  <Samu> im now testing my AI when under a low budget
11:23:45  <Samu> no more 2 billion loan
11:23:54  <Samu> £300k for this test
11:26:28  <DorpsGek> [OpenTTD/bananas-api] TrueBrain opened pull request #96: Change: [GitHub] switch default branch to "main" https://git.io/JcBL1
11:26:58  <Samu> dang, the estimator for number of vehicles is a little off
11:27:15  <Samu> or should i say, massively off, but i'll get to it later
11:42:34  <DorpsGek> [OpenTTD/bananas-api] glx22 approved pull request #96: Change: [GitHub] switch default branch to "main" https://git.io/JcBmF
11:44:34  <DorpsGek> [OpenTTD/bananas-frontend-cli] TrueBrain opened pull request #28: Several GitHub workflow related fixes https://git.io/JcBYc
11:44:34  <TrueBrain> missed a repo .. lol
11:44:35  <TrueBrain> oops
11:44:45  *** Kitrana1 has joined #openttd
11:46:22  <DorpsGek> [OpenTTD/bananas-frontend-cli] glx22 approved pull request #28: Several GitHub workflow related fixes https://git.io/JcBYX
11:48:10  <DorpsGek> [OpenTTD/bananas-frontend-cli] TrueBrain merged pull request #28: Several GitHub workflow related fixes https://git.io/JcBYc
11:49:51  *** Kitrana has quit IRC
11:50:40  <TrueBrain> right, time to spam some more ..
11:50:47  <DorpsGek> [OpenTTD/bananas-frontend-web] TrueBrain opened pull request #62: Change: [GitHub] switch default branch to "main" https://git.io/JcBOP
11:50:50  <DorpsGek> [OpenTTD/bananas-server] TrueBrain opened pull request #53: Change: [GitHub] switch default branch to "main" https://git.io/JcBOX
11:50:53  <DorpsGek> [OpenTTD/master-server] TrueBrain opened pull request #34: Change: [GitHub] switch default branch to "main" https://git.io/JcBO1
11:50:56  <DorpsGek> [OpenTTD/master-server-web] TrueBrain opened pull request #28: Change: [GitHub] switch default branch to "main" https://git.io/JcBOM
11:50:59  <DorpsGek> [OpenTTD/py-helpers] TrueBrain opened pull request #9: Change: [GitHub] switch default branch to "main" https://git.io/JcBOD
11:51:02  <DorpsGek> [OpenTTD/workflows] TrueBrain opened pull request #23: Change: [GitHub] switch default branch to "main" https://git.io/JcBOy
11:51:05  <DorpsGek> [OpenTTD/website] TrueBrain opened pull request #219: Change: [GitHub] switch default branch to "main" https://git.io/JcBOS
11:51:08  <DorpsGek> [OpenTTD/bananas-frontend-cli] TrueBrain opened pull request #29: Change: [GitHub] switch default branch to "main" https://git.io/JcBO9
11:51:38  <TrueBrain> that should be the last PRs of the day :) Not the last spam ... :P
11:52:21  <DorpsGek> [OpenTTD/bananas-frontend-web] glx22 approved pull request #62: Change: [GitHub] switch default branch to "main" https://git.io/JcB3v
11:52:58  <DorpsGek> [OpenTTD/bananas-server] glx22 approved pull request #53: Change: [GitHub] switch default branch to "main" https://git.io/JcB3O
11:53:38  <DorpsGek> [OpenTTD/master-server-web] glx22 approved pull request #28: Change: [GitHub] switch default branch to "main" https://git.io/JcB3C
11:54:25  <DorpsGek> [OpenTTD/master-server] glx22 approved pull request #34: Change: [GitHub] switch default branch to "main" https://git.io/JcB30
11:55:00  <DorpsGek> [OpenTTD/py-helpers] glx22 approved pull request #9: Change: [GitHub] switch default branch to "main" https://git.io/JcB3K
11:55:28  <DorpsGek> [OpenTTD/workflows] glx22 approved pull request #23: Change: [GitHub] switch default branch to "main" https://git.io/JcB3i
11:57:40  <DorpsGek> [OpenTTD/website] glx22 commented on pull request #219: Change: [GitHub] switch default branch to "main" https://git.io/JcBsT
11:58:22  <DorpsGek> [OpenTTD/bananas-frontend-cli] glx22 approved pull request #29: Change: [GitHub] switch default branch to "main" https://git.io/JcBs3
11:58:44  <TrueBrain> right ... lets do this ..
12:00:04  <TrueBrain> hmm .. I tried this out before .. but now I cannot retrigger the checks for some reason
12:00:05  <TrueBrain> lol
12:00:21  <TrueBrain> that is annoying ..
12:00:48  <TrueBrain> no matter how you plan stuff ...
12:02:18  <DorpsGek> [OpenTTD/bananas-api] TrueBrain dismissed a review for pull request #96: Change: [GitHub] switch default branch to "main" https://git.io/JcBmF
12:02:19  <TrueBrain> and I guess this spams even more?
12:02:21  <DorpsGek> [OpenTTD/bananas-api] TrueBrain updated pull request #96: Change: [GitHub] switch default branch to "main" https://git.io/JcBL1
12:02:21  <TrueBrain> ugh, yes ...
12:03:13  <DorpsGek> [OpenTTD/bananas-api] TrueBrain commented on pull request #96: Change: [GitHub] switch default branch to "main" https://git.io/JcBGZ
12:03:51  <DorpsGek> [OpenTTD/bananas-frontend-web] TrueBrain dismissed a review for pull request #62: Change: [GitHub] switch default branch to "main" https://git.io/JcB3v
12:03:54  <DorpsGek> [OpenTTD/bananas-frontend-web] TrueBrain updated pull request #62: Change: [GitHub] switch default branch to "main" https://git.io/JcBOP
12:04:18  <DorpsGek> [OpenTTD/bananas-server] TrueBrain dismissed a review for pull request #53: Change: [GitHub] switch default branch to "main" https://git.io/JcB3O
12:04:21  <DorpsGek> [OpenTTD/bananas-server] TrueBrain updated pull request #53: Change: [GitHub] switch default branch to "main" https://git.io/JcBOX
12:04:34  <DorpsGek> [OpenTTD/master-server] TrueBrain dismissed a review for pull request #34: Change: [GitHub] switch default branch to "main" https://git.io/JcB30
12:04:37  <DorpsGek> [OpenTTD/master-server] TrueBrain updated pull request #34: Change: [GitHub] switch default branch to "main" https://git.io/JcBO1
12:04:53  <DorpsGek> [OpenTTD/master-server-web] TrueBrain dismissed a review for pull request #28: Change: [GitHub] switch default branch to "main" https://git.io/JcB3C
12:04:56  <DorpsGek> [OpenTTD/master-server-web] TrueBrain updated pull request #28: Change: [GitHub] switch default branch to "main" https://git.io/JcBOM
12:05:12  <DorpsGek> [OpenTTD/py-helpers] TrueBrain dismissed a review for pull request #9: Change: [GitHub] switch default branch to "main" https://git.io/JcB3K
12:05:15  <DorpsGek> [OpenTTD/py-helpers] TrueBrain updated pull request #9: Change: [GitHub] switch default branch to "main" https://git.io/JcBOD
12:05:26  <DorpsGek> [OpenTTD/bananas-api] glx22 approved pull request #96: Change: [GitHub] switch default branch to "main" https://git.io/JcBGP
12:05:29  <DorpsGek> [OpenTTD/workflows] TrueBrain dismissed a review for pull request #23: Change: [GitHub] switch default branch to "main" https://git.io/JcB3i
12:05:32  <DorpsGek> [OpenTTD/workflows] TrueBrain updated pull request #23: Change: [GitHub] switch default branch to "main" https://git.io/JcBOy
12:05:58  <DorpsGek> [OpenTTD/website] TrueBrain commented on pull request #219: Change: [GitHub] switch default branch to "main" https://git.io/JcBGN
12:06:26  <DorpsGek> [OpenTTD/website] TrueBrain updated pull request #219: Change: [GitHub] switch default branch to "main" https://git.io/JcBOS
12:06:44  <DorpsGek> [OpenTTD/bananas-frontend-cli] TrueBrain dismissed a review for pull request #29: Change: [GitHub] switch default branch to "main" https://git.io/JcBs3
12:06:47  <DorpsGek> [OpenTTD/bananas-frontend-cli] TrueBrain updated pull request #29: Change: [GitHub] switch default branch to "main" https://git.io/JcBO9
12:07:54  <TrueBrain> at least it is a chatty day ;)
12:08:14  <DorpsGek> [OpenTTD/bananas-frontend-web] glx22 approved pull request #62: Change: [GitHub] switch default branch to "main" https://git.io/JcBZM
12:08:32  <DorpsGek> [OpenTTD/bananas-server] glx22 approved pull request #53: Change: [GitHub] switch default branch to "main" https://git.io/JcBZ9
12:08:43  <DorpsGek> [OpenTTD/master-server-web] glx22 approved pull request #28: Change: [GitHub] switch default branch to "main" https://git.io/JcBZd
12:08:48  <Kitrana1> heh
12:08:55  *** Kitrana1 is now known as Kitrana
12:09:09  <DorpsGek> [OpenTTD/master-server] glx22 approved pull request #34: Change: [GitHub] switch default branch to "main" https://git.io/JcBZp
12:09:20  <DorpsGek> [OpenTTD/website] TrueBrain updated pull request #219: Change: [GitHub] switch default branch to "main" https://git.io/JcBOS
12:09:39  <DorpsGek> [OpenTTD/py-helpers] glx22 approved pull request #9: Change: [GitHub] switch default branch to "main" https://git.io/JcBn3
12:09:57  <DorpsGek> [OpenTTD/workflows] glx22 approved pull request #23: Change: [GitHub] switch default branch to "main" https://git.io/JcBnW
12:10:27  <DorpsGek> [OpenTTD/bananas-frontend-cli] glx22 approved pull request #29: Change: [GitHub] switch default branch to "main" https://git.io/JcBn2
12:11:32  <DorpsGek> [OpenTTD/website] glx22 approved pull request #219: Change: [GitHub] switch default branch to "main" https://git.io/JcBnb
12:11:54  <DorpsGek> [OpenTTD/bananas-api] TrueBrain merged pull request #96: Change: [GitHub] switch default branch to "main" https://git.io/JcBL1
12:12:04  <DorpsGek> [OpenTTD/bananas-frontend-web] TrueBrain merged pull request #62: Change: [GitHub] switch default branch to "main" https://git.io/JcBOP
12:12:15  <DorpsGek> [OpenTTD/bananas-server] TrueBrain merged pull request #53: Change: [GitHub] switch default branch to "main" https://git.io/JcBOX
12:12:34  <DorpsGek> [OpenTTD/master-server] TrueBrain merged pull request #34: Change: [GitHub] switch default branch to "main" https://git.io/JcBO1
12:12:44  <DorpsGek> [OpenTTD/master-server-web] TrueBrain merged pull request #28: Change: [GitHub] switch default branch to "main" https://git.io/JcBOM
12:12:56  <DorpsGek> [OpenTTD/py-helpers] TrueBrain merged pull request #9: Change: [GitHub] switch default branch to "main" https://git.io/JcBOD
12:13:06  <DorpsGek> [OpenTTD/workflows] TrueBrain merged pull request #23: Change: [GitHub] switch default branch to "main" https://git.io/JcBOy
12:13:23  <DorpsGek> [OpenTTD/bananas-frontend-cli] TrueBrain merged pull request #29: Change: [GitHub] switch default branch to "main" https://git.io/JcBO9
12:15:38  <DorpsGek> [OpenTTD/website] TrueBrain merged pull request #219: Change: [GitHub] switch default branch to "main" https://git.io/JcBOS
12:17:37  *** glx has joined #openttd
12:17:37  *** ChanServ sets mode: +v glx
12:18:21  <TrueBrain> tnx a lot glx :) Much appreciated!
12:18:42  <glx> android app is nice for that :)
12:19:13  <TrueBrain> now lets hope I didn't forget anything .. but things will break sooner or later :P
12:20:54  <glx> at least now all repos should use common shared stuff
12:21:06  <TrueBrain> yeah .. they should now all be in sync again :D
12:21:09  <TrueBrain> which is really nice
12:21:09  <Rubidium> I dub ye sirs spamalot ;)
12:21:32  <TrueBrain> too bad @say doesn't work anymore, but I am sure DorpsGek greatly appreciates that honor :D
12:22:01  <TrueBrain> right .. 1 more repo to switch ... BaNaNaS itself. But that needs a code-change
12:22:13  <TrueBrain> first, lets send the rest to production
12:22:33  <glx> there's also the big one to switch ;)
12:23:07  <TrueBrain> yeah, and eints
12:23:36  <TrueBrain> I suggest to do the big one some date later, just to give this a moment to rest
12:23:46  <TrueBrain> eints .. is .. it needs more attention, lets keep it at that :P
12:25:19  <Rubidium> can you make DorpsGek reserve #10000 for that? As in, when #9999 gets made it quickly makes #10000 for making that change?
12:25:34  <DorpsGek> [OpenTTD/DorpsGek] TrueBrain created new tag: 2.1.1 https://git.io/JcB4U
12:25:42  <TrueBrain> Rubidium: haha, sure :P
12:26:30  <TrueBrain> 600 more issues/PRs .. oof
12:27:32  <Rubidium> GC, STUN, TURN could all be separate PRs ;)
12:28:03  <TrueBrain> haha :D
12:28:22  <TrueBrain> I honestly did consider splitting it up over multiple PRs .. but in the end voted against it :P
12:28:45  *** DorpsGek has quit IRC
12:28:49  <TrueBrain> bye DorpsGek
12:28:51  <TrueBrain> be back soon
12:28:54  *** DorpsGek has joined #openttd
12:28:54  *** ChanServ sets mode: +o DorpsGek
12:29:19  <TrueBrain> wb DorpsGek
12:29:49  <DorpsGek> [OpenTTD/master-server] TrueBrain created new tag: 1.0.14 https://git.io/JcBBr
12:29:56  <DorpsGek> [OpenTTD/OpenTTD] rubidium42 approved pull request #9298: Change: move sensitive information to secrets.cfg and private information to private.cfg https://git.io/JcBBP
12:30:05  <TrueBrain> \o/
12:30:18  <DorpsGek> [OpenTTD/OpenTTD] TrueBrain merged pull request #9298: Change: move sensitive information to secrets.cfg and private information to private.cfg https://git.io/JGLne
12:30:23  <glx> oh of course there's the tag spam wave now :)
12:30:34  <DorpsGek> [OpenTTD/OpenTTD] TrueBrain updated pull request #9017: Feature: play easier together with friends from behind home routers https://git.io/JOk7C
12:30:35  <Rubidium> it's not the SQLite one though
12:31:07  <TrueBrain> Rubidium: awwwhhh, no SQLite? :P
12:31:30  <Rubidium> though practically SQLite is stupid. Make it a git repository, like the database for bananas
12:32:43  <TrueBrain> what are we talking about now, sorry? :P
12:32:52  <DorpsGek> [OpenTTD/master-server-web] TrueBrain created new tag: 1.0.11 https://git.io/JcBR8
12:33:11  <TrueBrain> you mean to say a git repo per savegame? :P
12:33:21  <glx> hehe tagging is slower than all the PRs ;)
12:33:42  <TrueBrain> glx: yup .. needs greater care from my side :)
12:33:53  <Rubidium> TrueBrain: yes! Wouldn't that be wonderful(ly bad)
12:34:03  <TrueBrain> Rubidium: let me get right on that :D
12:34:19  <glx> let's use yaml :)
12:34:27  <TrueBrain> no no, TOML
12:34:35  <TrueBrain> if you want to hurt people, do it the right way
12:34:56  <DorpsGek> [OpenTTD/eints] TrueBrain created new tag: openttd-github-1.1.5 https://git.io/JcBRi
12:38:59  <TrueBrain> Rubidium: btw, if it would make #9017 easier, I can indeed split it up over several PRs. They just have to be accepted close to each other, as the Game Coordinator assumes everything to be there :D
12:41:57  <DorpsGek> [OpenTTD/nml] glx22 commented on pull request #193: Update: changelog for 0.6.0 https://git.io/JcBEm
12:42:49  <DorpsGek> [OpenTTD/bananas-server] TrueBrain opened pull request #54: Feature: a setting to set the branch of the GitHub index repository https://git.io/JcBEu
12:44:06  <DorpsGek> [OpenTTD/bananas-server] TrueBrain updated pull request #54: Feature: a setting to set the branch of the GitHub index repository https://git.io/JcBEu
12:47:55  <DorpsGek> [OpenTTD/bananas-api] TrueBrain opened pull request #97: Feature: a setting to set the branch of the GitHub index repository https://git.io/JcBu5
12:47:57  <Rubidium> TrueBrain: that potentially makes it pointless to split it up, as it makes other things much more painful. It's just a lot to go through, as now I've only skimmed through it mostly because so much looked familiar as I have seen it several times already
12:48:00  <TrueBrain> okay, those two commits should allow me to change BaNaNaS too ..
12:48:19  <TrueBrain> Rubidium: yeah .. I have the same issue. Going blind on the code is a thing at a certain point :)
12:48:57  <TrueBrain> with more work I could make the game-coordinator accept in-between PRs .. but that is not completely trivial
12:49:15  <TrueBrain> (I personally don't like such huge PRs, also the reason I splitted up the saveload PRs so much)
12:52:39  <TrueBrain> would make it into 8 PRs, and a few bumps of the game-coordinator version field
12:53:00  <TrueBrain> I kinda like that plan .. as I know the end-goal is reachable now, as 9017 clearly shows that :P
12:53:30  <TrueBrain> will see what I can do there :)
12:55:16  <peter1138> Oops. White chocolate magnum.
12:55:27  <TrueBrain> how did that happen?! :P
12:56:58  <TrueBrain> Rubidium: the days of doing things NoAI-style are a bit over in those regards ;) Hihi :D
12:57:06  <TrueBrain> one massive branch that was ... :P
12:59:47  <TrueBrain> possibly if I go down that route, I also make the game-coordinator accept older clients, which would remove the need for master-server completely
12:59:52  <TrueBrain> initially I planned to leave both running
13:00:09  <TrueBrain> but .. maintenance-wise that is yet-another-repo to maintain
13:00:38  <TrueBrain> now if I would just get in GitHub CoPilot, that would be a perfect piece of work to test that with :P
13:00:45  <TrueBrain> +beta
13:00:51  <TrueBrain> get in the beta ..
13:00:52  <TrueBrain> typing is hard :D
13:04:09  <glx> oh we did it for cmake
13:04:48  <glx> but yeah big PRs are hard to review
13:04:50  <TrueBrain> only took, what, 12 months? :P
13:05:02  <TrueBrain> and in the end I believe it was YOLO accepted? :D
13:05:08  <glx> yup
13:05:20  <glx> followed by fix PRs
13:05:39  <TrueBrain> those will come for the STUN stuff anyway too :P
13:05:45  <TrueBrain> no matter how you slice it :D
13:05:47  <glx> because nobody test things before it's live
13:06:04  <TrueBrain> well, "nobody" .. just not enough bodies :D
13:06:24  <TrueBrain> same with OpenGL for example :P
13:06:40  <TrueBrain> I do respect game studios a lot more because of this btw
13:06:48  <TrueBrain> we can pitch about multiple patches the first few days
13:06:49  <glx> I noticed "content" console command doesn't rescan at the end of download
13:06:59  <TrueBrain> but holy crap, nearly impossible to find shit players run into :D
13:07:12  <TrueBrain> glx: ha, that somewhat wouldn't surprise me
13:07:26  <TrueBrain> that whole command seems to be implemented in a rush
13:07:46  <TrueBrain> I couldn't figure out how to work with it :P
13:08:28  <TrueBrain> (except that it allows "download EVERYTHING" ofc :P)
13:08:33  <TrueBrain> allows = allowed
13:08:37  <glx> I see how it can be done, but the issue is it can stall a dedicated server during the scan
13:08:43  <glx> I think
13:09:12  <TrueBrain> the GUI rescan stalls too, doesn't it?
13:09:51  <glx> yeah, but I can't remember if it's doable in game
13:11:05  *** snail_UES_ has joined #openttd
13:13:02  <glx> anyway common usage for a dedicated server is, I think, start server, execute content commands, close, edit cfg
13:25:24  <Samu> wow, NoNoCAB ships are always 100% usage
13:25:38  <Samu> unless they're industry based
13:26:08  <Samu> im comparing my AI performance to NoNoCAB and ShipAI
13:27:11  <Samu> it's not doing too well
13:27:34  <Samu> i still fail at estimating the correct number of ships for a route
13:31:42  <Samu> seems that I have to do this part of the code different than if they were road vehicles
13:32:38  <glx> you can remove congestion detection, ships are not blocking
13:32:57  <Samu> NoNoCAB uses full load
13:33:15  <glx> yes full load is better for ships
13:33:18  <Samu> it is impressive how it can manage the cargo waiting at both sides of the station
13:34:45  <Samu> when i do full load, i get my ai just massing ships, because one station has cargo waiting, and end up with many ships waiting for full load at the other
13:34:59  <Samu> i can't balance
13:38:11  <Samu> ShipAI seems to have it easier
13:38:47  <Samu> only needs to manage full load at one dock, since it's doing industries
13:40:39  <Samu> i know that my ai later on gets strong profits, proof of yesterdays 5000 ships
13:40:46  <Samu> but it starts really poor
13:46:57  <TrueBrain> so who can I tempt to review https://github.com/OpenTTD/bananas-server/pull/54 and https://github.com/OpenTTD/bananas-api/pull/97 :) It is really simple what it does, really :)
13:47:06  <TrueBrain> they are nearly identical even!
13:51:09  *** gelignite has joined #openttd
13:52:02  <DorpsGek> [OpenTTD/bananas-server] glx22 approved pull request #54: Feature: a setting to set the branch of the GitHub index repository https://git.io/JcB5E
13:52:58  <DorpsGek> [OpenTTD/bananas-api] glx22 approved pull request #97: Feature: a setting to set the branch of the GitHub index repository https://git.io/JcB5b
13:53:02  <TrueBrain> tnx!! :D
13:53:17  <DorpsGek> [OpenTTD/bananas-api] TrueBrain merged pull request #97: Feature: a setting to set the branch of the GitHub index repository https://git.io/JcBu5
13:53:22  <DorpsGek> [OpenTTD/bananas-server] TrueBrain merged pull request #54: Feature: a setting to set the branch of the GitHub index repository https://git.io/JcBEu
13:53:32  <TrueBrain> lets see if it actually works on staging now :D
13:53:51  <esselfe> Just a suggestion for the devs: it would be nice if the dedicated server code also tells which company number a client joined. Like "*** esselfe has joined the game (Client #3, company #1)"
13:54:10  <TrueBrain> esselfe: feel free to make a Pull Request for that! :D
13:54:19  <esselfe> I can try
13:54:27  <esselfe> I only know C
13:54:34  <TrueBrain> at worst you fail, but you for sure would have learnt something along the way :)
13:58:58  <TrueBrain> ha, master -> main works on staging :D
13:59:05  <TrueBrain> lets roll this shit out to production, shall we? :D
13:59:44  <glx> esselfe: isn't the join company message follow the join game message ?
14:00:34  <DorpsGek> [OpenTTD/bananas-server] TrueBrain created new tag: 1.2.0 https://git.io/JcBNV
14:00:37  <DorpsGek> [OpenTTD/bananas-api] TrueBrain created new tag: 1.2.0 https://git.io/JcBNw
14:01:30  <DorpsGek> [OpenTTD/bananas-frontend-web] TrueBrain created new tag: 1.0.11 https://git.io/JcBNx
14:09:01  <TrueBrain> right, runs on production too :D
14:09:10  <TrueBrain> sweet, that should be enough noise for today :)
14:09:24  <TrueBrain> tnx again glx, it is much appreciated :)
14:09:35  <glx> was easy reviews :)
14:09:48  <TrueBrain> without them I couldn't have done it in a day ;)
14:10:26  <TrueBrain> I am still happy with 60Hz (and even 144Hz for me personally) .. SO SMOOTH! :D
14:10:35  <peter1138> https://www.pcgamesn.com/nexus-mods-collections
14:10:48  <peter1138> "Nexus Mods is getting collections, but creators won’t be able to delete their mods" < Who would create such a system...!?
14:10:54  <TrueBrain> peter1138: yeah, I was talking about that yesterday
14:11:00  <glx> popcorn :)
14:11:01  <TrueBrain> they went total apeshit over it
14:11:04  <peter1138> Well fu.. sorry :(
14:11:09  <esselfe> glx: no the join company message only appears if the player changes company after
14:12:16  <TrueBrain> peter1138: I tried to understand why they went this nuclear (mod-makers) over not being able to delete their content ..
14:12:21  <TrueBrain> haven't really found an answer yet
14:12:55  <peter1138> Lack of control.
14:13:04  <TrueBrain> once you put something on the internet ........
14:14:42  <TrueBrain> guess it is by luck that we started off with that as a rule, and didn't have to change it :)
14:17:02  <TrueBrain> I think they stole the idea from us btw :P Just saying :D
14:19:04  <TrueBrain> guess this is the perfect timing to push my "preset" PR, right? :D
14:24:33  <peter1138> :)
14:47:21  <Samu> eish, the canal pathfinder is slow as hell
14:47:28  <Samu> competing at 10k ops, medium speed
14:49:40  <Samu> I must applaud ShipAI
14:49:53  <Samu> uses the least memory, and is the fastest pathfinder
14:52:13  <Samu> NoNoCAB pathfinder also seems to be slow, but i dont know how it works for sure
14:52:34  <Samu> its just that it has great vehicle management
14:54:23  <Samu> tldr: my pathfinder sucks, confirmed
14:54:27  *** Progman has joined #openttd
15:08:44  <DorpsGek> [OpenTTD/nml] FLHerne commented on issue #219: Add AIR_DRAG_DEFAULT and AIR_DRAG_NONE constants https://git.io/JGozA
15:17:48  *** felix has quit IRC
15:46:31  *** WormnestAndroid has quit IRC
15:47:57  *** WormnestAndroid has joined #openttd
15:56:34  *** sla_ro|master has quit IRC
16:17:26  *** tokai has joined #openttd
16:17:27  *** ChanServ sets mode: +v tokai
16:19:58  *** andythenorth has joined #openttd
16:29:10  *** HerzogDeXtEr has joined #openttd
16:34:06  <Samu> this 'pathfinder.cost.estimate_multiplier' really does wonders
16:34:51  <Samu> "Every estimate is multiplied by this value. Use 1 for a 'perfect' route, higher values for faster pathfinding."
16:36:26  <esselfe> no I can't find where to insert code nor have any idea on how to implement this, sorry
16:44:20  *** andythenorth has quit IRC
16:52:08  <Samu> Yexo, are you around?
16:54:32  *** andythenorth has joined #openttd
17:05:54  *** yeoldetoast has quit IRC
17:12:09  *** WormnestAndroid has quit IRC
17:17:51  *** WormnestAndroid has joined #openttd
17:54:58  *** Flygon has quit IRC
17:56:29  *** felix has joined #openttd
18:27:21  *** andythenorth has quit IRC
18:41:48  *** andythenorth has joined #openttd
18:51:05  <DorpsGek> [OpenTTD/OpenTTD] DorpsGek pushed 1 commits to master https://git.io/Jc0U0
18:51:06  <DorpsGek>   - Update: Translations from eints (by translators)
19:09:17  *** andythenorth has quit IRC
19:10:09  <TrueBrain> Rubidium: I am thinking about splitting up #9017 like this: https://gist.github.com/TrueBrain/75c016cdb90efa18cb79f3f6bae9af25
19:16:11  *** WormnestAndroid has quit IRC
19:20:23  <Rubidium> I would add an empty string/value for invite code/game script in step 2 when writing and ignore it when reading. That way you can prevent the later NetworkInfo bumps while remaining compatible. I know that would technically mean adding pointless things for that PR, though it won't really hurt that PR either. After step 2, you could do steps 3, 4 and 6 in parallel
19:21:00  <TrueBrain> sure, but bumping NetworkInfo isn't that much work honestly :D But I will keep it in mind, yeah
19:21:25  <Rubidium> mostly as the NetworkInfo is something that is used by others I'd try to keep the number of bumps low, for the GC protocol there is no concern for others
19:21:57  <TrueBrain> especially as it is used by others, maybe empty fields can be more confusing than several bumps in a short period of time?
19:25:02  <Rubidium> well, empty would mean "no invite code" (which would be right) and "no game script" I guess (which could be wrong, but it's not as important as it worked without that information for a decade), but only for a few versions.
19:26:32  <TrueBrain> not really a fan, personally. Also for arguments like: what if I get really busy for the next 6 months and I never finish invite codes, etc :) But okay .. also bikeshedding :P
19:26:57  <TrueBrain> step 1 takes the most amount of time anyway, so lets do that first
19:27:01  <TrueBrain> and see what happens from there :D
19:27:29  <Rubidium> with having 2 "intermediate" versions you effectively complicate the logic as someone might want to implement all (recent) versions and have trouble finding something with V5 and V6. But then, that's also not really my problem either ;)
19:27:50  <Rubidium> so it's fine if you want to do it differently
19:28:28  <TrueBrain> we could also just stop doing this way of NetworkInfo
19:28:33  <TrueBrain> and use a more descriptive format
19:28:38  <TrueBrain> removing the whole issue of versions :P
19:28:54  <TrueBrain> I would like to say: JSON, but I am afraid of the hurt that will give me :D
19:29:11  <TrueBrain> the packet is a bit weird as in that it grows on the top
19:29:12  <TrueBrain> instead of the bottom
19:29:50  <TrueBrain> so I am wondering what if we change, with going to GC, the format of that packet to something new, that is a bit more friendly to addition
19:36:29  <Rubidium> TOML?
19:36:36  <TrueBrain> :D You know it ;)
19:36:55  <TrueBrain> maybe put it in an SQLite db and transfer that?
19:37:00  <Rubidium> TB's own markup language, right?
19:37:14  <glx> no TOML is a real thing
19:37:36  <TrueBrain> doesn't mean it cannot stand for TB's Own Markup Language :P
19:38:09  <TrueBrain> but in all seriousness, there are so many things wrong with TOML
19:38:11  <TrueBrain> but most of all
19:38:15  <TrueBrain> the M doesn't stand for Markup
19:38:25  <glx> minimum
19:38:31  <TrueBrain> Minimal, even
19:38:34  <glx> obviously ;)
19:39:04  <TrueBrain> I just cannot believe Python eco-system picked it as language to use
19:39:12  <TrueBrain> the whole worlds switched to YAML, but they went with TOML
19:39:20  <TrueBrain> and sure, choices are good .. but TOML?
19:39:39  <glx> and nobody uses the TOML part and keep the old way
19:39:43  <TrueBrain> it is like .ini but worse, basically
19:40:22  <glx> ini with sub groups, silly
19:40:53  <glx> haha and indentation is allowed but not required
19:40:54  <TrueBrain> not sure if it is a superset of ini .. might be interesting
19:41:02  <nielsm> https://msgpack.org/
19:41:14  <TrueBrain> nielsm: haha, there are always worse way to do things, yes :P
19:41:25  <TrueBrain> (when talking about human-readable languages, that is)
19:41:27  <Rubidium> "TOML is backward-incompatible with INI", so I guess it's not a real superset
19:41:39  <glx> https://en.wikipedia.org/wiki/TOML <-- the example without indentation would be a pain to read
19:43:00  <glx> at least we could swith openttd.cfg to TOML ;)
19:43:30  <TrueBrain> yeah ...... lets not plz :P
19:43:51  <Rubidium> practically an ini format would be fine for the network info too. The major caveat is that it adds a lot of overhead, especially when sending it to the clients. So you probably have to consider running it through gzip or so... which makes things probably a real PITA
19:44:01  <nielsm> regardless the format, it should be something with a library that gives a sensible way of working with it in C++
19:44:18  <TrueBrain> Rubidium: yeah, you are absolutely right
19:44:21  <TrueBrain> so: not for this set of PRs :)
19:44:31  <TrueBrain> scoping is hard :P
19:44:42  <TrueBrain> nielsm: yes, 1000x yes to that :)
19:44:53  <glx> anyway older versions of openttd will still require new version to send data in old format
19:45:13  <TrueBrain> glx: nope, as we switch from UDP to TCP :)
19:45:23  <TrueBrain> it is a clean moment to not have to do backwards compatibility :D
19:45:57  <glx> ha yeah ideal time to rethink the protocol
19:46:07  *** sla_ro|master has joined #openttd
19:46:09  <TrueBrain> that is why it was on my mind, as it allows a clean moment to do this .. but ..
19:46:11  <TrueBrain> in the balance of things
19:46:19  <TrueBrain> NetworkInfo didn't change in how many years
19:47:38  <TrueBrain> and we already slipped code in 1.11.2 to use NetworkInfo, so fine :D
19:47:45  <TrueBrain> can't let that trick go to waste :P
19:49:41  <Rubidium> *if* we were to no remove fields, then adding them to the end would make it easier to make it somewhat forward compatible
19:50:03  <TrueBrain> currently removing fields already means sending an empty one :P
19:50:35  <TrueBrain> (must be sure that that 0.4 client can still query a 1.11 server after all :P)
19:53:09  <glx> maybe we can number the fields, and add an extra byte before the field data to tell which field it is (currently there are 16 known fields)
19:53:30  <Rubidium> well, it can't... though a 1.11 client can query a 0.4 server ;)
19:53:31  <glx> and one is unused
19:54:00  <TrueBrain> Rubidium: ha, fair .. because it is written the wrong way around, funny :)
19:54:50  <TrueBrain> glx: personally, I would rather use names as keys, and figure out a smart way to send the listing to the client without a huge overhead
19:56:10  <glx> yeah main issue with my idea is what to skip for unknown id
19:56:24  <TrueBrain> basically, we need a similar solution as #9322 does for savegames :)
19:57:43  *** iSoSyS has joined #openttd
19:58:43  <Rubidium> that might not be such a bad idea
19:59:03  <TrueBrain> the code is not generic enough to do that easily, sadly enough :P
19:59:21  <Rubidium> fill the NetworkGameInfo structure and then run the savegame code over it ;)
19:59:30  <TrueBrain> ha, yeah, that would work :D
19:59:33  <Rubidium> and instead of writing to a file, write it to a packet
20:00:15  *** sla_ro|master has quit IRC
20:00:16  *** iSoSyS has quit IRC
20:00:17  <TrueBrain> come to think of it, it is kinda what the master-server does .. as in: it reads the binary packet, assigns keys to the fields, and after that just knows it is a key->value list, not caring about the keys (at all)
20:01:00  <Rubidium> solves the duplication of keys from the GC to the client too
20:01:19  <TrueBrain> it would make the listing even easier, as that could be send as a huge table
20:01:36  <TrueBrain> well, I am an idiot, you mean the same
20:01:59  <TrueBrain> I am really tempted to try this now :P But okay, lets first get #9322 approved ;)
20:02:37  <Rubidium> and... your savegame reader can probably be trivially changed to convert the binary format to json
20:03:00  <TrueBrain> yeah, it just has a function where you give it a header + rows and it does its thing
20:03:28  <TrueBrain> it does mean SaveLoad field-types will be part of the network protocol too
20:03:31  <TrueBrain> that can be interesting :P
20:05:00  <TrueBrain> can we also make the TCP reader streaming?
20:05:28  <TrueBrain> so it can add servers to the list visually already while still receiving the listing
20:05:53  <Rubidium> that would get you a few steps closer to PR#9999
20:06:05  <TrueBrain> awh, aren't you going to do that? :P
20:08:00  <TrueBrain> either way, it is an interesting idea to use SaveLoad code for that; it reuses existing code, it makes the packet forward compatible, and it should be relative easy to use on all sides of the line
20:11:35  *** andythenorth has joined #openttd
20:17:09  * Rubidium ponders going into hiding for a while ;)
20:19:23  <TrueBrain> To build this you mean right? :)
20:20:19  <DorpsGek> [OpenTTD/OpenTTD] rubidium42 approved pull request #9322: Feature: store table headers for each chunk in savegame https://git.io/Jc0nH
20:20:23  *** snail_UES_ is now known as Guest1298
20:20:24  *** snail_UES_ has joined #openttd
20:20:32  <andythenorth> what have we done now?
20:20:39  <TrueBrain> Do I dare .. :p
20:22:02  <DorpsGek> [OpenTTD/OpenTTD] TrueBrain merged pull request #9322: Feature: store table headers for each chunk in savegame https://git.io/JGlmd
20:23:46  *** Guest1298 has quit IRC
20:53:19  <Samu> my ai builds really weird canals with a estimate_multiplier of 10
20:56:08  <Samu> lol https://i.imgur.com/moHQB76.png
20:56:29  <glx> TrueBrain: now I have conflicts to fix :)
20:57:09  <Samu> and I can't beat ShipAI
20:57:12  <Samu> :9
20:58:44  *** sla_ro|master has joined #openttd
21:03:10  <TrueBrain> glx: and PR it after fixing those conflicts ;)
21:10:30  <glx> ok first commit conflicts seems ok (intro game loads)
21:13:00  <glx> now the 32 conflicting files commit :)
21:16:31  <glx> and of course they will conflict again in the next commit
21:22:15  *** snail_UES_ is now known as Guest1300
21:22:15  *** snail_UES_ has joined #openttd
21:24:47  *** WormnestAndroid has joined #openttd
21:25:08  *** Samu has quit IRC
21:32:49  *** WormnestAndroid has quit IRC
21:38:53  *** nielsm has quit IRC
21:39:38  *** WormnestAndroid has joined #openttd
21:48:52  *** sla_ro|master has quit IRC
22:15:05  *** andythenorth has quit IRC
22:27:04  *** gelignite has quit IRC
22:33:46  <glx> TrueBrain: I think I spotted an error https://github.com/OpenTTD/OpenTTD/blob/1d9912134df71a0adee5de92b83b9bb361f1f151/src/saveload/goal_sl.cpp#L45 should use 'slt'
22:37:39  *** WormnestAndroid has quit IRC
22:39:30  *** WormnestAndroid has joined #openttd
22:50:28  <TrueBrain> Yup. Can you make a PR? :D
22:50:59  <glx> I'm in the middle of conflicts
22:51:20  <TrueBrain> I will take care of it tomorrow, no worries :)
22:52:19  *** WormnestAndroid has quit IRC
23:10:50  *** Westie has quit IRC
23:12:05  *** Westie has joined #openttd
23:46:01  *** Progman has quit IRC

Powered by YARRSTE version: svn-trunk