IRC logs for #openttd on OFTC at 2024-05-20
            
01:52:40 *** Wormnest has quit IRC (Quit: Leaving)
02:06:12 *** johnfranklin has joined #openttd
02:06:12 <johnfranklin> Is GPL something one can regret?
02:34:38 *** emperorjake has joined #openttd
02:34:38 <emperorjake> I know at least one author who regretted making their sets GPL after they were butchered by Gadg8eer
02:38:31 *** reldred has joined #openttd
02:38:31 <reldred> See if I butcher someoneโ€™s GPL work I at least try and make sure it never leaves GitHub ๐Ÿคฃ
02:39:03 <reldred> Coz as soon as something goes up on bananas youโ€™ll be paying child support for the rest of your life
02:40:06 *** D-HUND has joined #openttd
02:41:52 <audigex> Yeah thatโ€™s the thing that would potentially bother me about GPL - if someone genuinely butchered the work
02:41:52 <audigex> Itโ€™s unlikely but possible, even if theyโ€™re well intentioned but lacking in skill
02:43:36 *** debdog has quit IRC (Ping timeout: 480 seconds)
03:14:08 *** wensimehrp has joined #openttd
03:14:08 <wensimehrp> If I decompile some proprietary grf and distribute the decompiled code using gpl... With totally remade graphics, then is it illegal?
03:14:08 <wensimehrp> (reminds me of a 20-year-old game)
04:01:53 *** HerzogDeXtEr1 has joined #openttd
04:03:00 *** kamnet has joined #openttd
04:03:00 <kamnet> wensimehrp: If you don't have the license to do so, technically yes. Is somebody going to actually come after you for it? Extremely unlikely. I've done so myself at least twice. ๐Ÿ™‚
04:07:41 *** HerzogDeXtEr has quit IRC (Ping timeout: 480 seconds)
04:43:28 <DorpsGek> [OpenTTD/OpenTTD] eints-sync[bot] pushed 1 commits to master https://github.com/OpenTTD/OpenTTD/commit/fb4a370d3fd4287a1d025f181f8cb0d8e648680a
04:43:29 <DorpsGek> - Update: Translations from eints (by translators)
05:49:08 *** XYZ_ has joined #openttd
05:51:34 *** XYZ has quit IRC (Ping timeout: 480 seconds)
06:13:13 *** tokai|noir has joined #openttd
06:13:14 *** ChanServ sets mode: +v tokai|noir
06:20:01 *** tokai has quit IRC (Ping timeout: 480 seconds)
06:22:52 *** Flygon has quit IRC (Quit: A toaster's basically a soldering iron designed to toast bread)
06:47:21 *** SigHunter has quit IRC (Ping timeout: 480 seconds)
06:51:30 *** SigHunter has joined #openttd
07:13:37 <DorpsGek> [OpenTTD/survey-web] survey-summary[bot] pushed 1 commits to main https://github.com/OpenTTD/survey-web/commit/20d811d7db50fba9a2c3c05ce0d4a54a0551dd73
07:13:38 <DorpsGek> - Add: summary for week 20 of 2024 (by OpenTTD Survey)
07:16:01 *** SigHunter has quit IRC (Ping timeout: 480 seconds)
07:22:18 *** SigHunter has joined #openttd
07:22:28 *** peter1138 has joined #openttd
07:22:28 <peter1138> 2.3% of JGRPP 0.59.0 use NPF for ships...
07:30:21 *** SigHunter has quit IRC (Ping timeout: 480 seconds)
07:43:05 *** gelignite has joined #openttd
07:49:46 <DorpsGek> [OpenTTD/nml] ahyangyi commented on pull request #330: Fix `nearby_tile_same_grf`, `nearby_tile_other_grf` and `nearby_tile_original_gfx`, they all compared to wrong constants https://github.com/OpenTTD/nml/pull/330#issuecomment-2119874257
07:51:07 <DorpsGek> [OpenTTD/nml] LordAro approved pull request #331: Update: Python versions for regression https://github.com/OpenTTD/nml/pull/331#pullrequestreview-2065587499
07:51:57 <DorpsGek> [OpenTTD/nml] LordAro commented on pull request #330: Fix `nearby_tile_same_grf`, `nearby_tile_other_grf` and `nearby_tile_original_gfx`, they all compared to wrong constants https://github.com/OpenTTD/nml/pull/330#issuecomment-2119877513
07:53:15 <DorpsGek> [OpenTTD/nml] ahyangyi commented on pull request #330: Fix `nearby_tile_same_grf`, `nearby_tile_other_grf` and `nearby_tile_original_gfx`, they all compared to wrong constants https://github.com/OpenTTD/nml/pull/330#issuecomment-2119879422
07:54:14 <DorpsGek> [OpenTTD/nml] ahyangyi updated pull request #330: Fix `nearby_tile_same_grf`, `nearby_tile_other_grf` and `nearby_tile_original_gfx`, they all compared to wrong constants https://github.com/OpenTTD/nml/pull/330
07:56:08 *** ahyangyi has joined #openttd
07:56:08 <ahyangyi> ...ah, the commit checker... I guess I could only fix that when at home :S
08:34:03 *** SigHunter has joined #openttd
08:35:46 *** kuhnovic has joined #openttd
08:35:46 <kuhnovic> Wow, I have read some complaints here and there about the performance of std::unordered_map, but it really is slow
08:36:41 <kuhnovic> At least when you use one in YAPF
08:38:15 <kuhnovic> peter1138: didn't you recently create a patch with YAPF using stl containers? Did you see something similar?
08:39:15 <peter1138> I replaced "FixedSizeArray" with a std::deque, so not really comparable.
08:40:54 <peter1138> My performance measurements (probably naรฏve) didn't problems.
08:41:19 <peter1138> See
08:41:48 <kuhnovic> Ah ok, makes sense, I wouldn't expect any problems there
08:42:25 <kuhnovic> We do have our own hashmap implementation, but it's such a PITA to work with.
08:42:56 *** SigHunter has quit IRC (Ping timeout: 480 seconds)
08:45:15 <peter1138> We probably have multiple ๐Ÿ˜„
08:45:54 <peter1138> Of course, we also have vehicle hashes that have support bucket collisions by storing next pointers in Vehicle.
09:22:55 *** SigHunter has joined #openttd
09:31:39 <kuhnovic> Oh yeah we also have one in the AyStar implementation ๐Ÿ˜›
09:34:26 *** SigHunter has quit IRC (Ping timeout: 480 seconds)
09:40:55 <peter1138> These work fine but they're not really suited to stl conversion.
09:51:54 <kuhnovic> Tbh I'd rather use an open source implementation rather than a homebrew one
09:51:56 *** SigHunter has joined #openttd
09:52:18 <kuhnovic> Technically ours is an open source implementation too if we're being pedantic
09:52:56 *** truebrain has joined #openttd
09:52:56 <truebrain> Open Source 3rdparty, solves your linguistic issue ๐Ÿ˜›
09:59:10 *** _jgr_ has joined #openttd
09:59:10 <_jgr_> There's no shortage of open source container implementations to choose from ๐Ÿ˜›
10:07:30 <DorpsGek> [OpenTTD/nml] glx22 merged pull request #331: Update: Python versions for regression https://github.com/OpenTTD/nml/pull/331
10:09:31 *** _glx_ has joined #openttd
10:09:31 <_glx_> ahyangyi: You'll need to rebase too while at it
10:34:14 <ahyangyi> Yeah, need #331 ๐Ÿ˜›
10:58:20 *** Ox7C5 has joined #openttd
11:05:18 <kuhnovic> _jgr_: The problem of having too many options ๐Ÿ˜›
11:25:39 *** andythenorth has joined #openttd
11:25:39 <andythenorth> https://cdn.discordapp.com/attachments/1008473233844097104/1242075765806334003/image.png?ex=664c84b3&is=664b3333&hm=6efe07ded3ecd34c96c3700992f721f93bba1ef046c96b1377be862e7409ecdc&
11:25:39 <andythenorth> did we bloat RAM use recently? ๐Ÿ˜› or is my GS really inefficient?
11:25:52 <andythenorth> map is 256x256
11:28:42 <peter1138> Did we bloat WindowServer too?
11:37:37 <andythenorth> no Apple do that as part of the service
11:41:25 <DorpsGek> [OpenTTD/OpenTTD] PeterN opened pull request #12702: Codechange: Use std::filesystem::exists instead of access. https://github.com/OpenTTD/OpenTTD/pull/12702
12:19:38 <Eddi|zuHause> anything that didn't bloat?
12:26:30 <andythenorth> https://cdn.discordapp.com/attachments/1008473233844097104/1242091077176000565/image.png?ex=664c92f5&is=664b4175&hm=0801ba31c3302f3e012f2cb4579ba71c32ebc1b64e397b864f4d29b4b40b129f&
12:26:30 <andythenorth> restarted OpenTTD
12:26:56 <andythenorth> https://cdn.discordapp.com/attachments/1008473233844097104/1242091186529894432/image.png?ex=664c930f&is=664b418f&hm=dfb841722d8ad4d4c17f7874a91e564e55c85f3806775a78423c0c0a82cd8a69&
12:26:56 <andythenorth> and with my save reloaded
12:32:19 <andythenorth> https://cdn.discordapp.com/attachments/1008473233844097104/1242092544364384388/image.png?ex=664c9453&is=664b42d3&hm=d99e3a16513b6200713767540e8836c6a91cb415ea420534c57f7f9781d28541&
12:32:19 <andythenorth> hmm the obvious candidate isn't to blame ๐Ÿ˜›
12:33:42 <_jgr_> You should take that memory usage number with a big pinch of salt
13:11:07 <andythenorth> ๐Ÿง‚
13:11:24 <andythenorth> hmm....could we have map-global permanent registers per grf?
13:11:35 <andythenorth> ironic I know, given I'm complaining about memory use
13:11:54 <andythenorth> I've remembered why I had to use GS to do FIRS primary production increases
13:12:30 <andythenorth> I wanted to space them out over time, so they only happen occasionally, but grf can't do that
14:15:05 *** SigHunter_ has joined #openttd
14:18:24 *** SigHunter has quit IRC (Ping timeout: 480 seconds)
14:29:17 <peter1138> "Oh just store that config as json" sounded fine... until that json was stored in json, as a string. Mmm.
14:36:17 <LordAro> lol.
15:02:10 *** Wormnest has joined #openttd
15:06:41 <kuhnovic> That moment when you've been performance measurements for a while, and you suddenly realize asserts are still on
15:08:33 <_glx_> I hope it's at least a release build
15:09:25 <kuhnovic> RelWithDebInfo
15:10:09 *** Flygon has joined #openttd
15:15:00 <_jgr_> For the most part asserts aren't going to make a great deal of difference
15:16:22 <_jgr_> It still gives a useful signal
15:19:39 <kuhnovic> The water regions and YAPF both have quite a lot of asserts, which could make a noticable difference
15:19:55 <kuhnovic> As well as stl containers. I'm mostly turning them off just to be safe.
15:20:14 <kuhnovic> Well, rather unsafe... but sure ๐Ÿ˜›
16:47:33 *** toktik is now known as Guest7080
16:47:37 *** toktik has joined #openttd
16:48:04 *** Guest7080 has quit IRC (Remote host closed the connection)
16:50:19 *** wallaby2 is now known as wallabra
16:58:50 <DorpsGek> [OpenTTD/OpenTTD] PeterN opened pull request #12703: Codechange: Use L"" instead of _T("") https://github.com/OpenTTD/OpenTTD/pull/12703
17:08:03 <andythenorth> grf global storage then? ๐Ÿ˜›
17:28:51 *** Wolf01 has joined #openttd
17:30:17 <DorpsGek> [OpenTTD/OpenTTD] LordAro approved pull request #12703: Codechange: Use L"" instead of _T("") https://github.com/OpenTTD/OpenTTD/pull/12703#pullrequestreview-2066675007
17:34:26 <DorpsGek> [OpenTTD/OpenTTD] PeterN merged pull request #12703: Codechange: Use L"" instead of _T("") https://github.com/OpenTTD/OpenTTD/pull/12703
17:44:40 <_glx_> ah left over from win9x support
17:45:46 <_glx_> though `_T` macro is actually the proper way when not using explicitely W version of the function
17:54:55 *** yeah_its_gloria has joined #openttd
17:54:55 <yeah_its_gloria> wouldn't it be good to switch to the Unicode variants explicitly anyway
17:55:34 <andythenorth> Hmm donโ€™t need grf-global storage, GS can do it
17:56:02 <andythenorth> Should be fine
17:56:22 <andythenorth> GS can also write to the text window
17:59:31 <peter1138> Would be better to drop Windows support ๐Ÿ™‚
18:04:43 <andythenorth> Nobodyโ€™s going to let my GS read FIRS industry registers, right? ๐Ÿ˜›
18:10:04 *** Ox7C5 has quit IRC ()
19:07:55 <Eddi|zuHause> correct
19:11:49 <DorpsGek> [OpenTTD/nml] ahyangyi updated pull request #330: Fix `nearby_tile_same_grf`, `nearby_tile_other_grf` and `nearby_tile_original_gfx`, they all compared to wrong constants https://github.com/OpenTTD/nml/pull/330
19:15:38 <DorpsGek> [OpenTTD/nml] glx22 approved pull request #330: Fix `nearby_tile_same_grf`, `nearby_tile_other_grf` and `nearby_tile_original_gfx`, they all compared to wrong constants https://github.com/OpenTTD/nml/pull/330#pullrequestreview-2066854474
19:16:27 <DorpsGek> [OpenTTD/nml] glx22 merged pull request #330: Fix `nearby_tile_same_grf`, `nearby_tile_other_grf` and `nearby_tile_original_gfx`, they all compared to wrong constants https://github.com/OpenTTD/nml/pull/330
19:16:41 <_glx_> slightly reworked the message ๐Ÿ˜‰
19:21:23 <peter1138> Good cos it was a bit odd.
19:24:09 <peter1138> How long has that been wrong? ๐Ÿ˜„
19:29:40 <peter1138> Hmm, Windows 7 is 1%, Windows 8.1 is 0.3%, Windows Vista (lol) 0.2%
19:30:02 <truebrain> the game stil runs on Vista?!
19:32:58 <andythenorth> macOS Vista?
19:42:34 <yeah_its_gloria> Windows Ventura
19:42:36 <Eddi|zuHause> isn't vista basically the same as 7, just with a new paint?
19:43:37 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 commented on pull request #12420: Add: support for admin to change company allow lists https://github.com/OpenTTD/OpenTTD/pull/12420#pullrequestreview-2066898456
19:47:02 <yeah_its_gloria> I'd say vista was 7 before they had the hardware for it
19:51:33 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 commented on pull request #12692: Fix: Force console to use LTR layouting. https://github.com/OpenTTD/OpenTTD/pull/12692#issuecomment-2121099891
19:53:17 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 approved pull request #12694: Fix: Hide empty house 'classes' in house picker. https://github.com/OpenTTD/OpenTTD/pull/12694#pullrequestreview-2066915821
19:56:17 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 approved pull request #12701: Fix #12681: Abstract filetype not set for loads of network client join savegames https://github.com/OpenTTD/OpenTTD/pull/12701#pullrequestreview-2066920302
19:56:51 *** gelignite has quit IRC (Quit: Stay safe!)
19:59:56 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 commented on pull request #12702: Codechange: Use std::filesystem::exists instead of access. https://github.com/OpenTTD/OpenTTD/pull/12702#pullrequestreview-2066925645
20:03:24 <peter1138> Hmm.
20:04:19 <peter1138> Changed language and the game crashed in the blitter. Oof.
20:05:12 <DorpsGek> [OpenTTD/OpenTTD] PeterN updated pull request #12702: Codechange: Use std::filesystem::exists instead of access. https://github.com/OpenTTD/OpenTTD/pull/12702
20:17:11 <LordAro> probably not important
20:18:24 <DorpsGek> [OpenTTD/OpenTTD] PeterN opened pull request #12704: Fix: Make progress bars obey language direction. https://github.com/OpenTTD/OpenTTD/pull/12704
20:19:41 <DorpsGek> [OpenTTD/OpenTTD] LordAro approved pull request #12704: Fix: Make progress bars obey language direction. https://github.com/OpenTTD/OpenTTD/pull/12704#pullrequestreview-2066956552
20:32:42 *** HerzogDeXtEr1 has quit IRC (Read error: Connection reset by peer)
20:54:09 <DorpsGek> [OpenTTD/OpenTTD] PeterN opened pull request #12705: Fix: Crash if interface scale or font size changes with chat box present. https://github.com/OpenTTD/OpenTTD/pull/12705
21:47:34 *** Wolf01 has quit IRC (Quit: Once again the world is quick to bury me.)
22:04:55 <_glx_> peter1138: about 12 years, but stations were not supported for most of that time ๐Ÿ™‚
23:40:12 <DorpsGek> [OpenTTD/OpenTTD] PeterN merged pull request #12704: Fix: Make progress bars obey language direction. https://github.com/OpenTTD/OpenTTD/pull/12704