⏴ go to previous day
00:44:32  *** Smedles has quit IRC (Remote host closed the connection)

00:45:51  *** Smedles has joined #openttd

02:19:38  <DorpsGek> [OpenTTD/OpenTTD] irasponsible opened issue #16018: [Bug]: Lowering or Levelling terrain on multiplayer server causes immediate desync https://github.com/OpenTTD/OpenTTD/issues/16018

02:27:15  <DorpsGek> [OpenTTD/OpenTTD] James103 commented on issue #16018: [Bug]: Lowering or Levelling terrain on multiplayer server causes immediate desync https://github.com/OpenTTD/OpenTTD/issues/16018

02:32:25  <talltyler> Oops

02:33:00  *** Flygon has quit IRC (Quit: A toaster's basically a soldering iron designed to toast bread)

02:36:20  <DorpsGek> [OpenTTD/OpenTTD] James103 commented on pull request #15822: Change: Lowering tiles when terraforming makes rocky slopes https://github.com/OpenTTD/OpenTTD/pull/15822#pullrequestreview-5259131331

02:38:35  <peter1138> talltyler, likely just check for DoCommandFlag::Execute at terraform_cmd.cpp:263 before calling RandomRange.

02:39:45  *** irasponsible has joined #openttd

02:39:45  <irasponsible> I hadn't thought to try *raising* land - surely that wouldn't call the "place rocks on lowered terrain", right? Weird that that would also desync

02:40:58  <peter1138> As you are not charged extra for rocks appearing (I hope), we don't need to pretend to place rocks in test mode? Might also depend on pass.

02:57:00  <talltyler> Creating rocks won’t cost anything, but possibly if you’re lowering a tile more than one step, the rocks that get placed in the first step need to be cleared in the second step? I don’t imagine that gets tested though.

02:58:45  <talltyler> It’s bedtime though, so I’ll test this tomorrow before opening a PR. Looks like we’ll get a beta4 tomorrow maybe 🙂

03:23:28  *** Philemon has joined #openttd

03:31:05  *** Phileman has quit IRC (Ping timeout: 480 seconds)

05:44:14  <DorpsGek> [OpenTTD/OpenTTD] rubidium42 merged pull request #16013: Codechange: use 'enum class' for VehicleSoundEvent https://github.com/OpenTTD/OpenTTD/pull/16013

07:41:56  *** jfs_ has joined #openttd

08:25:00  <peter1138> Hm.

09:22:25  *** jfs_ has quit IRC (Ping timeout: 480 seconds)

09:26:31  <DorpsGek> [OpenTTD/OpenTTD] eints-sync[bot] pushed 1 commits to master https://github.com/OpenTTD/OpenTTD/commit/913bded89b8b8cd0e6c5dced7f3e6748639dd87d

09:26:32  <DorpsGek>   - Update: Translations from eints (by translators)

09:32:40  <peter1138[d]> https://cdn.discordapp.com/attachments/1008473233844097104/1551164179636035664/image.png?ex=6ab0f9b7&is=6aafa837&hm=ab6b40b283edb585c9ccf79378a3b01057f43a7085e52e2a98757d2dea3bde4d&

09:32:40  <peter1138[d]> There was PR to add this, but we ignored it.

09:39:27  <Rubidium> oh, I thought you were aware of that and, while reviewing, got sucked into the "this and this and this and that can be done better"-time-sinkhole

09:40:55  <peter1138> I was aware of it, but no, what I was doing is something I experimented with a while ago (Dec 2024...)

10:01:52  <peter1138> This was at one point less code than the original, too.

10:01:58  <peter1138> Sadly it's a bit more now.

10:18:06  <peter1138> Hmm, should I add the company HQ? :D

10:41:13  <Rubidium> I'm reading 'scope creep' ;)

10:59:39  <peter1138> :-)

11:03:33  *** WormnestAndroid has quit IRC (Read error: Connection reset by peer)

