IRC logs for #openttd on OFTC at 2021-06-21
            
00:36:24 *** WormnestAndroid has joined #openttd
00:47:50 *** tokai has joined #openttd
00:47:50 *** ChanServ sets mode: +v tokai
00:54:52 *** tokai|noir has quit IRC (Ping timeout: 480 seconds)
01:04:20 <DorpsGek> [OpenTTD/OpenTTD] glx22 commented on issue #9386: Building on MacOS fails with "bus error: 10" https://git.io/Jngox
01:59:18 *** bryjen has joined #openttd
02:41:47 *** Flygon has joined #openttd
03:01:22 *** bryjen has quit IRC (Quit: Leaving)
03:02:32 *** D-HUND has joined #openttd
03:05:54 *** debdog has quit IRC (Ping timeout: 480 seconds)
03:08:20 *** tokai|noir has joined #openttd
03:08:21 *** ChanServ sets mode: +v tokai|noir
03:15:23 *** tokai has quit IRC (Ping timeout: 480 seconds)
03:26:52 *** glx has quit IRC ()
03:27:57 *** WormnestAndroid has quit IRC (Remote host closed the connection)
03:29:41 *** WormnestAndroid has joined #openttd
04:37:01 *** roadt__ has joined #openttd
04:44:02 *** roadt_ has quit IRC (Ping timeout: 480 seconds)
05:36:05 *** Gustavo6046 has joined #openttd
05:38:32 *** Gustavo6046_ has quit IRC (Ping timeout: 480 seconds)
06:07:38 *** snail_UES_ has quit IRC (Quit: snail_UES_)
06:13:38 *** tokai has joined #openttd
06:13:38 *** ChanServ sets mode: +v tokai
06:14:01 *** Gustavo6046 has quit IRC (Ping timeout: 480 seconds)
06:16:00 *** Gustavo6046 has joined #openttd
06:20:24 *** tokai|noir has quit IRC (Ping timeout: 480 seconds)
06:41:50 *** sla_ro|master has joined #openttd
07:18:36 *** D-HUND is now known as debdog
07:25:10 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain commented on pull request #9390: Fix: don't allow deleting SettingDesc to prevent compilers getting confused https://git.io/Jn6KT
07:25:14 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain closed pull request #9390: Fix: don't allow deleting SettingDesc to prevent compilers getting confused https://git.io/JnwpB
07:25:38 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain commented on pull request #9389: Fix bf500c3: reduce the amount of vars to track by using std::string_view https://git.io/Jn6Kn
07:25:41 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain closed pull request #9389: Fix bf500c3: reduce the amount of vars to track by using std::string_view https://git.io/Jnwtg
07:30:23 *** HerzogDeXtEr has joined #openttd
07:46:08 <TrueBrain> so using std::variant bumps MacOS minimum to 10.14 only because of the exception it can throw .. as that is embedded in the libc++ dylib ..
07:46:14 <TrueBrain> that is an evil reason to bump the minimum :P
07:47:06 <TrueBrain> (well, std::visit does, but who is counting)
07:47:19 <Rubidium> discrete mathematicians?
08:00:55 <peter1138> Ju
08:01:08 <peter1138> Just drop OS X support :D
08:08:24 <TrueBrain> well, according to Steam, we would lose ...... well, almost nobody :P
08:08:26 <TrueBrain> similar with Linux btw
08:08:34 <TrueBrain> funny how Steam really is a Windows platform
08:09:18 * debdog would have dropped win support, too
08:09:49 <TrueBrain> just develop for the machines we are on
08:09:55 <TrueBrain> and if it happens to work on another, good for you
08:09:55 <TrueBrain> ?
08:09:56 <TrueBrain> :D
08:17:04 *** tokai|noir has joined #openttd
08:17:04 *** ChanServ sets mode: +v tokai|noir
08:23:33 *** tokai has quit IRC (Ping timeout: 480 seconds)
08:34:38 <peter1138> Sounds right.
08:53:21 <Xaroth> if it compiles, ship it
08:58:37 <peter1138> Well, that is the problem...
09:00:09 <LordAro> ships keep getting lost
09:02:03 <Xaroth> build more buoys
09:03:05 <TrueBrain> build bigger canal
09:05:59 <Xaroth> Flood the entire continent
09:06:20 <Xaroth> ... I think I once again find myself on some government watchlist.
09:07:30 *** WormnestAndroid has quit IRC (Ping timeout: 480 seconds)
09:09:39 <TrueBrain> can confirm
09:14:56 *** WormnestAndroid has joined #openttd
09:19:15 *** tokai has joined #openttd
09:19:15 *** ChanServ sets mode: +v tokai
09:26:19 *** esselfe has quit IRC (Read error: No route to host)
09:26:20 *** tokai|noir has quit IRC (Ping timeout: 480 seconds)
09:27:01 *** iSoSyS has joined #openttd
09:27:57 *** iSoSyS has quit IRC ()
09:40:14 <Timberwolf> I have an urge for coffee.
09:43:45 <Timberwolf> Also, grinding the beans gives me something to do while I wait for this Terraform run.
09:43:51 <Timberwolf> Ooh, it was unexpectedly fast.
09:44:00 <LordAro> no coffee for you!
09:45:22 <Timberwolf> I realise I also need a new build of a lambda, that uses Docker-in-Docker so I have plenty of time for coffee setup now :)
09:47:34 <Eddi|zuHause2> i herd u liek containerz, so we put some containerz in ur containerz
10:20:08 *** Samu has joined #openttd
10:37:56 *** Eddi|zuHause2 has quit IRC (Remote host closed the connection)
10:38:35 *** Eddi|zuHause2 has joined #openttd
10:40:14 *** andythenorth has joined #openttd
10:42:22 *** tokai|noir has joined #openttd
10:42:22 *** ChanServ sets mode: +v tokai|noir
10:49:20 *** tokai has quit IRC (Ping timeout: 480 seconds)
11:42:49 <peter1138> https://twitter.com/mikeysan/status/1406730249027473410
11:47:14 *** WormnestAndroid has quit IRC (Remote host closed the connection)
11:47:27 *** WormnestAndroid has joined #openttd
12:10:15 *** Wuzzy has joined #openttd
12:12:41 *** glx has joined #openttd
12:12:41 *** ChanServ sets mode: +v glx
12:49:15 *** snail_UES_ has joined #openttd
13:10:22 <andythenorth> well
13:10:33 <andythenorth> goes it redesign FIRS Steeltown?
13:34:18 <andythenorth> hmm
13:39:17 <FLHerne> Well, personally I'd be much more interested in the 'better West Country' economy you've mentioned, because Steeltown isn't really my thing
13:42:45 *** nielsm has joined #openttd
13:47:36 <andythenorth> yes
13:47:53 <andythenorth> I am only poking at Steeltown because it makes literally no sense currently
13:48:12 <andythenorth> everything -> vehicles cargo -> nowhere, pointless
13:48:25 <andythenorth> it's alright for the first 50 years
13:48:29 <andythenorth> then it's FAIL
13:52:41 <Samu> I like the idea of No subsidies!
13:52:46 <Samu> makes AI vs AI a bit fairer
14:00:14 *** gelignite has joined #openttd
14:00:19 <peter1138> Is that "NoSubsidies" like "NoAI"?
14:00:30 <Samu> no, it's a new game setting
14:00:36 <peter1138> Can gamescripts set up subsidies...
14:01:02 <Samu> I think they can, but haven't tested it
14:08:54 <Samu> towns building tunnels, it's still a weird thing to see
14:48:15 <nielsm> I don't think there's a subsidies API no, you can make goals and reward the player but the multiplier to all delivery payments thing isn't possible
14:51:31 <Samu> my ai does some weird stuff with aircraft, he went from coleman count to kelling k1, totally ignored the bakewell lb-9 for some reason
15:13:19 *** namad7 has joined #openttd
15:21:22 *** roadt__ has quit IRC (Ping timeout: 480 seconds)
15:22:26 *** roadt__ has joined #openttd
15:23:17 *** namad7 has quit IRC ()
15:25:45 *** WormnestAndroid has quit IRC (Ping timeout: 480 seconds)
15:27:41 *** WormnestAndroid has joined #openttd
15:46:01 *** Progman has joined #openttd
16:00:11 <Samu> AIVehicle.GetBuildWithRefitCapacity is returning -1 for some reason
16:00:46 <Samu> need to investigate better with visual studio
16:09:27 <glx> unbuildable engine, invalid cargo, or failure to build vehicle or refit it once built
16:10:17 *** WormnestAndroid has quit IRC (Ping timeout: 480 seconds)
16:10:55 *** WormnestAndroid has joined #openttd
16:16:35 <Samu> found the problem!
16:17:01 <Samu> it can't get a free unit number
16:17:13 <glx> too many vehicles
16:17:26 <Samu> but i'm only querying for capacity
16:17:41 <Samu> GetBuildWithRefitCapacity
16:17:45 <glx> yes but it needs to build a vehicle for that
16:18:31 <glx> as capacity may depend on a newgrf callback
16:19:48 <Samu> well, the limit is 200 aircraft, he's at the limit
16:22:36 <Samu> there's a DC_QUERY_COST flag in use and a DC_AUTOREPLACE is what it needs
16:24:24 <Samu> I was so used to 5000 max vehicles, I never triggered this bug before
16:24:25 *** sla_ro|master has quit IRC ()
16:24:44 <glx> DC_AUTOREPLACE is added later in case of reffiting in CmdBuildVehicle
16:26:53 <glx> maybe some reordering is needed, like setting subflags, and then testing subflags instead of flags for unit_num
16:29:45 *** Taschi has joined #openttd
16:29:45 <Samu> https://i.imgur.com/5y4Wa3p.png primary capacity = -1, making calculated income totally wrong
16:30:27 <glx> incorrect handling of -1 is your bug ;)
16:31:09 <Taschi> hello
16:31:33 <Taschi> -1 should just be banned, it is the worst number and keeps creating problems for C++ devs
16:32:08 <glx> it's standard error code
16:32:29 <Taschi> i know, that's the joke
16:34:33 *** Gustavo6046_ has joined #openttd
16:37:57 *** Progman has quit IRC (Remote host closed the connection)
16:38:29 <glx> something like https://github.com/OpenTTD/OpenTTD/compare/master...glx22:refit maybe
16:38:50 *** Gustavo6046 has quit IRC (Ping timeout: 480 seconds)
16:38:50 *** Gustavo6046_ is now known as Gustavo6046
16:43:31 <Samu> im testing refitted to passengers, which is the default cargo
16:44:09 <glx> then it's not a refit
16:44:41 <Samu> it isn't, but still... it should handle it
16:47:20 <Samu> subflags is DC_QUERY_COST
16:48:49 <Samu> gonna test your change, but i think it won't work
16:49:11 <glx> I think it will for non default refit
16:49:24 <glx> needs more for default cargo
16:54:38 <glx> for default cargo you can use AIEngine::GetCapacity()
16:55:52 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 opened pull request #9391: Fix #9386: use variant instead of unique_ptr to prevent compilers failing on the code generation https://git.io/JnPSD
16:59:04 <glx> and of course it seems there's no way to get default mail capacity for aircraft
16:59:13 <glx> (in scripts)
17:02:21 <Taschi> i'm interested in contributing code to OpenTTD but haven't worked on any C++ project of this scale before and am a tiny bit lost
17:02:32 <Taschi> would it be possible for someone to "mentor" me a bit?
17:06:09 <glx> depends on what you want to do :)
17:06:11 <Taschi> i am a professional Java EE dev with some experience in release management and build systems so i don't need to be taught how to program, but i need help to get the "lay of the land"
17:10:14 <Taschi> for example i'm having trouble finding something that's good as a first issue to work on, i have already seen the "good first issue" tag on Github but a lot of them are a) unclear on whether they actually should be fixed/implemented or b) already have pull requests
17:11:17 <glx> usually if there's a PR it's linked in the issue
17:12:35 <Taschi> yes
17:14:20 <glx> Rubidium: of course MacOS doesn't like it :)
17:16:08 <Rubidium> glx: it doesn't like 10.14; all the warnings are in mm files
17:16:34 <glx> yes, typical when you change the minimum version, it deprecates a lot
17:16:40 <glx> and it's annoying
17:16:57 <Rubidium> that's well beyond my scope ;)
17:21:02 *** Flygon has quit IRC (Quit: A toaster's basically a soldering iron designed to toast bread)
17:21:16 <FLHerne> Taschi: https://github.com/OpenTTD/OpenTTD/issues/9188 looks straightforward, and somewhat uncontroversial
17:22:06 <FLHerne> (TrueBrain wasn't convinced it's useful, but I do and so does whoever reported it, and there's no harm from implementing it :p)
17:23:03 <TrueBrain> you have a fun way of reading words :P
17:23:33 <Taschi> to reproduce / test that, i'd have to run a server and then somehow cause it to crash... what would be the best way to do it? Would just killing the process via Task Manager be sufficient?
17:24:09 <FLHerne> That would work I think
17:26:10 *** Wolf01 has joined #openttd
17:30:40 <glx> Rubidium: based on the compile log, some warnings should be easy to fix, but some will require bigger changes
17:31:13 <Taschi> oh, no, i misunderstood that - the netsave is done when you're playing on a server, and your client loses the network connection
17:31:31 <Taschi> which i guess would happen when killing the server
17:31:43 <glx> yes it's easy to trigger locally :)
17:37:21 <Taschi> yeah, i've managed to trigger that
17:37:53 <DorpsGek> [OpenTTD/OpenTTD] Taschi120 commented on issue #9188: `autosave/netsave.sav` is not numbered (unlike regular `autosave/autosave#.sav`) https://git.io/J3guT
17:42:10 <DorpsGek> [OpenTTD/OpenTTD] SamuXarick opened issue #9392: Bug Report https://git.io/JnPbi
17:42:23 <glx> oh seems we don't set https://cmake.org/cmake/help/v3.9/variable/CMAKE_OSX_DEPLOYMENT_TARGET.html at the right place, and that's probably why we have MACOSX_DEPLOYMENT_TARGET in CI env
17:43:34 <DorpsGek> [OpenTTD/OpenTTD] glx22 commented on pull request #9391: Fix #9386: use variant instead of unique_ptr to prevent compilers failing on the code generation https://git.io/JnPbd
17:44:22 <glx> Samu: you again forgot to rename the issue ;)
17:44:42 <Samu> oh, sorry
17:46:43 <Samu> i dont know how to quote multiline code
17:46:48 <DorpsGek> [OpenTTD/OpenTTD] glx22 commented on issue #9392: AIVehicle.GetBuildWithRefitCapacity returns -1 when at the max num of vehicles https://git.io/JnPbi
17:46:59 <glx> with shift
17:47:13 <glx> but you can manually edit the link too
17:48:25 <glx> Lfirst-Llast
17:59:11 *** Beer has joined #openttd
18:16:50 *** Taschi has quit IRC (Quit: Leaving)
18:27:06 *** sla_ro|master has joined #openttd
18:27:48 *** Gustavo6046 has quit IRC (Quit: ZNC 1.8.2 - https://znc.in)
18:29:44 *** Gustavo6046 has joined #openttd
19:12:16 <Samu> im trying flags & DC_QUERY_COST at the unit_num
19:18:05 <Samu> hmm the vehicle was not undone
19:35:25 *** tokai has joined #openttd
19:35:25 *** ChanServ sets mode: +v tokai
19:40:02 *** Gustavo6046_ has joined #openttd
19:42:15 *** tokai|noir has quit IRC (Ping timeout: 480 seconds)
19:44:40 *** Gustavo6046 has quit IRC (Ping timeout: 480 seconds)
19:44:40 *** Gustavo6046_ is now known as Gustavo6046
19:45:50 *** Speeder__ has quit IRC (Ping timeout: 480 seconds)
19:52:52 *** WormnestAndroid has quit IRC (Ping timeout: 480 seconds)
19:55:03 *** WormnestAndroid has joined #openttd
20:18:55 <Samu> nevermind, there was no vehicle to undo
20:24:02 <andythenorth> yo
20:29:56 <Samu> my commit message may need editing
20:31:48 <Samu> https://github.com/OpenTTD/OpenTTD/compare/master...SamuXarick:BuildWithRefitCapacity-max-vehicles-fix?expand=1
20:32:09 <Samu> it seems a too simple fix
20:32:38 <Samu> too good to be true
20:43:44 <DorpsGek> [OpenTTD/OpenTTD] SamuXarick opened pull request #9393: Fix #9392: Return a valid value with GetBuildWithRefitCapacity when AIs are maxed out in vehicles https://git.io/JnXWb
20:48:09 *** Gustavo6046_ has joined #openttd
20:49:07 *** WormnestAndroid has quit IRC (Ping timeout: 480 seconds)
20:51:50 *** Gustavo6046 has quit IRC (Ping timeout: 480 seconds)
20:52:01 *** Progman has joined #openttd
20:53:08 *** gelignite has quit IRC (Quit: Stay safe!)
20:53:33 *** Gustavo6046 has joined #openttd
20:55:39 <Samu> AI now replaced all Coleman Count with Bakewell Luckett LB-9
20:55:43 <Samu> im happy
20:57:20 *** Gustavo6046_ has quit IRC (Ping timeout: 480 seconds)
21:00:32 *** Gustavo6046 has quit IRC (Quit: ZNC 1.8.2 - https://znc.in)
21:00:59 *** Gustavo6046 has joined #openttd
21:10:24 *** Samu has quit IRC (Quit: Leaving)
21:19:18 *** sla_ro|master has quit IRC ()
21:31:07 *** Wolf01 has quit IRC (Quit: Once again the world is quick to bury me.)
21:34:02 *** Progman has quit IRC (Remote host closed the connection)
21:35:41 *** WormnestAndroid has joined #openttd
21:42:42 *** Wormnest has joined #openttd
21:46:24 <DorpsGek> [OpenTTD/OpenTTD] michicc opened pull request #9394: Codechange: [OSX] Use more exact enum names where introduced with the 10.12 SDK. https://git.io/JnXum
22:10:28 *** nielsm has quit IRC (Ping timeout: 480 seconds)
22:12:22 *** jottyfan has joined #openttd
22:12:42 *** jottyfan has quit IRC ()
22:16:25 *** andythenorth has quit IRC (Quit: andythenorth)
22:19:19 <DorpsGek> [OpenTTD/OpenTTD] glx22 commented on pull request #9394: Codechange: [OSX] Use more exact enum names where introduced with the 10.12 SDK. https://git.io/JnX2n
22:30:26 *** HerzogDeXtEr has quit IRC (Read error: Connection reset by peer)
22:37:16 *** Wormnest has quit IRC (Quit: Leaving)
22:42:20 *** tokai|noir has joined #openttd
22:42:20 *** ChanServ sets mode: +v tokai|noir
22:49:25 *** tokai has quit IRC (Ping timeout: 480 seconds)
23:41:34 <DorpsGek> [OpenTTD/OpenTTD] michicc updated pull request #9394: Codechange: [OSX] Use more exact enum names where introduced with the 10.12 SDK. https://git.io/JnXum
23:50:01 *** Wuzzy has quit IRC (Remote host closed the connection)