Config
Log for #openttd on 30th June 2023:
Times are UTC Toggle Colours
01:19:34  *** Flygon has joined #openttd
01:33:15  *** Flygon has quit IRC
01:59:27  *** Wormnest has quit IRC
02:26:55  *** Flygon has joined #openttd
02:29:56  *** D-HUND has joined #openttd
02:33:32  *** debdog has quit IRC
03:00:05  *** herms has quit IRC
03:05:18  *** herms has joined #openttd
03:43:03  <DorpsGek> [OpenTTD/team] jacobzheng opened issue #433: [zh_CN] Translator access request https://github.com/OpenTTD/team/issues/433
03:45:07  *** triple_xx has quit IRC
03:45:07  *** jfs has quit IRC
03:45:07  *** frosch123 has quit IRC
03:45:07  *** _glx_ has quit IRC
03:45:07  *** brickblock19280 has quit IRC
03:45:07  *** DorpsGek_v has quit IRC
03:45:07  *** bigyihsuan has quit IRC
03:45:07  *** locosage has quit IRC
03:45:07  *** _jgr_ has quit IRC
03:45:07  *** andythenorth has quit IRC
03:45:07  *** peter1138 has quit IRC
03:45:07  *** truebrain has quit IRC
03:45:07  *** _zephyris has quit IRC
03:45:31  *** DorpsGek_v has joined #openttd
04:09:41  *** keikoz has joined #openttd
04:35:28  *** D-HUND is now known as debdog
06:05:31  *** gelignite has joined #openttd
06:38:22  *** DDR has quit IRC
07:48:12  *** soylent_cow[m] has joined #openttd
07:49:37  <soylent_cow[m]> HI! 🌊
07:51:59  *** _aD has joined #openttd
07:54:31  <LordAro> looks like the bridge fell over (has fallen over?) again
07:58:26  *** peter1138 has joined #openttd
07:58:26  <peter1138> Nope.
08:00:01  <peter1138> Well. I guess it did but nobody's been speaking on Discord all night 😄
08:05:53  <LordAro> very good
08:06:49  <peter1138> Oh, how is it past 9am already 😦
08:08:05  <LordAro> a lot easier when you're on holiday
08:16:59  *** m3henry has quit IRC
08:22:42  *** _aD has quit IRC
08:27:34  <pickpacket> Daily in three minutes. Should probably join the teams meeting now, because I'll have forgotten it again in three minutes
08:33:23  *** truebrain has joined #openttd
08:33:23  <truebrain> poor bridge, did it die again? I feel sorry for it ...
08:35:47  <LordAro> oom?
08:35:51  <truebrain> dunno
08:36:51  <truebrain> yeah, seems that way
08:37:07  <truebrain> it had a huge jump in memory usage in the night of the 29th
08:37:14  <truebrain> I know the Discord connection dropped
08:37:38  <truebrain> so if I would guess, it seems like the discord library is leaking some memory .. either by me not freeing everything properly, or a bug in the library
08:37:42  <truebrain> leaking memory in Python is hard
08:39:45  <truebrain> yeah, every bump in memory I can correlate back to a reconnection to the Discord backend
08:39:49  <truebrain> so something is fishy there 😛
08:39:53  <LordAro> fun
08:40:01  <truebrain> not something for now 🙂
09:15:00  *** tony_pixel has joined #openttd
09:15:00  <tony_pixel> truebrain: Unless you're opening files and not closing them
09:15:19  <truebrain> even then it is hard, as you need to keep the variable alive too!
09:15:37  <truebrain> as soon as the variable leaves scope, it closes the file (well, when the GC comes around, but you get what I mean)
09:15:47  <truebrain> so I am rather puzzled how this Discord library actually leaks ..
09:15:55  <truebrain> it means it keeps a reference alive ..
09:16:56  <peter1138> Won't be long before another C++ rewrite...
09:20:50  <truebrain> truebrain: (well, unless you define `closefd=False` ofc, but if you shoot yourself in the foot .... it tends to hurt :P)
09:21:18  <pickpacket> truebrain: I'm very puzzled to hear of memory leaks in python at all. Have never heard of that happening before
09:21:59  <truebrain> You misunderstand. The Python application leaks. Not Python itself. And of course any application can leak if you do stupid shit 🙂
09:22:09  <pickpacket> true
09:23:04  <pickpacket> weird though because it must mean that the application creates objects that it keeps references to so that the GC doesn't collect them
09:23:31  <truebrain> and welcome to 7 minutes ago 🙂
09:24:43  <pickpacket> lol
09:25:04  * pickpacket is having last-day-before-vacation brain issues
09:26:35  <DorpsGek> [OpenTTD/OpenTTD] bjornwarmedal updated pull request #11076: Changing exclusive transport rights https://github.com/OpenTTD/OpenTTD/pull/11076
09:35:40  <DorpsGek> [OpenTTD/OpenTTD] Kuhnovic updated pull request #9642: Feature: Orientation of rail and road depots can be changed https://github.com/OpenTTD/OpenTTD/pull/9642
09:37:18  <DorpsGek> [OpenTTD/OpenTTD] Kuhnovic commented on pull request #9642: Feature: Orientation of rail and road depots can be changed https://github.com/OpenTTD/OpenTTD/pull/9642#pullrequestreview-1506867959
09:38:33  <DorpsGek> [OpenTTD/OpenTTD] Kuhnovic updated pull request #9642: Feature: Orientation of rail and road depots can be changed https://github.com/OpenTTD/OpenTTD/pull/9642
09:39:12  <DorpsGek> [OpenTTD/OpenTTD] Kuhnovic updated pull request #9642: Feature: Orientation of rail and road depots can be changed https://github.com/OpenTTD/OpenTTD/pull/9642
09:46:24  <DorpsGek> [OpenTTD/workflows] TrueBrain opened pull request #32: Change: [CI] Allow running update-cdn from the GitHub website https://github.com/OpenTTD/workflows/pull/32
09:47:08  <DorpsGek> [OpenTTD/workflows] TrueBrain merged pull request #32: Change: [CI] Allow running update-cdn from the GitHub website https://github.com/OpenTTD/workflows/pull/32
09:52:24  <truebrain> `<botocore.awsrequest.AWSRequest object at 0x7f7bb89e85d0>` .. tnx AWS CLI .. I see you have the best way to report errors .. this is so appreciated ...
09:52:27  <truebrain> (seriously, wtf?)
10:01:18  <LordAro> raise repr(obj)
10:01:30  <LordAro> or possibly just raise obj
10:14:13  <DorpsGek> [OpenTTD/team] glx22 commented on issue #433: [zh_CN] Translator access request https://github.com/OpenTTD/team/issues/433
10:59:59  <peter1138> > ERROR:  operator does not exist: boolean | boolean
11:00:06  <peter1138> Oh come on PostgreSQL...
11:16:14  <pickpacket> peter1138: you didn't know what you do for a living the other day. I'm guessing that whatever it is involves PostgreSQL today :D
11:20:56  <peter1138> Jack of all trades, master of none.
11:22:17  <pickpacket> same here
11:23:41  <pickpacket> manager asked me the other day which of our tasks I feel the most comfortable doing. They're so diverse right now... I don't feel like I *know* how to solve any of them, but I could take on any of them and figure it out
11:25:44  <peter1138> Aright, how do I rewrite EOMONTH()...
11:32:52  <Rubidium_> you don't
11:33:03  <Rubidium_> never write date mangling functions yourself
11:38:06  <truebrain> someone had a nightmare reading that sentence 😄
11:39:57  <LordAro> we once had a bug that would only occur in the week where US DST was applied but UK DST was not
11:40:04  <LordAro> or possibly vice versa
11:41:19  <DorpsGek> [OpenTTD/OpenTTD] bjornwarmedal updated pull request #11076: Changing exclusive transport rights https://github.com/OpenTTD/OpenTTD/pull/11076
11:41:31  <LordAro> iirc it was eventually worked out that msys was setting a unix-like TZ env var value that native Windows programs didn't understand
11:45:43  *** gelignite has quit IRC
11:47:03  <truebrain> timezones ... everyone hates them 😛
11:59:25  *** _aD has joined #openttd
11:59:45  <LordAro> truebrain: is that you asking for extra permissions?
12:00:00  <truebrain> yeah, it is
12:00:03  <truebrain> no clue why it emailed everyone
12:00:05  <truebrain> was a bit pointless
12:00:07  <LordAro> goodo
12:00:24  <DorpsGek> [OpenTTD/workflows] TrueBrain opened pull request #33: Change: [CI] switch to GitHub App for triggering repository-dispatch events https://github.com/OpenTTD/workflows/pull/33
12:00:29  <truebrain> this ^^ will give context to the mail you got 🙂
12:01:38  <DorpsGek> [OpenTTD/workflows] TrueBrain updated pull request #33: Change: [CI] switch to GitHub App for triggering repository-dispatch events https://github.com/OpenTTD/workflows/pull/33
12:02:38  <LordAro> ooh!
12:02:50  <LordAro> does this mean that nightlies will stop "timing out" ?
12:02:58  <truebrain> they time out?
12:03:12  <truebrain> and no, this just changes the use of a PAT to the use of a GitHub App with a private/public key 🙂
12:03:19  <LordAro> doesn't glx have to keep pressing refresh on something?
12:03:36  <truebrain> owh, the workflows repository suspends after 90 days of inactivity
12:03:46  <truebrain> something on my list to address too; but that is not this PR 🙂
12:03:55  <LordAro> ah right
12:05:56  <truebrain> hmm .. I tested 1 of the 4 flows in the PR .. but there is no test like live testing!@
12:06:09  <DorpsGek> [OpenTTD/workflows] TrueBrain merged pull request #33: Change: [CI] switch to GitHub App for triggering repository-dispatch events https://github.com/OpenTTD/workflows/pull/33
12:10:24  <DorpsGek> [OpenTTD/workflows] TrueBrain opened pull request #34: Change: [CI] switch to Cloudflare R2 for CDN https://github.com/OpenTTD/workflows/pull/34
12:11:46  <truebrain> now to make PRs for the other repositories .. will be fun 😛
12:12:03  <LordAro> woop
12:13:58  <truebrain> very hard to test .. not sure how to test it actually .. hmm
12:18:22  *** _glx_ has joined #openttd
12:18:22  <_glx_> this merge just reset the timer I think 🙂
12:18:27  <truebrain> it does
12:18:30  <_glx_> we're fine for 90 days
12:21:15  <_glx_> TrueBrainviaGitHub: draft exist for this reason 😉
12:21:27  <truebrain> true true
12:21:41  <truebrain> it was more a mental note than anything else
12:21:45  <truebrain> you could already review it, if you like 🙂
12:21:50  <truebrain> so in that sence, it is not a draft 😛
12:24:56  <truebrain> https://github.com/OpenTTD/OpenGFX/branches
12:24:56  <truebrain> Are those branches still useful?
12:28:33  <DorpsGek> [OpenTTD/workflows] glx22 commented on pull request #34: Change: [CI] switch to Cloudflare R2 for CDN https://github.com/OpenTTD/workflows/pull/34#pullrequestreview-1507101550
12:29:05  <DorpsGek> [OpenTTD/workflows] TrueBrain commented on pull request #34: Change: [CI] switch to Cloudflare R2 for CDN https://github.com/OpenTTD/workflows/pull/34#pullrequestreview-1507102233
12:37:25  <DorpsGek> [OpenTTD/OpenGFX] TrueBrain opened pull request #84: Change: [CI] update actions to latest and minor touchups https://github.com/OpenTTD/OpenGFX/pull/84
12:38:36  <truebrain> owh, some more stuff to do there ..
12:40:05  <truebrain> hard to test ...
12:40:07  <DorpsGek> [OpenTTD/OpenGFX] TrueBrain updated pull request #84: Change: [CI] update actions to latest and minor touchups https://github.com/OpenTTD/OpenGFX/pull/84
12:41:15  <truebrain> I guess this is the same for OpenMSX and OpenSFX .. I will first make OpenGFX work, move it to the `actions` repository after, making it a reusing workflow
12:42:31  <truebrain> owh, also lacking concurrency statements ..
12:43:37  <DorpsGek> [OpenTTD/OpenGFX] TrueBrain updated pull request #84: Change: [CI] update actions to latest and minor touchups https://github.com/OpenTTD/OpenGFX/pull/84
12:43:44  <truebrain> what else did we miss when updating the actions over the last few years ...
12:44:54  <truebrain> master -> main rename
12:45:06  <truebrain> but that is something for a bit later
12:47:55  <_glx_> wow it was very outdated
12:48:08  <truebrain> I think we both surpressed it existed 😄
12:48:20  <_glx_> it's never used 😉
12:49:01  <pickpacket> I've been thinking that it would be nice to see in the town listing whether or not a one has built a statue in a town or not. What do you guys think?
12:53:05  <DorpsGek> [OpenTTD/OpenGFX] glx22 approved pull request #84: Change: [CI] update actions to latest and minor touchups https://github.com/OpenTTD/OpenGFX/pull/84#pullrequestreview-1507135620
12:53:54  <DorpsGek> [OpenTTD/OpenGFX] TrueBrain merged pull request #84: Change: [CI] update actions to latest and minor touchups https://github.com/OpenTTD/OpenGFX/pull/84
12:54:19  <truebrain> I didn't know about `shell: bash` making `-e` explicit btw; might be nice to do that everywhere
12:55:12  <_glx_> probably
12:55:40  <truebrain> wow, this workflow is kinda weird .. but owh well, when moving to reusing workflow that can be fixed up 🙂 First, getting the CDN to work 😛
12:56:13  <LordAro> truebrain: i'd rather be explicit about it, rather than "bash" implicitly setting it
12:56:30  <_glx_> it's not bash, it's github action
12:56:43  <truebrain> and it would be unwise to be "explicit" about it, as we were doing it wrong 🙂
12:56:51  <truebrain> (it was missing `-ofailpipe` for example)
12:56:58  <LordAro> "shell: bash" suggests bash :p
12:56:59  <truebrain> letting GitHub take care of those nuances is much better 😄
12:57:28  <truebrain> they promise fail-fast when using an explicit shell
12:58:08  <truebrain> (guess the only reason it is not enabled by default as it would break many workflows 😛 )
12:58:45  <_glx_> we explicitely disable fail-fast with matrix
12:58:45  <truebrain> ugh, 6 minutes between runs for testing ... "joy"
12:59:40  <truebrain> off to a bad start .. 3 things failed in the 3rd step alone 😄
13:08:55  <truebrain> second attempt was a lot better .. a silly typo ...
13:12:54  *** HerzogDeXtEr has joined #openttd
13:15:07  <_glx_> progress 🙂
13:15:49  <LordAro> https://www.theregister.com/2023/06/28/microsofts_github_gmp_project/ oof.
13:16:18  <truebrain> hmm .. `mimetype` is not installed on GitHub runners
13:16:19  <truebrain> odd
13:16:48  <LordAro> mimetype? never heard of that
13:17:18  <truebrain> it does exactly what you think it does
13:17:42  <LordAro> quite
13:18:10  <truebrain> when uploading things to an S3, you need to tell the content-type of the object
13:18:13  <LordAro> i'd probably usually use `file` or one of the xdg-things
13:18:16  <truebrain> so when downloading, the browser handles that nicely
13:18:24  <truebrain> `file` has .. similar-ish support, but it is so bad
13:18:28  <truebrain> it doesn't even know markdown files
13:18:35  <LordAro> or just look at the extension :p
13:19:01  <truebrain> I kinda don't want to program in all extensions we have on the CDN, and the ones we will get 😛
13:19:20  <truebrain> in case you missed the memo, we are trying to make things easier for ourselves 😄
13:19:27  <LordAro> :o
13:19:44  <truebrain> seems `file` doesn't even use `/etc/mime.types` .. weird
13:20:27  <_glx_> LordAro: yeah workflows can be "nasty"
13:20:51  <truebrain> hmm .. what package supplies mimetype .. how to figure that out ..
13:21:23  <LordAro> truebrain: https://packages.ubuntu.com/search?searchon=contents&keywords=mimetype&mode=exactfilename&suite=kinetic&arch=any
13:21:54  <truebrain> perl?! owh my ...
13:22:13  <truebrain> did not expect a binary to be in a lib package
13:22:14  <truebrain> odd
13:23:17  <LordAro> perl packages are great
13:23:23  *** nielsm has joined #openttd
13:24:20  <truebrain> let's agree to disagree on that statement 🙂
13:24:36  <LordAro> :p
13:24:39  <truebrain> anyway, tnx for that URL 🙂
13:24:47  <LordAro> they're very well organised
13:25:21  <LordAro> foo::bar::baz perl library is at libfoo-bar-baz-perl
13:26:35  <truebrain> it sounds like you found the one positive thing about Perl? 😛
13:29:20  <LordAro> haha
13:29:34  <LordAro> i find myself liking perl a lot these days
13:29:45  <LordAro> mostly stockholm syndrome i think
13:32:18  *** keikoz has quit IRC
13:40:05  <DorpsGek> [OpenTTD/OpenGFX] TrueBrain opened pull request #85: Change: [CI] Use Cloudflare R2 as CDN https://github.com/OpenTTD/OpenGFX/pull/85
13:41:00  <truebrain> Okay, that seems to work .. now moving it to reusing workflows, and apply it on 3 other repos
13:43:11  *** keikoz has joined #openttd
14:53:46  *** Kitrana has quit IRC
15:13:13  *** Wormnest has joined #openttd
15:31:36  <peter1138> Okay, PostgreSQL is annoying me now.
15:31:45  <peter1138> It is rather stricter than MS SQL.
15:32:19  <peter1138> I recreated the schema but need to make a few tweaks, but doesn't allow changed when indexes, foreign keys or views reference them.
15:32:31  <peter1138> Which is good and proper, but I'm not used to it :p
15:51:29  <LordAro> hehe
16:01:34  <peter1138> LOL
16:01:41  <peter1138> After a full day trying to get this to work...
16:02:02  <peter1138> I now need to make updatable views work, or give up.
16:04:13  <peter1138> https://cdn.discordapp.com/attachments/1008473233844097104/1124369813544898570/image.png
16:04:14  <peter1138> Default font if you change your language to German?
16:05:22  <Rubidium_> does that make the dropdown look weird?
16:06:03  <peter1138> In what way (they're not actually drop downs after all...)
16:07:30  <Rubidium_> just badly aligned
16:07:54  <peter1138> Not badly aligned, just not aligned at all.
16:08:15  <peter1138> Our GUI system doesn't support that natively, so the window would have to manually set up the sizes. And I've not bothered.
16:08:23  <LordAro> peter1138: :D
16:14:32  <pickpacket> LordAro: I’ve worked with well-written object oriented perl code. It was nice
16:22:16  *** Compu has joined #openttd
16:24:11  *** Compu has quit IRC
16:24:15  <peter1138> Strange fantasy you have there.
16:29:22  <DorpsGek> [OpenTTD/OpenTTD] rubidium42 opened pull request #11086: Codechange: use std::string instead of a temporary buffer for iconv calls https://github.com/OpenTTD/OpenTTD/pull/11086
16:30:49  *** Compu has joined #openttd
17:06:09  <DorpsGek> [OpenTTD/OpenTTD] PeterN approved pull request #11086: Codechange: use std::string instead of a temporary buffer for iconv calls https://github.com/OpenTTD/OpenTTD/pull/11086#pullrequestreview-1507570713
17:15:26  <DorpsGek> [OpenTTD/OpenTTD] rubidium42 merged pull request #11086: Codechange: use std::string instead of a temporary buffer for iconv calls https://github.com/OpenTTD/OpenTTD/pull/11086
17:22:00  <pickpacket> pickpacket: imagine all the coders, writing lines of peeeerl... ohoho-o-o. You may say, I'm a dreameee-eer, but I'm not the only one
17:22:05  <pickpacket> bah
2022023-06-30T22:35:12  <talltyler> Psst, #10755 needs somebody to hit merge (or disagree) 🙂
22:48:45  *** tokai has joined #openttd
22:48:45  *** ChanServ sets mode: +v tokai
23:53:14  *** m1cr0m4n has quit IRC
23:57:14  *** m1cr0man has joined #openttd

Powered by YARRSTE version: svn-trunk