11:03:36  *** WormnestAndroid has joined #openttd

11:42:24  <DorpsGek> [OpenTTD/OpenTTD] PeterN opened pull request #16019: Change: Industry cargo chain window redesign https://github.com/OpenTTD/OpenTTD/pull/16019

11:52:49  <irasponsible> https://cdn.discordapp.com/attachments/1008473233844097104/1551199450872156241/image.png?ex=6ab11a90&is=6aafc910&hm=ccb9b4527b01fa4469e239076124793e6566487a61655e347b91e590c129acd5&

11:52:49  <irasponsible> Should you swap the Grey and Brown industry colours? Have houses (which essentially do nothing) be the same colour as the background, instead of processing industries (which are much more important than houses). Just seems a bit like it de-emphasises the wrong thing

11:54:15  <peter1138> Yeah, colours can be tweaked, I just picked them basically at random.

11:54:59  <peter1138> No consideration for colour blind players, but also there's no key either.

11:57:09  <peter1138> Also wondering about including cargo stubs on the outer edges so can see when a chain ends more clearly.

11:59:33  <peter1138> Complex industry chains often just have everything as Processing industries too.

12:00:02  <irasponsible> https://cdn.discordapp.com/attachments/1008473233844097104/1551201265730719794/image.png?ex=6ab11c41&is=6aafcac1&hm=f35245e542b21009feb28d54616792b1b356b10b7523133602c97c71b7dd979c&

12:00:02  <irasponsible> https://cdn.discordapp.com/attachments/1008473233844097104/1551201266758197322/image.png?ex=6ab11c41&is=6aafcac1&hm=4207ab33f9b5a9cc6e8db92ebcfa559660ec7f9269e10922867f13c64c028b53&

12:00:02  <irasponsible> https://cdn.discordapp.com/attachments/1008473233844097104/1551201267328753765/image.png?ex=6ab11c42&is=6aafcac2&hm=5fa0b5fa56239127460356f9274b9c04e57459411f66d3d8ff78e439e20b4bd9&

12:00:02  <irasponsible> better example. i also wonder if it's clearer to keep the solid outline matching the outline on the connectors?

12:00:02  <irasponsible> it's reasonably colour-blind friendly, it's all labelled, but it might be good to to group goods instead of alphabetise+optimise? - without colour it's a little hard to tell which goods go where (chemicals and engineering supplies in this example) and grouping up could help with that

12:01:48  <peter1138> Maybe, though drawing it as a button does make it clearer that you can click on it. (Though the cargo labels and lines are also clickable)

12:03:47  <irasponsible> the solid outlines make it a clearer diagram to read, i think? they don't make it as clear it's clickable, but it's pretty easy to find that out anyway

12:08:14  <irasponsible> https://cdn.discordapp.com/attachments/1008473233844097104/1551203328216793160/image.png?ex=6ab11e2d&is=6aafccad&hm=6afb32c60ecb61cb47abac8f3b698c816f1bf9e1f88698d9d2a96953918a1c6a&

12:08:14  <irasponsible> come to think of it, thetwo greens are really similar, could push "natural" and "extractive" into the same box for simplicity's sake?

12:12:46  <peter1138> Or an entirely different colour.

12:20:08  *** jfs_ has joined #openttd

12:26:44  <DorpsGek> [OpenTTD/OpenTTD] github-advanced-security[bot] commented on pull request #16019: Change: Industry cargo chain window redesign https://github.com/OpenTTD/OpenTTD/pull/16019#pullrequestreview-5260560070

12:30:29  <peter1138> Ah, the old false alert.

12:31:39  <peter1138> Now to do the doxygen :o

12:39:41  *** Flygon has joined #openttd

12:53:08  <DorpsGek> [OpenTTD/OpenTTD] PeterN updated pull request #16019: Change: Industry cargo chain window redesign https://github.com/OpenTTD/OpenTTD/pull/16019

