IRC logs for #openttd on OFTC at 2025-08-06
            
00:19:11 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler commented on pull request #14477: Feature: Allow stations under bridges and bridges over stations. https://github.com/OpenTTD/OpenTTD/pull/14477#issuecomment-3157013617
00:23:47 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler opened issue #14500: [Bug]: Wrong error message trying to bridge over industry https://github.com/OpenTTD/OpenTTD/issues/14500
00:37:52 <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on pull request #14477: Feature: Allow stations under bridges and bridges over stations. https://github.com/OpenTTD/OpenTTD/pull/14477#issuecomment-3157041201
02:03:32 <talltyler> Oh. 🙂
02:04:16 *** Wormnest has quit IRC (Quit: Leaving)
02:11:18 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler commented on pull request #14477: Feature: Allow stations under bridges and bridges over stations. https://github.com/OpenTTD/OpenTTD/pull/14477#issuecomment-3157180075
02:23:19 <peter1138> Sute, I need to make the tilespecs function station-type aware. Then the minimum can be set to 3, which is required for default built-in waypoints.
02:24:46 <peter1138> If you think 3 is too much, feel free to replace the 20+ year old default waypoint graphics :~}
02:27:44 <reldred> Heh
03:38:33 *** Zathras has joined #openttd
03:42:06 *** Zathras_11 has quit IRC (Ping timeout: 480 seconds)
03:51:06 *** dh1 has quit IRC (Ping timeout: 480 seconds)
04:11:56 *** keikoz has joined #openttd
04:20:45 *** dh1 has joined #openttd
04:47:54 <DorpsGek> [OpenTTD/OpenTTD] eints-sync[bot] pushed 1 commits to master https://github.com/OpenTTD/OpenTTD/commit/73a69fd171c972bf9923b3d472b07d4c4fa14ce3
04:47:55 <DorpsGek> - Update: Translations from eints (by translators)
05:04:23 *** Flygon has quit IRC (Read error: Connection reset by peer)
05:22:58 *** dh1 has quit IRC (Ping timeout: 480 seconds)
05:32:28 *** tokai has joined #openttd
05:32:28 *** ChanServ sets mode: +v tokai
05:46:08 <DorpsGek> [OpenTTD/OpenTTD] Release workflow was not successful https://github.com/OpenTTD/OpenTTD/actions/runs/16768172853
06:04:17 *** dh1 has joined #openttd
06:12:21 *** dh1 has quit IRC (Ping timeout: 480 seconds)
06:37:41 *** tokai|noir has joined #openttd
06:37:41 *** ChanServ sets mode: +v tokai|noir
06:44:46 *** tokai has quit IRC (Ping timeout: 480 seconds)
06:51:59 <DorpsGek> [OpenTTD/OpenTTD] Release workflow was not successful https://github.com/OpenTTD/OpenTTD/actions/runs/16768172853
06:52:31 <truebrain> Seems uploading also runs in the same issue 😛
07:10:07 <DorpsGek> [OpenTTD/OpenTTD] Release workflow was not successful https://github.com/OpenTTD/OpenTTD/actions/runs/16768172853
07:10:33 *** Wolf01 has joined #openttd
07:19:14 <truebrain> Fixed some more Cloudflare OWASP blocking stuff; release now uploaded 🙂
07:21:36 <LordAro> \o/
07:47:08 *** Smedles has quit IRC (Quit: http://quassel-irc.org - Chat comfortably. Anywhere.)
07:47:20 *** Smedles has joined #openttd
07:47:55 *** dh1 has joined #openttd
08:32:59 *** SigHunter_ has quit IRC ()
08:35:46 *** SigHunter has joined #openttd
08:47:38 *** SigHunter has quit IRC ()
08:50:17 *** SigHunter has joined #openttd
09:12:51 *** jinks has quit IRC (Remote host closed the connection)
09:14:22 *** jinks has joined #openttd
09:32:59 <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on pull request #14477: Feature: Allow stations under bridges and bridges over stations. https://github.com/OpenTTD/OpenTTD/pull/14477#issuecomment-3158642120
09:39:35 <peter1138> Okay, how do you set up your workplace git repository systems?
09:41:57 <LordAro> gitlab
09:42:03 <LordAro> or were you looking for something more specific?
09:42:19 <peter1138> :)
09:43:40 <peter1138> More about the workflow side. Do you have a single main repository with multiple branches that everyone contributes to, or something more like the github way where each developer has their own repository with their own branches. And then merges / pull requests, etc...
09:45:13 <LordAro> single repo (no forks), but no one contributes to main directly - all done via merge requests
09:45:32 <LordAro> and then releases/3.23 release branch which gets things backported into
09:45:47 <LordAro> i took a lot of inspiration from OTTD when setting it all up ;)
09:47:05 <LordAro> previously in svn land we still had branches, but there was almost no reviewing at all (things were merged when they themselves were happy with it) and people committed directly to releases branches (or trunk) and things were forward ported. Things broke a lot
09:49:28 <peter1138> I set up ours with forks, which worked fine when it was basically me working on the main thing, and one other guy was working on something else.
09:49:46 <peter1138> Then we had an external developer come in, and forks also worked fine.
09:50:11 <LordAro> mm, our repo is too big and unweildy to effectively handle forks
09:50:23 <peter1138> Now I've got a couple of internal developers joining, and forks seems awkward, but pushing to branches in the main repository feels awkward too.
09:50:37 <peter1138> (But mainly cos I'm used to OpenTTD :))
09:51:19 <LordAro> branches-in-the-same-repo do make working on the same branch much more convenient than faffing around with 20 different remotes (whether or not you should be doing that is another matter)
10:06:29 <peter1138> Mmm.
10:06:53 <peter1138> Of course it can be changed, but if others are entrenched in the methods then it gets awkward :)
10:20:08 <peter1138> Ah ha, of course, the CI runs for the branches, which is both... good and bad...
10:31:42 <rito12_51026> Hi I have troubles with BaNaNaS, when I try to upload a file it stops before 5%, it goes to 4.92 and then back to 3.24 , any suggestions?
10:36:58 *** dh1 has quit IRC (Quit: My Mac has gone to sleep. ZZZzzz…)
10:44:53 <andythenorth> we have one repo, it's branch and merge
10:44:56 <andythenorth> you'd hate it 😛
10:45:10 <andythenorth> probably about 8 or so people using it most months
10:52:33 <pickpacket> rito12_51026: can you be more specific? What are you trying to upload and how big is it? Are you able to check the web browser's dev console to see if it gives errors?
11:18:59 <rito12_51026> https://cdn.discordapp.com/attachments/1008473233844097104/1402611914189705246/0.1.0.zip?ex=68948ba3&is=68933a23&hm=3ea2e333d2b6ced1fbcd0bffbc3e318673670757636a44ab40a68cf024703c9b&
11:18:59 <rito12_51026> I try to upload this 4.38 MB file.
11:31:57 <ahyangyi> LordAro: I read it as "... faffing around with 20 different emotes" at first 😱
11:35:26 <rito12_51026> I got this
11:35:32 <rito12_51026> `Error: tus: unexpected response while creating upload, originated from request (method: POST, url: https://bananas-api.openttd.org/new-package/tus/, response code: 404, response text: 404: Not Found , request id: n/a) `
11:53:26 <LordAro> rito12_51026: sounds like it might require a truebrain to look at the backend
12:01:35 <rito12_51026> I also got that
12:01:51 <rito12_51026> `Error: tus: failed to upload chunk at offset 0, caused by [object ProgressEvent], originated from request (method: PATCH, url: https://bananas-api.openttd.org/new-package/tus/35ec21e3868bc035e0ed53c5c1ebb736, response code: n/a, response text: n/a, request id: n/a) `
12:11:06 <truebrain> rito12_51026: can you give it another try?
12:12:07 <rito12_51026> Yes
12:17:44 <reldred> TrueBrain in the depths of the machine hitting things randomly with a pipe wrench.
12:23:17 <rito12_51026> And error `Error: tus: failed to resume upload, caused by [object ProgressEvent], originated from request (method: HEAD, url: https://bananas-api.openttd.org/new-package/tus/b357b550f7da0c6328ffaddd5bcca675, response code: n/a, response text: n/a, request id: n/a) `
12:25:03 <LordAro> reldred: not like there's any other way
12:37:17 *** WormnestAndroid has quit IRC (Read error: Connection reset by peer)
12:37:18 *** WormnestAndroid has joined #openttd
12:37:19 *** WormnestAndroid has quit IRC (Read error: Connection reset by peer)
12:37:21 *** WormnestAndroid has joined #openttd
12:37:23 *** WormnestAndroid has quit IRC (Read error: Connection reset by peer)
12:37:39 *** WormnestAndroid has joined #openttd
13:22:25 <peter1138> https://www.youtube.com/watch?v=_b8UVPtwJaA They copied my idea for icons in text :o
13:43:42 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler updated pull request #13289: Feature: Draw infinite water when all borders are water https://github.com/OpenTTD/OpenTTD/pull/13289
14:19:40 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler commented on pull request #13289: Feature: Draw infinite water when all borders are water https://github.com/OpenTTD/OpenTTD/pull/13289#issuecomment-3160383080
14:26:51 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler approved pull request #14312: Add: [Badges] Vehicle variable to query badges of current rail or road types. https://github.com/OpenTTD/OpenTTD/pull/14312#pullrequestreview-3092902080
14:54:32 <DorpsGek> [OpenTTD/OpenTTD] Kuhnovic commented on pull request #13289: Feature: Draw infinite water when all borders are water https://github.com/OpenTTD/OpenTTD/pull/13289#issuecomment-3160516075
15:04:02 <peter1138> Are we unhiding the freeform edges configuration setting?
15:09:55 *** Wormnest has joined #openttd
15:11:30 *** gelignite has joined #openttd
15:12:57 <talltyler> No, I’d set it automatically (and invisibly) if all map edges are water.
15:13:52 <talltyler> I wouldn’t say it makes my life easier, per se, but it does reduce the amount of code in the PR.
15:23:05 <peter1138> If it's using the setting behind the scenes, then I feel much happier about it.
15:23:46 <peter1138> I got the sense of there being multiple different checks for very similar things with the first iteration.
15:23:53 <talltyler> I’ll make that change later then. 🙂
15:24:39 <peter1138> Well, just an opinion, if it's hard work, don't feel pressured.
15:29:05 <talltyler> Nah, it’s not much work, and if that’s what it takes for an approval 😉
15:38:33 <peter1138> See, that feels bad :/
15:39:21 <truebrain> rito12_51026: It works when I try to upload your grf, and there are now server side messages. So this would require some more digging on your side, like what the network tab is doing etc. Hopefully someone here can guide you through that a bit. But we need to know what goes wrong on network level here.
15:39:38 <truebrain> Now? No. Lol
16:17:26 *** Flygon has joined #openttd
16:33:32 *** Smedles has quit IRC (Quit: http://quassel-irc.org - Chat comfortably. Anywhere.)
16:33:44 *** Smedles has joined #openttd
16:44:50 *** xava has joined #openttd
16:45:01 *** xava has quit IRC ()
17:33:31 <DorpsGek> [OpenTTD/OpenTTD] KogentaSan left a comment on commit: Update: Translations from eints https://github.com/OpenTTD/OpenTTD/commit/d545cec9a2633a626a09aeec933c8d91554ffaa2#r163547701
17:33:56 <rito12_51026> truebrain: Unfortunately I have no access to network's logs, but I will try other networks.
17:34:05 <truebrain> I meant in your browser, the network tab 🙂
17:34:10 <truebrain> it gives more visibility in the calls being made
17:39:45 <talltyler> I didn’t mean an approval from you, just that something about my current implementation isn’t doing it for reviewers in general. 🙂
17:40:36 <FLHerne> talltyler: re. the infinite water thing, checking of existing maps would also have to include the object/slope/dock/etc. checks
17:41:13 <talltyler> Hmm, definitely a corner case but good point.
17:41:19 <FLHerne> I do like the look of it
17:42:20 <FLHerne> I don't think it's an especially rare case, there's only a few tiles border on typical mapgen and people do like to take up space
17:43:33 <FLHerne> or just make the edge fairly clearly visible
17:44:01 <FLHerne> which might be the better option tbh, otherwise people will get frustrated by trying to build on it
17:44:45 <FLHerne> especially if the coastline is uneven and far from the map border, the boundary will be quite arbitrary
17:45:12 <talltyler> Ah, I was thinking about preventing griefing. Unintentional confusion is a likelier case. 🙂
17:45:29 <rito12_51026> truebrain: I'm afraid that my mobile browser do not have that
17:46:02 <FLHerne> fwiw my favourite of your screenshots is "2nd attempt"
17:46:16 <FLHerne> it's subtle but unambiguously a delinator
17:47:07 <FLHerne> 6th would work fine for gameplay, but stands out so much it kind of defeats the point to a degree
17:47:38 <FLHerne> the dotted line in 3rd is hard for me to see
17:48:52 *** peter1138[d] has quit IRC (Quit: User went offline on Discord a while ago)
18:01:38 <kuhnovic> FLHerne: talltyler For water region invalidation I had a similar problem where I had to catch any construction event. I was able to cover nearly all cases by adding some logic to the clear landscape command. That seems to be called for pretty much every construction action as a general ensure-the-tile-is-buildable check.
18:02:21 <kuhnovic> Object placement was a notable exception to this, which was reported as a bug. Let me find the PRs for you.
18:16:03 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 commented on pull request #14450: Codechange: Add font cache factory providers. https://github.com/OpenTTD/OpenTTD/pull/14450#pullrequestreview-3093666536
18:20:44 <kuhnovic> talltyler: PR11750 , I haven't been able to find the one about objects yet. The GitHub android app is a bit annoying.
19:03:17 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 approved pull request #14492: Codechange: Extract function to draw custom rail station foundations. https://github.com/OpenTTD/OpenTTD/pull/14492#pullrequestreview-3093858429
19:23:06 *** dh1 has joined #openttd
19:26:01 *** gelignite has quit IRC ()
19:29:46 <kuhnovic> Ah, 11750 contains both the landscape clear logic as well as the logic for objects 🙂
20:22:17 <DorpsGek> [OpenTTD/OpenTTD] Rau771 commented on pull request #13289: Feature: Draw infinite water when all borders are water https://github.com/OpenTTD/OpenTTD/pull/13289#issuecomment-3161486847
20:27:20 *** Smedles has quit IRC (Quit: http://quassel-irc.org - Chat comfortably. Anywhere.)
20:27:32 *** Smedles has joined #openttd
20:43:04 *** Dead_Bush_Sanpai_ has joined #openttd
20:45:00 *** Wolf01 has quit IRC (Quit: Once again the world is quick to bury me.)
20:50:35 <DorpsGek> [OpenTTD/OpenTTD] Rau771 commented on pull request #13289: Feature: Draw infinite water when all borders are water https://github.com/OpenTTD/OpenTTD/pull/13289#issuecomment-3161567102
20:58:13 <DorpsGek> [OpenTTD/OpenTTD] Rau771 commented on pull request #13289: Feature: Draw infinite water when all borders are water https://github.com/OpenTTD/OpenTTD/pull/13289#issuecomment-3161587834
21:30:32 <peter1138> Uploading content to bananas on a mobile phone is not a use case I'd envisaged...
22:17:00 *** keikoz has quit IRC (Ping timeout: 480 seconds)
22:20:56 *** xava has joined #openttd
22:27:37 *** xava has quit IRC (Quit: Leaving)
22:34:32 <yiffgirl> what's the etiquette regarding adding new strings?
22:35:27 *** Dead_Bush_Sanpai has joined #openttd
22:35:54 *** Dead_Bush_Sanpai_ has quit IRC (Read error: Connection reset by peer)
22:36:08 <yiffgirl> oh
22:36:08 <yiffgirl> strings.h is automatically generated, i don't need to edit it myself. whoops
22:36:55 <belajalilija> yiffgirl: Depends how long they are
22:37:06 *** Dead_Bush_Sanpai has quit IRC ()
22:37:35 *** Dead_Bush_Sanpai has joined #openttd
22:37:38 <talltyler> Yeah, only update `english.txt`. The game will generate StringIDs for you when you compile, and all translations are handled by the Web Translator, never in a PR. (Never added, changed in very rare circumstances like batch edits)
22:44:38 <peter1138> f
22:44:49 <peter1138> > /* This file is automatically generated. Do not modify */
22:44:54 <peter1138> Might be a clue :)
22:46:57 <DorpsGek> [OpenTTD/OpenTTD] PeterN merged pull request #14492: Codechange: Extract function to draw custom rail station foundations. https://github.com/OpenTTD/OpenTTD/pull/14492
22:53:25 <yiffgirl> peter1138: i genuinely have no clue how i managed to miss that notice. how embarrassing...
22:53:59 <yiffgirl> https://cdn.discordapp.com/attachments/1008473233844097104/1402786816704385095/image.png?ex=68952e87&is=6893dd07&hm=4e2cd32c0d0f93913c4d45e930b2e6722a213114d92ba42f5db2caac0bc13107&
22:53:59 <yiffgirl> hmm
22:54:57 <yiffgirl> [don't worry, i'm not going to make a pr while it's in this state.]