IRC logs for #openttd on OFTC at 2021-07-09
⏴ go to previous day
00:03:34 *** WormnestAndroid has joined #openttd
00:18:13 *** Gustavo6046 has quit IRC (Remote host closed the connection)
00:18:14 *** Gustavo6046 has joined #openttd
00:28:27 *** WormnestAndroid has quit IRC (Ping timeout: 480 seconds)
00:35:33 *** WormnestAndroid has joined #openttd
00:43:33 *** WormnestAndroid has quit IRC (Ping timeout: 480 seconds)
00:46:39 *** WormnestAndroid has joined #openttd
00:50:48 *** snail_UES_ has quit IRC (Quit: snail_UES_)
00:54:59 *** yeoldetoast has joined #openttd
01:14:14 *** WormnestAndroid has quit IRC (Ping timeout: 482 seconds)
01:21:01 *** WormnestAndroid has joined #openttd
01:53:59 *** WormnestAndroid has quit IRC (Ping timeout: 480 seconds)
01:57:21 *** WormnestAndroid has joined #openttd
02:44:58 *** debdog has quit IRC (Ping timeout: 480 seconds)
02:46:32 *** ChanServ sets mode: +v tokai
02:53:15 *** tokai|noir has quit IRC (Ping timeout: 480 seconds)
05:35:42 *** yeoldetoast has quit IRC (Quit: My Mac Mini has gone to sleep. ZZZzzz…)
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:47:08 <peter1138> If my method declaration reaches column 298, it's time to split into multiple lines, right? :D
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 (Read error: Connection reset by peer)
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.
10:21:09 *** EER has quit IRC (Remote host closed the connection)
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: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: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:32 *** outtatime has joined #openttd
10:57:39 <Samu> now i'm testing an extreme scenario
10:59:02 *** bro has quit IRC (Remote host closed the connection)
11:07:55 <Samu> is now just this: return [m_nearCityPairArray, m_usedCitiesPassTable, m_usedCitiesMailTable];
11:09:51 *** blathijs has quit IRC (Quit: tot later)
12:01:20 *** virtualrandomnumber has joined #openttd
12:01:34 *** virtualrandomnumber has quit IRC ()
12:08:07 *** WormnestAndroid has quit IRC (Remote host closed the connection)
12:10:22 *** WormnestAndroid has joined #openttd
12:16:32 *** blathijs has joined #openttd
12:32:24 *** snail_UES_ has joined #openttd
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: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: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 (Ping timeout: 480 seconds)
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 (Quit: andythenorth)
13:32:36 <Samu> oh rats, too complex for me
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: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 *** ChanServ sets mode: +v tokai
14:47:16 <Samu> wow, youtube ads are scam ads
14:47:40 <Samu> "wanna earn $1000 a day?"
14:52:02 *** roadt_ has quit IRC (Ping timeout: 480 seconds)
14:53:01 *** snail_UES_ has quit IRC (Quit: snail_UES_)
14:53:15 *** tokai|noir has quit IRC (Ping timeout: 480 seconds)
14:56:35 *** tokai has quit IRC (Ping timeout: 480 seconds)
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)
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:12 <Samu> huge profits but nah, not huge enough
16:19:30 *** andythenorth has quit IRC (Ping timeout: 480 seconds)
16:21:22 *** Wormnest has joined #openttd
16:32:08 *** andythenorth has joined #openttd
16:39:27 *** HerzogDeXtEr has joined #openttd
16:47:10 <glx> so yeah ScriptList will stay not supported :)
16:57:22 *** ChanServ sets mode: +v tokai
17:36:32 *** Flygon has quit IRC (Quit: A toaster's basically a soldering iron designed to toast bread)
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: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 (Quit: My Mac Mini has gone to sleep. ZZZzzz…)
18:12:21 <andythenorth> what can we talk about?
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: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: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: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 (Ping timeout: 480 seconds)
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:15:15 <TrueBrain> that moment that neither MacOS nor Emscripten is the slowest target :P
19:15:57 <TrueBrain> 64bit is the slowest of them all :P
19:16:09 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain merged pull request #9421: Codechange: split settings.ini over several files https://git.io/Jc7uM
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: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:19:12 <TrueBrain> (except for that one open comment)
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: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: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: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:44 <glx> but there was hidden side effects
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: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 (Ping timeout: 480 seconds)
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: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: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 (Ping timeout: 480 seconds)
20:37:34 *** WormnestAndroid has quit IRC (Ping timeout: 480 seconds)
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: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
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 (Quit: Leaving)
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:32:31 *** jottyfan has quit IRC (Quit: jottyfan)
21:34:30 *** ChanServ sets mode: +v tokai
21:35:04 *** WormnestAndroid has quit IRC (Ping timeout: 480 seconds)
21:35:20 *** WormnestAndroid has joined #openttd
21:41:30 *** tokai|noir has quit IRC (Ping timeout: 480 seconds)
21:44:11 *** Wolf01 has quit IRC (Quit: Once again the world is quick to bury me.)
21:50:58 *** andythenorth has quit IRC (Quit: andythenorth)
22:52:08 *** HerzogDeXtEr has quit IRC (Read error: Connection reset by peer)
23:33:24 *** yeoldetoast has quit IRC (Quit: My Mac Mini has gone to sleep. ZZZzzz…)
continue to next day ⏵