13:01:51  <DorpsGek> [OpenTTD/OpenTTD] spatulari opened pull request #16020: Feature: Build script https://github.com/OpenTTD/OpenTTD/pull/16020

13:12:20  <rito12_13> 566 lines ?

13:12:51  <DorpsGek> [OpenTTD/OpenTTD] spatulari requested changes for pull request #16019: Change: Industry cargo chain window redesign https://github.com/OpenTTD/OpenTTD/pull/16019#pullrequestreview-5260649318

13:14:43  <LordAro> lol no

13:19:23  <DorpsGek> [OpenTTD/OpenTTD] PeterN closed pull request #16020: Feature: Build script https://github.com/OpenTTD/OpenTTD/pull/16020

13:23:53  <DorpsGek> [OpenTTD/OpenTTD] rubidium42 commented on pull request #16019: Change: Industry cargo chain window redesign https://github.com/OpenTTD/OpenTTD/pull/16019#issuecomment-5750075504

13:25:17  <DorpsGek> [OpenTTD/team] javigomez opened issue #719: [ca_ES] Translator access request https://github.com/OpenTTD/team/issues/719

13:33:36  <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler opened pull request #16021: Fix #16018: Don't desync when terraforming due to unsafe RandomRange https://github.com/OpenTTD/OpenTTD/pull/16021

13:44:15  <peter1138[d]> https://cdn.discordapp.com/attachments/1008473233844097104/1551227492021440542/image.png?ex=6ab134ae&is=6aafe32e&hm=4d985c255d99519e93b8fc4baf805abcbb87400d85a50cce39345c056665521e&

13:44:15  <peter1138[d]> Hmm, the shorter headers make the window quite a bit narrow, nice perk?

13:49:10  <andythenorth[d]> looks good

13:49:54  <andythenorth[d]> what does it do now when I've added far too many inputs and outputs?

13:50:10  <andythenorth[d]> https://cdn.discordapp.com/attachments/1008473233844097104/1551228982522024006/image.png?ex=6ab13611&is=6aafe491&hm=d467ad3a482470e71d7a03d3e96988d27407ea79a6bf19c44362559eff7ffd19&

14:01:18  <talltyler> Oh, duh, of course that would break regression

14:03:11  <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler updated pull request #16021: Fix #16018: Don't desync when terraforming due to unsafe RandomRange https://github.com/OpenTTD/OpenTTD/pull/16021

14:07:46  <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler opened pull request #16022: Doc: Update changelog for 16.0-beta4 https://github.com/OpenTTD/OpenTTD/pull/16022

14:14:02  <DorpsGek> [OpenTTD/website] 2TallTyler opened pull request #406: Add: Release post for OpenTTD 16.0-beta4 release https://github.com/OpenTTD/website/pull/406

14:30:29  <DorpsGek> [OpenTTD/OpenTTD] rubidium42 approved pull request #16021: Fix #16018: Don't desync when terraforming due to unsafe RandomRange https://github.com/OpenTTD/OpenTTD/pull/16021#pullrequestreview-5260823320

14:35:27  <peter1138[d]> andythenorth[d]: Basically the same but with less redundant connection lines

14:47:55  <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler merged pull request #16021: Fix #16018: Don't desync when terraforming due to unsafe RandomRange https://github.com/OpenTTD/OpenTTD/pull/16021

14:47:58  <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler closed issue #16018: [Bug]: Lowering or Levelling terrain on multiplayer server causes immediate desync https://github.com/OpenTTD/OpenTTD/issues/16018

15:18:25  <peter1138[d]> andythenorth[d]: What set/economy is that?

15:23:55  *** bfgiant has joined #openttd

15:23:55  <bfgiant> peter1138[d]: Looks like in-work "FIRS6"

