Config
Log for #openttd on 9th July 2021:
Times are UTC Toggle Colours
00:03:34  *** WormnestAndroid has joined #openttd
00:18:13  *** Gustavo6046 has quit IRC
00:18:14  *** Gustavo6046 has joined #openttd
00:28:27  *** WormnestAndroid has quit IRC
00:35:33  *** WormnestAndroid has joined #openttd
00:43:33  *** WormnestAndroid has quit IRC
00:46:39  *** WormnestAndroid has joined #openttd
00:50:48  *** snail_UES_ has quit IRC
00:54:59  *** yeoldetoast has joined #openttd
00:59:55  *** iSoSyS has joined #openttd
01:03:30  *** iSoSyS has quit IRC
01:14:14  *** WormnestAndroid has quit IRC
01:21:01  *** WormnestAndroid has joined #openttd
01:53:59  *** WormnestAndroid has quit IRC
01:57:21  *** WormnestAndroid has joined #openttd
02:41:36  *** D-HUND has joined #openttd
02:44:58  *** debdog has quit IRC
02:46:32  *** tokai has joined #openttd
02:46:32  *** ChanServ sets mode: +v tokai
02:53:15  *** tokai|noir has quit IRC
02:55:24  *** glx has quit IRC
03:03:41  *** Flygon has joined #openttd
05:35:42  *** yeoldetoast has quit IRC
05:43:51  *** andythenorth has joined #openttd
05:48:01  *** andythenorth has quit IRC
06:21:53  *** yeoldetoast has joined #openttd
07:49:29  *** andythenorth has joined #openttd
08:45:19  *** Samu has joined #openttd
08:47:08  <peter1138> If my method declaration reaches column 298, it's time to split into multiple lines, right? :D
08:48:16  <LordAro> yes
08:49:18  <andythenorth> how big is your monitor :P
08:51:33  <peter1138> Alternatively, refactor so they are class variables instead of method parameters.
08:51:37  <peter1138> andythenorth, 2560x1440 :)
08:51:43  <peter1138> I still have to scroll.
08:52:46  *** Samu has quit IRC
08:54:24  *** Samu has joined #openttd
09:07:46  <peter1138> Okay, AsyncLazy<T> is very... well, lazy.
09:08:47  <peter1138> I can set that up in the constructor instead, and forgot about passing tons of parameters.
09:13:22  <peter1138> Probably some pedantic logic pattern rule that says it should never be done.
09:46:14  *** Strom has quit IRC
09:47:49  *** Strom has joined #openttd
09:54:17  *** iSoSyS has joined #openttd
09:56:12  *** iSoSyS has quit IRC
10:21:09  *** EER has quit IRC
10:46:06  <Samu> it's that time again
10:46:47  <Samu> retest 4096x4096, goal 5k rvs, 5k ships, 5k aircraft, with monthly autosaves, hoping for no "too long to save" crash
10:49:53  <Samu> estimating 5 hours to complete
10:50:08  <Samu> or crash again
10:53:14  <Samu> I wish AILists could be saved
10:53:31  <Samu> I don't see why they can't
10:53:40  <Samu> their data is just indexes and value
10:54:38  <Samu> oh, the sorter
10:55:18  <Samu> sort by x, ascending, descending
10:56:17  <Samu> I wouldn't have to convert lists to table if that was possible
10:56:40  <peter1138> What are you even testing?
10:57:04  <Samu> i changed some AILists into Tables
10:57:19  <Samu> tables can be saved directly without conversion
10:57:31  <Samu> takes less ops to save
10:57:32  *** outtatime has joined #openttd
10:57:39  <Samu> now i'm testing an extreme scenario
10:57:42  <Samu> with autosave
10:59:02  *** bro has quit IRC
11:07:37  <Samu> this code here https://github.com/SamuXarick/LuDiAI-AfterFix/blob/master/TownManager.nut#L345-L362
11:07:55  <Samu> is now just this: return [m_nearCityPairArray, m_usedCitiesPassTable, m_usedCitiesMailTable];
11:07:59  <Samu> a single line
11:09:51  *** blathijs has quit IRC
11:45:55  <DorpsGek> [OpenTTD/nml] 2TallTyler commented on pull request #193: Update: changelog for 0.6.0 https://git.io/JcFim
12:01:20  *** virtualrandomnumber has joined #openttd
12:01:34  *** virtualrandomnumber has quit IRC
12:08:07  *** WormnestAndroid has quit IRC
12:10:22  *** WormnestAndroid has joined #openttd
12:16:32  *** blathijs has joined #openttd
12:32:24  *** snail_UES_ has joined #openttd
12:58:31  *** glx has joined #openttd
12:58:31  *** ChanServ sets mode: +v glx
13:02:44  <Samu> I'm looking at script_instance.cpp, wondering if I can make code that would save AILists, then load it
13:03:24  <glx> not easy I think
13:03:35  <glx> AIList is not an internal squirrel type
13:04:12  *** tokai|noir has joined #openttd
13:04:12  *** ChanServ sets mode: +v tokai|noir
13:04:20  <Samu> gtw glx, *  - integer: a binary representation of the integer (int32).
13:04:25  <Samu> it's int64 now
13:04:41  <glx> oups
13:07:11  <DorpsGek> [OpenTTD/game-coordinator] glx22 commented on pull request #5: Add: small tweaks so the Game Coordinator understands Master Server data https://git.io/JcFFH
13:11:10  *** tokai has quit IRC
13:11:36  <Samu> OT_INSTANCE
13:11:42  <Samu> so, that's what an AIList is
13:11:52  <glx> yes and that can be anything
13:12:12  <glx> it means it's an external object
13:12:39  *** andythenorth has quit IRC
13:12:55  <Samu> there must be a way.
13:32:36  <Samu> oh rats, too complex for me
13:36:37  *** nielsm has joined #openttd
13:53:25  <Samu> arf... seems so easy in theory
13:54:15  <Samu> sorter_type (0 or 1), sort_ascending (false or true)
13:54:29  <Samu> then just the actual stuff in the list
13:54:38  <Samu> index and value
13:55:41  <Samu> it would save as a table
13:57:21  <Samu> table = { sorter_type, sort_ascending, { { index, value }, { index, value }, ... } }
13:58:06  <Samu> on load it would reconstruct the AIList from the saved table
13:58:50  <glx> needs a lot more that that
14:45:12  *** roadt__ has joined #openttd
14:46:19  *** tokai has joined #openttd
14:46:19  *** ChanServ sets mode: +v tokai
14:47:16  <Samu> wow, youtube ads are scam ads
14:47:40  <Samu> "wanna earn 00 a day?"
14:52:02  *** roadt_ has quit IRC
14:53:01  *** snail_UES_ has quit IRC
14:53:15  *** tokai|noir has quit IRC
14:56:35  *** tokai has quit IRC
14:57:38  <Samu> with yesterday's code: https://i.imgur.com/82uynLz.png, with today's code: https://i.imgur.com/czpzNaR.png
14:57:49  <Samu> same date
14:59:10  <Samu> 100k ops became 34k ops
14:59:22  <Samu> awesome improvement to the save routines
15:00:29  <Samu> I feel confident it will reach 5000 ships without "too long to save" now
15:22:14  <nielsm> remember that the best save routine is the one that doesn't need to do anything, because all the data are already in the correct format and ready to write as-is
15:23:08  <nielsm> (the problem is just that you then need to design the entire AI or GS around working with data in a format that's suitable for direct save/load)
15:29:56  <Samu> https://github.com/SamuXarick/LuDiAI-AfterFix/blob/907072a46ea20c47cb59ad66d9121ae1254662b7/RouteManager.nut#L107-L109
15:30:11  <Samu> can't change this one
15:30:32  <Samu> without rewritting tons
16:07:44  *** Progman has joined #openttd
16:09:07  *** andythenorth has joined #openttd
16:10:37  *** andythenorth has quit IRC
16:11:28  *** andythenorth has joined #openttd
16:16:03  <Samu> even with this amount of vehicles, I can't mass founding towns
16:16:13  <Samu> huge profits but nah, not huge enough
16:19:30  *** andythenorth has quit IRC
16:21:22  *** Wormnest has joined #openttd
16:32:08  *** andythenorth has joined #openttd
16:39:27  *** HerzogDeXtEr has joined #openttd
16:46:40  <glx> https://github.com/OpenTTD/OpenTTD/compare/master...glx22:script_list_save <-- and that just checks it's a ScriptList and get the object, no real saving, and I don't know how loading would look (and probably don't want to try)
16:47:10  <glx> so yeah ScriptList will stay not supported :)
16:57:22  *** tokai has joined #openttd
16:57:22  *** ChanServ sets mode: +v tokai
17:19:25  <Samu> i'll take a look at it
17:19:28  <Samu> thx
17:36:32  *** Flygon has quit IRC
17:37:48  *** Wolf01 has joined #openttd
17:38:01  <Samu> hum, still no idea what's happening
17:41:07  <Samu> what to do with real_instance, visual studio isn't helping much
17:41:58  <glx> oh and savegames are broken
17:42:16  <glx> so don't try it on something you want to keep :)
17:53:48  <glx> force pushed a version not breaking saves
17:55:23  <Samu> aha, ScriptList *list = (ScriptList *)real_instance;
17:55:28  <Samu> got something:o
17:57:06  <glx> BTW saving is the easy part
17:57:27  <glx> loading will be a pain to interface with the vm
18:10:58  *** yeoldetoast has quit IRC
18:12:17  <andythenorth> well
18:12:21  <andythenorth> what can we talk about?
18:12:22  <andythenorth> lunch?
18:26:53  <TrueBrain> Just to repeat yesterday: so anyone a suggestion or opinion on the open comment on https://github.com/OpenTTD/OpenTTD/pull/9421 ? Would be nice to have this accepted before next nightly ...
18:28:04  <DorpsGek> [OpenTTD/game-coordinator] TrueBrain commented on pull request #5: Add: small tweaks so the Game Coordinator understands Master Server data https://git.io/JcbpK
18:38:39  <TrueBrain> glx / LordAro / Rubidium / michi_cc anyone? Pretty please? Packed with a ribbon and everything?
18:38:59  <DorpsGek> [OpenTTD/OpenTTD] Xaroth approved pull request #9421: Codechange: split settings.ini over several files https://git.io/JcNeN
18:50:14  <DorpsGek> [OpenTTD/OpenTTD] michicc commented on pull request #9421: Codechange: split settings.ini over several files https://git.io/JcNJI
18:53:15  <DorpsGek> [OpenTTD/OpenTTD] DorpsGek pushed 1 commits to master https://git.io/JcNJ6
18:53:16  <DorpsGek>   - Update: Translations from eints (by translators)
18:54:54  <TrueBrain> michi_cc: tnx for comment; but I am having a hard time deciphering what you mean, sorry :P
18:55:37  *** tokai|noir has joined #openttd
18:55:37  *** ChanServ sets mode: +v tokai|noir
18:55:38  <michi_cc> The settings in that file don't really affect anything what we call "driver" in the source code.
18:55:47  <TrueBrain> ah! That way
18:55:56  <TrueBrain> I am fine renaming it, but nobody is giving me inspiration what to call these settings :P
18:56:07  <TrueBrain> they don't fit in "gui_settings.ini" I guess?
18:56:13  <michi_cc> multimedia :P
18:56:16  <TrueBrain> sure
18:56:28  <TrueBrain> at least finalyl a suggestion of someone :D Tnx :)
18:56:54  <michi_cc> One that is probably 20 years out-of-date :)
18:57:25  <TrueBrain> to me this is bikeshedding anyway
18:57:33  <TrueBrain> what-ever we name it, someone will disagree anyway :D
18:59:33  <michi_cc> The would fit into an ui_settings.ini (and GUI is UI, too).
18:59:49  <DorpsGek> [OpenTTD/OpenTTD] TrueBrain dismissed a review for pull request #9421: Codechange: split settings.ini over several files https://git.io/JcNeN
18:59:51  <TrueBrain> I now already called it multimedia_settings.ini :P
18:59:52  <DorpsGek> [OpenTTD/OpenTTD] TrueBrain updated pull request #9421: Codechange: split settings.ini over several files https://git.io/Jc7uM
19:00:10  <TrueBrain> I failed to find a better name than "driver" for sound/music, but I think this works just fine :)
19:00:26  <TrueBrain> I tried not to put different groups in the same file, to make finding settings a tiny bit easier :)
19:01:23  <DorpsGek> [OpenTTD/OpenTTD] TrueBrain commented on pull request #9421: Codechange: split settings.ini over several files https://git.io/JcNTe
19:01:52  <TrueBrain> in general, a lot of settings could use a touch-up to what they are named, what they do, and in which category they are :P
19:02:36  <TrueBrain> gui.keep_all_autosave as example, I fail to see how that is "gui" :P
19:02:45  *** tokai has quit IRC
19:03:36  <TrueBrain> gui.refresh_rate <- pretty sure that should not be in "gui" :D Ghehe :)
19:04:27  <TrueBrain> now hoping for an approval before the nightly starts ....... :D
19:07:47  <DorpsGek> [OpenTTD/OpenTTD] glx22 approved pull request #9421: Codechange: split settings.ini over several files https://git.io/JcNTx
19:08:21  <TrueBrain> \o/ <3
19:15:15  <TrueBrain> that moment that neither MacOS nor Emscripten is the slowest target :P
19:15:46  <glx> mingw ?
19:15:51  <TrueBrain> yup
19:15:57  <TrueBrain> 64bit is the slowest of them all :P
19:16:07  <DorpsGek> [OpenTTD/OpenTTD] TrueBrain closed issue #9386: Building on MacOS fails with "bus error: 10" https://git.io/Jngox
19:16:10  <DorpsGek> [OpenTTD/OpenTTD] TrueBrain merged pull request #9421: Codechange: split settings.ini over several files https://git.io/Jc7uM
19:16:11  <glx> and it's a debug build
19:16:45  <glx> oups too late
19:16:52  <glx> nightly started 10 minutes ago
19:16:57  <DorpsGek> [OpenTTD/OpenTTD] TrueBrain approved pull request #9422: Codechange: roughly half number of strings for settings https://git.io/JcNIV
19:17:13  <glx> no conflicts ?
19:17:40  <DorpsGek> [OpenTTD/OpenTTD] TrueBrain approved pull request #9418: Codechange: Don't explicitly unset _generating_world outside of genworld.cpp https://git.io/JcNID
19:18:01  <glx> just checked, there are of course conflicts
19:18:12  <TrueBrain> glx: or did it? (nightly) :P
19:18:46  <TrueBrain> glx: https://github.com/OpenTTD/OpenTTD/pull/9395 <- was that PR otherwise ready?
19:19:12  <TrueBrain> (except for that one open comment)
19:20:29  <glx> yeah it's ready
19:20:42  <glx> (I'll touch a lot with the other PR)
19:21:29  <TrueBrain> will you pick up the wrong "{" there too?
19:22:42  <TrueBrain> glx: what do you think .. I am tempted to flip the if (!netsave) statement into a if (netsave) ?
19:22:52  <TrueBrain> just feels weird, to have !netsave, bit weird to read?
19:24:13  <glx> #9397 will solve that with prefix
19:24:39  <TrueBrain> cool
19:24:50  <TrueBrain> yeah, that sounds better anyway
19:24:55  <TrueBrain> as this boolean is really weird, honestly
19:24:57  <DorpsGek> [OpenTTD/OpenTTD] glx22 approved pull request #9395: Change #9188: Netsave now behaves like autosave https://git.io/JcNts
19:25:04  <TrueBrain> so if you fix that with 9397, let me fix the { real quick :P
19:26:41  <DorpsGek> [OpenTTD/OpenTTD] TrueBrain dismissed a review for pull request #9395: Change #9188: Netsave now behaves like autosave https://git.io/JcNts
19:26:44  <DorpsGek> [OpenTTD/OpenTTD] TrueBrain updated pull request #9395: Change #9188: Netsave now behaves like autosave https://git.io/JnQY9
19:26:48  <TrueBrain> (and also some other minor stuff that annoyed me :P)
19:26:59  <TrueBrain> (dots, capitals, etc)
19:27:52  <DorpsGek> [OpenTTD/OpenTTD] TrueBrain approved pull request #9395: Change #9188: Netsave now behaves like autosave https://git.io/JcNtP
19:28:04  <glx> haha self approve :)
19:28:07  <TrueBrain> kinda :P
19:28:18  <TrueBrain> I btw changed the commit message, as I really didn't like this one
19:28:47  <TrueBrain> "behaves like" is such an unclear statement .. as I am really sure it doesn't save every 1 game-month now :P
19:29:13  <TrueBrain> still don't like my message, but at least a bit more to the point .. maybe when you squash you can put the cherry on top :P
19:30:34  <DorpsGek> [OpenTTD/OpenTTD] TrueBrain approved pull request #9368: Codechange: Remove last FOR_XXX macros https://git.io/JcNqL
19:36:13  <DorpsGek> [OpenTTD/OpenTTD] glx22 merged pull request #9368: Codechange: Remove last FOR_XXX macros https://git.io/JZNwn
19:36:21  <glx> bye bye macros
19:36:43  <TrueBrain> we will not miss you
19:36:51  <TrueBrain> someone on Discord disagree'd btw (shocker, someone disagreeing)
19:36:58  <TrueBrain> he said the macros were fine
19:37:06  <glx> they worked yes
19:37:44  <glx> but there was hidden side effects
19:37:49  <TrueBrain> yup :)
19:37:55  <TrueBrain> it just always makes me laugh, that no matter what you do
19:37:59  <TrueBrain> someone is going to disagree
19:38:05  <TrueBrain> and claims to the "old way of doing stuff"
19:38:12  <TrueBrain> like demanding a setting back in the UI that was removed in 2014 :P
19:38:35  <glx> yeah a one time set :)
19:38:52  <glx> once you change it you usually never go back
19:42:40  <TrueBrain> okay, building the nightly succeeded :D \o/
19:43:48  *** WormnestAndroid has quit IRC
19:44:07  <DorpsGek> [OpenTTD/OpenTTD] glx22 merged pull request #9395: Change #9188: Netsave now behaves like autosave https://git.io/JnQY9
19:44:30  *** WormnestAndroid has joined #openttd
19:44:48  *** jottyfan has joined #openttd
19:45:36  <DorpsGek> [OpenTTD/OpenTTD] TrueBrain commented on pull request #9395: Change #9188: Netsave now behaves like autosave https://git.io/JcNY2
19:45:55  <DorpsGek> [OpenTTD/OpenTTD] TrueBrain closed issue #9188: `autosave/netsave.sav` is not numbered (unlike regular `autosave/autosave#.sav`) https://git.io/J3guT
20:19:02  <Samu> protected
20:19:18  <Samu> i want to get some data from protected
20:19:30  <Samu> do i just move it to public?
20:20:49  *** yeoldetoast has joined #openttd
20:26:26  <Samu> what is a enum
20:26:32  <Samu> int, uint? something else
20:26:47  <Samu> can i convert enum to int64?
20:31:44  <DorpsGek> [OpenTTD/OpenTTD] rubidium42 dismissed a review for pull request #9422: Codechange: roughly half number of strings for settings https://git.io/JcNIV
20:31:47  <DorpsGek> [OpenTTD/OpenTTD] rubidium42 updated pull request #9422: Codechange: roughly half number of strings for settings https://git.io/Jc7ao
20:33:25  <DorpsGek> [OpenTTD/OpenTTD] TrueBrain approved pull request #9422: Codechange: roughly half number of strings for settings https://git.io/JcNCp
20:34:05  *** nielsm has quit IRC
20:37:34  *** WormnestAndroid has quit IRC
20:38:03  *** WormnestAndroid has joined #openttd
20:40:53  <DorpsGek> [OpenTTD/OpenTTD] glx22 updated pull request #9397: Feature: Persistant rotation of numbered autosave after restart https://git.io/JndWW
20:44:15  <DorpsGek> [OpenTTD/OpenTTD] glx22 commented on pull request #9397: Feature: Persistant rotation of numbered autosave after restart https://git.io/JcNl4
20:50:08  <Samu> glx https://github.com/OpenTTD/OpenTTD/compare/master...SamuXarick:save-script-list
20:50:18  <Samu> just tried dealing with the save code
20:53:34  <DorpsGek> [OpenTTD/OpenTTD] rubidium42 merged pull request #9422: Codechange: roughly half number of strings for settings https://git.io/Jc7ao
20:53:56  <Rubidium> thanks TrueBrain
21:02:37  <glx> Samu: not the cleanest solution ;) and I think loading won't like you SQSL_TABLE usage
21:10:30  <andythenorth> is there a way to remove delivery requirements for desert towns and towns above snowline?
21:17:20  <DorpsGek> [OpenTTD/OpenTTD] Taschi120 commented on pull request #9395: Change #9188: Netsave now behaves like autosave https://git.io/JcNRL
21:19:15  *** Samu has quit IRC
21:22:35  <andythenorth> goes it a fork then?
21:23:45  <glx> I don't think there's a way, except game script maybe
21:28:09  <andythenorth> fork time
21:32:31  *** jottyfan has quit IRC
21:34:30  *** tokai has joined #openttd
21:34:30  *** ChanServ sets mode: +v tokai
21:35:04  *** WormnestAndroid has quit IRC
21:35:20  *** WormnestAndroid has joined #openttd
21:41:30  *** tokai|noir has quit IRC
21:44:11  *** Wolf01 has quit IRC
21:50:58  *** andythenorth has quit IRC
22:52:08  *** HerzogDeXtEr has quit IRC
23:31:12  *** Wuzzy has joined #openttd
23:33:24  *** yeoldetoast has quit IRC

Powered by YARRSTE version: svn-trunk