15:26:07  <peter1138[d]> https://cdn.discordapp.com/attachments/1008473233844097104/1551253127674462288/image.png?ex=6ab14c8e&is=6aaffb0e&hm=7f8515652fdd26d890ff551839707f1f705ce0a0456d13e04f8bcf6391dd180b&

15:26:07  <peter1138[d]> Could do with the magical "how wide does this string need to be given we have X lines available" function.

15:29:30  *** Wolf01 has joined #openttd

15:29:50  *** SigHunter has quit IRC (Ping timeout: 480 seconds)

15:40:16  *** SigHunter has joined #openttd

15:55:05  *** Flygon has quit IRC (Read error: Connection reset by peer)

16:04:15  <DorpsGek> [OpenTTD/OpenTTD] rubidium42 approved pull request #16022: Doc: Update changelog for 16.0-beta4 https://github.com/OpenTTD/OpenTTD/pull/16022#pullrequestreview-5261009035

16:10:36  <peter1138[d]> https://cdn.discordapp.com/attachments/1008473233844097104/1551264324826173541/image.png?ex=6ab156fc&is=6ab0057c&hm=0528ea34407f84fcedd5154b81424fd0cecf32a40f6dae4da0de9ee830fc63f8&

16:10:36  <peter1138[d]> Yeah, wrapping helps.

16:11:23  <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler merged pull request #16022: Doc: Update changelog for 16.0-beta4 https://github.com/OpenTTD/OpenTTD/pull/16022

16:28:34  <andythenorth[d]> peter1138[d]: it is FIRS 6, but FIRS 5 Steeltown would do the same at Port or Wharf

16:29:42  *** gelignite has joined #openttd

16:39:20  <peter1138[d]> https://cdn.discordapp.com/attachments/1008473233844097104/1551271555600744470/image.png?ex=6ab15db8&is=6ab00c38&hm=8b65073f8881e58027628728eb8d00d9e8f0293c824ddfc94bb941acbc555171&

16:39:20  <peter1138[d]> Yeah, the port is still horrible, but still better IMHO.

16:40:20  <andythenorth[d]> it is

16:42:01  <andythenorth[d]> trying to think of ways to re-orientate or squash the connecting lines

16:42:45  <andythenorth[d]> can't think of any

16:43:27  <peter1138[d]> Well, possibly a further redesign to allow variable-height boxes.

16:44:03  <andythenorth[d]> in the FIRS cargoflow graphviz, I had to shortcut engineering supplies and farm supplies

16:44:27  <andythenorth[d]> because they go to so many places, adding edges for them makes the graph almost impossible to layout

16:44:52  <andythenorth[d]> so they have 'accepts engineering supplies' text added to the nodes

16:48:01  <peter1138[d]> That's not something that would have any positive effect here -- it would just hide the lines, but nothing else would change.

17:00:18  <talltyler> If someone wants to approve the beta4 website post, I can do the release whenever 🙂

17:02:39  <talltyler> Variable-height boxes might be nice, to make things fit and also show how "complicated" and industry is just based on its number of inputs/outputs.

17:03:51  <andythenorth[d]> I wondered about quite chunky increments

17:04:01  <andythenorth[d]> truly variable would be a bit irregular to look at

17:04:53  <DorpsGek> [OpenTTD/OpenTTD] Brickblock1 commented on pull request #15906: Fix #11127: use callback for dual-headed engine purchase capacity https://github.com/OpenTTD/OpenTTD/pull/15906#issuecomment-5751287345

17:05:41  <belajalilija> irasponsible: which two greens? :skull:

17:05:49  <belajalilija> i really like the black outlines

17:08:14  <peter1138[d]> https://cdn.discordapp.com/attachments/1008473233844097104/1551278827269787808/image.png?ex=6ab1647d&is=6ab012fd&hm=0cff9e6f9455db019e5092791c08eb6420af95894cb8ff32c95d3745b30a2636&

17:08:14  <peter1138[d]> What about all black outlines, no button-style look?

17:08:20  <peter1138[d]> As someone suggested earlier.

17:08:29  <DorpsGek> [OpenTTD/website] Kuhnovic approved pull request #406: Add: Release post for OpenTTD 16.0-beta4 release https://github.com/OpenTTD/website/pull/406#pullrequestreview-5261163880

17:09:11  <belajalilija> peter1138[d]: isnt it the same as what i responded to? i like it tho

17:09:24  <peter1138[d]> I wasn't paying attention to that.

17:09:29  <belajalilija> fair

17:09:30  <kuhnovic> andythenorth[d]: We could start a second column after x items and run the connecting lines in between

17:09:45  <belajalilija> idk if i like it because it looks good or i like it because it looks dated

17:10:11  <andythenorth[d]> peter1138[d]: I like this, they're all ok, the button effect is fine too, but not necessary

17:10:12  <peter1138[d]> I kinda think it's over-doing the black outlines.

17:10:38  <belajalilija> perhaps a little

17:11:10  <belajalilija> the button effect makes it looks more modern

17:37:40  <DorpsGek> [OpenTTD/nml] Brickblock1 opened pull request #444: Add: New capacity callbacks and misc_flags bit 5 constant https://github.com/OpenTTD/nml/pull/444

17:39:23  <DorpsGek> [OpenTTD/nml] Brickblock1 updated pull request #444: Add: New capacity callbacks and misc_flags bit 5 constant https://github.com/OpenTTD/nml/pull/444

17:50:06  <DorpsGek> [OpenTTD/nml] Brickblock1 updated pull request #444: Add: New capacity callbacks and misc_flags bit 5 constant https://github.com/OpenTTD/nml/pull/444

17:55:37  *** gelignite has quit IRC (Ping timeout: 480 seconds)

18:16:04  <peter1138[d]> https://cdn.discordapp.com/attachments/1008473233844097104/1551295900935004231/image.png?ex=6ab17464&is=6ab022e4&hm=a0fc551e3ca1f4cd18ac3d5df2c7b21b41adb2b3b8e5f46bff48595157cbd5e8&

18:16:04  <peter1138[d]> Wrapping helps greek

18:23:57  *** MinchinWeb[m] has quit IRC (Ping timeout: 480 seconds)

18:23:59  *** MinchinWeb[m] has joined #openttd

18:25:12  <peter1138[d]> Ah, shame this code does not work :-(

18:25:35  <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler created new tag: 16.0-beta4 https://github.com/OpenTTD/OpenTTD/releases/tag/16.0-beta4

18:26:05  <peter1138[d]> It'll break mid-word if it needs to, where as I need to break only on proper breaks.

18:26:25  <talltyler> I prefer buttons to black outlines, personally

18:26:49  <peter1138[d]> I think it fits the (Open)TTD style better.

18:26:52  <talltyler> (for the industry rectangles, the connectors are so much nicer with the black lines than the current yellow)

18:26:59  <talltyler> Agreed.

18:27:43  <andythenorth[d]> It's all greek to me

18:28:26  *** gelignite has joined #openttd

18:34:14  <DorpsGek> [OpenTTD/OpenTTD] PeterN updated pull request #16019: Change: Redesign industry cargo chain window https://github.com/OpenTTD/OpenTTD/pull/16019

18:40:33  <DorpsGek> [OpenTTD/nml] andythenorth commented on pull request #444: Add: New capacity callbacks and misc_flags bit 5 constant https://github.com/OpenTTD/nml/pull/444#pullrequestreview-5261383104

18:41:01  <DorpsGek> [OpenTTD/nml] andythenorth commented on pull request #444: Add: New capacity callbacks and misc_flags bit 5 constant https://github.com/OpenTTD/nml/pull/444#pullrequestreview-5261384044

18:41:20  <DorpsGek> [OpenTTD/nml] andythenorth commented on pull request #444: Add: New capacity callbacks and misc_flags bit 5 constant https://github.com/OpenTTD/nml/pull/444#pullrequestreview-5261384711

18:45:26  *** MinchinWeb[m] has quit IRC (Read error: Connection reset by peer)

18:45:42  *** MinchinWeb[m] has joined #openttd

18:50:09  <DorpsGek> [OpenTTD/website] 2TallTyler merged pull request #406: Add: Release post for OpenTTD 16.0-beta4 release https://github.com/OpenTTD/website/pull/406

18:57:02  <talltyler> Discord, TT-Forums, and Steam done for beta4.

18:58:17  <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler merged pull request #16017: Codefix: Comments in GS code stating that they are about AI https://github.com/OpenTTD/OpenTTD/pull/16017

19:06:23  *** WormnestAndroid has quit IRC (Ping timeout: 480 seconds)

19:07:17  *** WormnestAndroid has joined #openttd

19:11:18  *** WormnestAndroid has quit IRC (Read error: Connection reset by peer)

19:11:18  *** MinchinWeb[m] has quit IRC (Read error: Connection reset by peer)

19:11:42  *** WormnestAndroid has joined #openttd

19:12:43  *** MinchinWeb[m] has joined #openttd

19:23:52  *** WormnestAndroid has quit IRC (Ping timeout: 480 seconds)

19:27:36  *** WormnestAndroid has joined #openttd

19:39:46  *** gelignite has quit IRC ()

19:56:54  <michi_cc[d]> Reddit/X, too.

19:58:37  <andythenorth[d]> thanks 🙂

19:58:38  *** WormnestAndroid has quit IRC (Read error: Connection reset by peer)

19:59:47  *** WormnestAndroid has joined #openttd

20:12:45  <kuhnovic> You forgot the @ everyone on discord so I think very few people got the notification. Maybe you can edit it after the fact?

20:14:56  <kuhnovic> Not the end of the world of course, I'm happy that there is a new beta 🙂

20:21:59  <belajalilija> kuhnovic: should i?

20:22:15  <belajalilija> in Discord channel #developer-updates right?

20:22:25  <belajalilija> wait no

20:22:46  <belajalilija> Discord channel #openttd-releases

20:22:58  <belajalilija> wait i cant even write there xd

20:25:33  <kuhnovic> Let talltyler do it, maybe he can edit it

20:26:19  <talltyler> I intentionally didn’t do it, it’s such a small release

20:32:16  <kuhnovic> Fair enough

20:32:36  <kuhnovic> 2TT makes no mistakes 😁

20:33:46  <talltyler> Yes, except the mistake that necessitated a beta4 release today 😛

20:42:25  <kuhnovic> Just a minor oopsie 😛

20:43:15  <peter1138[d]> I'll do fediverse when the pub kicks me out

21:14:13  <DorpsGek> [OpenTTD/OpenTTD] jsrmalvarez commented on issue #16016: [Bug]: Snow Coverage below 50% on a height 1 map does not spawn any snow tiles. https://github.com/OpenTTD/OpenTTD/issues/16016

21:16:47  *** MinchinWeb[m] has quit IRC (Ping timeout: 480 seconds)

21:16:57  *** MinchinWeb[m] has joined #openttd

21:22:00  *** MinchinWeb[m] has quit IRC (Read error: Connection reset by peer)

21:22:16  *** MinchinWeb[m] has joined #openttd

21:39:34  *** Wolf01 has quit IRC (Quit: Once again the world is quick to bury me.)

21:50:37  <DorpsGek> [OpenTTD/OpenTTD] PeterN updated pull request #16019: Change: Redesign industry cargo chain window https://github.com/OpenTTD/OpenTTD/pull/16019

21:54:55  *** jfs_ has quit IRC (Ping timeout: 480 seconds)

22:41:28  *** Flygon has joined #openttd

continue to next day ⏵