Config
Log for #openttd on 26th April 2020:
Times are UTC Toggle Colours
00:28:34  *** Progman has quit IRC
01:13:33  *** Wolf01 has quit IRC
01:43:08  *** Flygon has joined #openttd
02:27:14  *** D-HUND has joined #openttd
02:30:31  *** debdog has quit IRC
03:08:20  *** glx has quit IRC
03:10:36  <DorpsGek_III> [OpenTTD/OpenTTD] James103 opened issue #8098: The transfer value in "Transfer: £[number] / Income: £[number]" can be negative. https://git.io/Jftgp
04:33:38  <Flygon> Well, this's annoying. Moved OpenTTD to my new computer. And whilst everything loads and simulates a lot smoother, the mouse cursor is weirdly choppy.
04:33:44  <Flygon> I wonder what's up. :D
05:52:04  *** andythenorth has joined #openttd
06:20:48  *** Progman has joined #openttd
06:27:08  <andythenorth> yo
06:42:24  *** Samu has joined #openttd
06:44:02  <Samu> i've been wondering what would happen if the other list has items sorted in another manner
07:10:02  <andythenorth> does anyone know if it's 'NML' or 'nml'?
07:10:09  <andythenorth> or is it a pointless question?
07:10:25  <andythenorth> docs dont' know
07:17:20  *** sla_ro|master has joined #openttd
07:26:43  <Samu> I don't know how to use the clone feature http://squirrel-lang.org/doc/squirrel2.html#d0e1264
07:31:50  <Samu> "couldn't detect real instance of class for non-static call", in english plz
07:32:06  <Samu> this._path_tiles = clone this._prev._path_tiles;
07:32:11  <Samu> i did this
07:46:48  <TrueBrain> andythenorth: consistency is for the weak :P :P
07:47:04  <andythenorth> hobgoblin of tiny minds
07:49:14  <DorpsGek_III> [OpenTTD/bananas-frontend-web] TrueBrain approved pull request #28: Fix: Upload entirely new packages failed. https://git.io/JftrJ
07:49:23  <andythenorth> @seen glx
07:49:23  <DorpsGek> andythenorth: glx was last seen in #openttd 9 hours, 25 minutes, and 41 seconds ago: <glx> and used to be a pain for bridges
07:49:46  <DorpsGek_III> [OpenTTD/bananas-frontend-web] TrueBrain approved pull request #29: Fix: Make the navigation bar wider to fit all the categories, and mov… https://git.io/JftrI
07:52:35  <DorpsGek_III> [OpenTTD/nml] andythenorth updated pull request #104: Doc: Changelog for 0.5.0 release https://git.io/JftvS
07:52:42  <andythenorth> FLHerne LordAro ^
07:52:54  <andythenorth> not perfection, but more of a release note than a bad list of commits
07:54:13  *** frosch123 has joined #openttd
07:56:14  <DorpsGek_III> [OpenTTD/nml] andythenorth opened issue #105: Document procedures for switches and random switches https://git.io/Jftrs
07:58:04  <DorpsGek_III> [OpenTTD/bananas-frontend-web] frosch123 merged pull request #28: Fix: Upload entirely new packages failed. https://git.io/Jft8S
07:58:18  <DorpsGek_III> [OpenTTD/bananas-frontend-web] frosch123 merged pull request #29: Fix: Make the navigation bar wider to fit all the categories, and mov… https://git.io/Jft4i
08:01:55  * andythenorth will be glad when all this docs yak shaving is done :)
08:05:30  <frosch123> there are no procedures in random_switches, are there?
08:06:34  <andythenorth> PR says there are https://github.com/OpenTTD/nml/pull/66
08:07:18  <andythenorth> does that work? o_O
08:07:19  <TrueBrain> hmm .. "upload update" link doesn't seem to work for me .. it starts a completely new upload, instead of an update .. hmm
08:07:36  <frosch123> andythenorth: caller vs callee :)
08:08:02  <frosch123> switches and random_switches can be callee, but only switch can be caller
08:08:19  <andythenorth> yes
08:08:26  <TrueBrain> owh, ofc, I need to upload something before it sees that .. lol .. forgot about that ..
08:08:31  <andythenorth> sorry, I hadn't even thought the other route would work :)
08:09:16  * andythenorth adjusts GH issue
08:09:19  <frosch123> TrueBrain: yes, you need to click "validate" once, after uploading files
08:09:40  <TrueBrain> so uploading new vs uploading update has no real difference :D
08:09:50  <frosch123> i thought about adding a second "validate" button after the upload stuff, that is labeled "autofill fields"
08:10:03  <frosch123> TrueBrain: no, it's the same until the api supports it :)
08:10:51  <andythenorth> I was thinking of something like 'Note that a random switch can also be called as a procedure by a switch; see [switch page]'
08:10:59  <andythenorth> worthwhile, or junk?
08:11:28  <frosch123> andythenorth: no idea, i assumed procedures would be documented where they are used, at the caller side
08:11:33  <TrueBrain> we need someone with Javascript knowledge .. I want to be able to drag/drop files :D
08:11:49  <frosch123> stackoverflow probably knows :)
08:11:52  <andythenorth> later
08:12:23  <frosch123> hmm, my changes are still not deployed?
08:12:26  <andythenorth> javascript murders productivity
08:13:10  <TrueBrain> ah, I see where "name" goes wrong .. I wonder how that come to be .. hmm .. interesting
08:13:45  <TrueBrain> basically, it never supported you not filling in a name :D Lets see ..
08:14:05  <frosch123> i guess merging two prs in succession breaks it :) the first pr is deployed, and the second one is blocked since deploy is already running?
08:14:54  <TrueBrain> "An error occurred (ValidationError) when calling the UpdateStack operation: Stack:arn:aws:cloudformation:***:***:stack/Live-Staging-BananasFrontendWeb/cfbb7080-8161-11ea-a132-02b8022d5b40 is in UPDATE_IN_PROGRESS state and can not be updated."
08:15:03  <TrueBrain> as shown by the GitHub Actions result :)
08:15:29  <TrueBrain> yes; merging a second PR before the first is deployed will always return that
08:15:35  <TrueBrain> I considered it unlikely enoug hto not care :P
08:15:40  <TrueBrain> just his the "re-run jobs"
08:15:43  <frosch123> well, we'll find another thing to fix :)
08:15:46  <TrueBrain> https://github.com/OpenTTD/bananas-frontend-web/runs/619342785?check_suite_focus=true
08:16:24  <frosch123> ok, pressed, let's see
08:16:29  <TrueBrain> at first I considered waiting for the cloudformation to be done, but that means things can break if you merge 3 PRs shortly after each other
08:16:31  <TrueBrain> so .. meh :P
08:16:39  <TrueBrain> unlikely events being unlikely
08:19:50  <Samu> the AIList method is bad after all :( https://i.imgur.com/Gb6OPjC.png
08:20:02  <Samu> memory usage is absurdly high
08:20:44  <Samu> cpu time is high
08:21:12  <frosch123> yay, worked, no more broken navigator
08:21:15  <Samu> it's sad, really, the number of ticks are so good though
08:21:31  <frosch123> also less weird empty space at top :p
08:22:08  <TrueBrain> nice :D
08:22:42  <TrueBrain> okay, this is odd .. I upload files, I hit validate, files gone
08:22:43  <TrueBrain> lol
08:23:29  <frosch123> before validate the js adds the files, validate asks the api for present files
08:23:40  <TrueBrain> yeah .. and poef .. gone :)
08:23:43  <TrueBrain> this was working a moment ago
08:24:35  <TrueBrain> owh, it is the JS client caching stuff
08:24:43  *** nielsm has joined #openttd
08:24:44  <TrueBrain> "you already uploaded this file so I am not realllllyyyy going to"
08:24:53  <TrueBrain> we should figure out how to fix that, tbh
08:25:17  <frosch123> ah, tusd does that
08:25:24  <frosch123> it "resumes" uploads :)
08:25:53  <frosch123> we need an api call to add files then
08:26:17  <frosch123> i know the file uuid, so i can tell the api to check for missing files
08:26:55  <frosch123> s/i/the frontend/
08:27:55  <TrueBrain> well, tusd, as we use it, doesn't really allow "resuming" after it restarted
08:28:01  <TrueBrain> but the client still things it is the same tusd
08:28:08  <Samu> which one do you think is less cpu intensive? this._path_tiles.extend(this._prev._path_tiles); or this._path_tiles = clone this._prev._path_tiles;
08:28:12  <TrueBrain> but it is also because the file remained on disk
08:28:19  <TrueBrain> so I guess this works out in production fine
08:28:19  <TrueBrain> meh
08:28:37  <TrueBrain> the javascript client keeps things in the localstore
08:28:40  <TrueBrain> localstorage
08:28:45  <TrueBrain> which allows it to check for resumes
08:29:19  <TrueBrain> this is a bit the problem with this whole implementation of tusd .. so few nobs to turn ..
08:31:15  <TrueBrain> hmm .. 7zip doesn't like the tarballs we create now ..
08:31:17  <TrueBrain> not sure what is going on
08:32:42  <TrueBrain> ah, default format changed
08:32:43  <TrueBrain> makes sense
08:34:48  <frosch123> https://github.com/tus/tus-js-client#tusdefaultoptions <- in that case we can try setting resume=false
08:35:41  <TrueBrain> I think, at least for now, that is best
08:36:11  <TrueBrain> removeFingerprintOnSuccess to true should also be enough, possibly
08:36:19  <TrueBrain> but as the docs say, we should set that to true anyway :D
08:36:34  <TrueBrain> lovely, these settings they strongly advise you to change .. :P
08:38:39  *** mcbanhas has joined #openttd
08:39:03  <mcbanhas> Henlo
08:39:46  <TrueBrain> okay, we were creating PAX tarballs .. not sure what OpenTTD would have done with those :P
08:39:51  <TrueBrain> we are back to GNU tarballs .. lot better :)
08:40:19  <Samu> looks like extend is less memory intensive than clone
08:43:04  <frosch123> ottd expects "ustar", but also supports something more ancient
08:43:29  <TrueBrain> ustar? That is also very old ..
08:43:49  <frosch123>  /* Check if we have the new tar-format (ustar) or the old one (a lot of zeros after 'link' field) */
08:44:16  <frosch123> apparently it's the "new tar-format" :)
08:44:34  <TrueBrain> 1991 new :P
08:44:48  <TrueBrain> PAX is 2001
08:44:49  <TrueBrain> lol
08:44:54  <TrueBrain> k, ustar it is
08:46:33  <DorpsGek_III> [OpenTTD/OpenTTD] James103 opened issue #8099: UI elements (incl. news message) do not resize properly, causing graphical glitches https://git.io/JftoL
08:50:03  <TrueBrain> right .. I was going to fix something else .. I was like: shall I make a ticket, I was like, no I can remember this, and now I forgot about it
08:51:03  <DorpsGek_III> [OpenTTD/nml] LordAro commented on pull request #104: Doc: Changelog for 0.5.0 release https://git.io/JftoO
08:52:23  <TrueBrain> that is what we call a stack-corruption
08:53:53  <DorpsGek_III> [OpenTTD/nml] frosch123 commented on pull request #104: Doc: Changelog for 0.5.0 release https://git.io/JftoG
08:54:26  <DorpsGek_III> [OpenTTD/bananas-api] TrueBrain opened pull request #44: Early-in-the-morning combined fixes https://git.io/Jfton
09:00:45  <DorpsGek_III> [OpenTTD/bananas-api] TrueBrain updated pull request #44: Early-in-the-morning combined fixes https://git.io/Jfton
09:00:47  <TrueBrain> I REMEMBERED! :D
09:00:55  <TrueBrain> oops, should have been a new PR. Owh well
09:01:45  <TrueBrain> right, next up: reloading in a save way ..
09:02:08  <TrueBrain> frosch123: I have been thinking to add an API endpoint that shows all the things you have pending for commit, so people understand it takes a bit of time before it lands live
09:02:16  <TrueBrain> and possibly a button to publish
09:02:29  <TrueBrain> but that is maybe for the wishlist
09:03:21  <DorpsGek_III> [OpenTTD/bananas-api] TrueBrain opened issue #45: Add an API endpoint to show the pending changes https://git.io/Jfto2
09:03:30  <TrueBrain> and all this, for as you point out, for ~200 users :P
09:04:54  <frosch123> can i nitpick on commit messages? :p
09:05:13  <frosch123> As GitHub doesn't track folders, only files, we need to do some manual cleanup. <- git, not github
09:06:45  <frosch123> does os.walk survive, if you delete folders while iterating?
09:09:33  <TrueBrain> you for sure can nitpick, will fix
09:09:53  <TrueBrain> I tested it; os.walk() does survive .. didn't even do what I wanted him to do .. in result I have to walk a few times
09:10:00  <TrueBrain> seems it doesn't really return an iterator
09:10:33  <TrueBrain> it is a generator
09:10:35  <TrueBrain> so it is a bit odd, tbh
09:14:06  <TrueBrain> no, I guess it makes sense
09:14:11  <TrueBrain> to find the folders it already listed the parent
09:14:15  <TrueBrain> no need to relist it again
09:14:18  <TrueBrain> so yeah, this should be good
09:15:19  <DorpsGek_III> [OpenTTD/bananas-api] TrueBrain updated pull request #44: Early-in-the-morning combined fixes https://git.io/Jfton
09:15:24  <TrueBrain> frosch123: a free extra commit, and fixed the commit message :)
09:19:11  <DorpsGek_III> [OpenTTD/bananas-api] frosch123 approved pull request #44: Early-in-the-morning combined fixes https://git.io/Jfto5
09:19:18  <frosch123> bonus commits :)
09:21:38  <TrueBrain> frosch123: from the API, do you want the full license, or just the name?
09:22:23  <frosch123> for what?
09:22:43  <TrueBrain> you wanted to be able to fetch the licenses from the API
09:22:48  <TrueBrain> so they are no longer hardcoded
09:22:59  <TrueBrain> I wonder what the API should return
09:23:01  <TrueBrain> just "GPL v2"
09:23:02  <TrueBrain> the text
09:23:04  <TrueBrain> a link
09:23:26  <frosch123> ah, a dict of (enum-value, user-readable-text, deprecated-bool)
09:23:50  <TrueBrain> licenses are HUGE
09:24:03  <TrueBrain> owh, "user-readable-text" is just "GPL v2"?
09:24:13  <TrueBrain> (your example is still .. pretty open to interpertation :D)
09:24:29  <frosch123> user-readable text can be the same as enum-value (see https://github.com/OpenTTD/bananas-frontend-web/issues/23 ), and deprecated would not be offered for new packages
09:25:22  <TrueBrain> you are still leaving a lot of dots for me to connect, but okay :)
09:25:26  <planetmaker> @logs
09:25:26  <DorpsGek> planetmaker: https://webster.openttdcoop.org/index.php?channel=openttd
09:25:34  <TrueBrain> for licenses, there is no difference between "enum" value and user-readable
09:26:07  <TrueBrain> so I guess: {"name": "GPLv2", "deprecated": False} is sufficient?
09:26:13  <frosch123> TrueBrain: there is a another feature request to get a link to the readme/changelog/license.txt of a package (https://github.com/OpenTTD/bananas-api/issues/27)
09:26:24  <TrueBrain> but that is after upload :)
09:26:46  <frosch123> TrueBrain: yes, as said in #23, "GNU General Public license, version 2" is too long to be useful
09:27:28  *** Wolf01 has joined #openttd
09:27:33  <TrueBrain> I am doing https://github.com/OpenTTD/bananas-api/issues/32, not #23 btw :P Just to be clear :D
09:28:10  <frosch123> yeah, but pick a response format that fits all of frontend's #23
09:28:34  <TrueBrain> the "ai" -> "AI" mapping sounds like a frontend thing more than an API?
09:29:02  <TrueBrain> hmm .. I could make "content-type" dynamic, I guess
09:29:24  <frosch123> well, yes, the frontend won't create the nav header dynamically :)
09:29:42  <TrueBrain> why not?
09:29:48  <frosch123> but i do not want to duplicate the supported branches
09:29:50  <TrueBrain> (honest question)
09:30:03  <frosch123> TrueBrain: because there are some thoughts in the order they are displayed
09:30:12  <frosch123> it's not alphabetic
09:30:16  <TrueBrain> ack
09:30:18  <frosch123> (or is it by accident? :)
09:30:39  <TrueBrain> in that case, lets not add it to the API just yet
09:31:13  <TrueBrain> I updated the swagger API with 3 endpoints, /config series
09:31:18  <TrueBrain> can you take a look before I implement them?
09:31:57  <frosch123> the term "audience" is still funny to me :)
09:32:10  <TrueBrain> I agree
09:32:32  <TrueBrain> but .. the only wiggle room I had while still following OAuth specs :)
09:32:38  <frosch123> will users understand the meaning of "master"?
09:33:05  <TrueBrain> reload? Better?
09:33:34  <frosch123> yep :)
09:35:46  <frosch123> want to add #42 to /config/user-audience ?
09:36:24  <TrueBrain> hmm .. it won't be dynamic
09:36:35  <TrueBrain> so I think it belongs in /user
09:36:49  <DorpsGek_III> [OpenTTD/bananas-api] TrueBrain merged pull request #44: Early-in-the-morning combined fixes https://git.io/Jfton
09:36:55  <frosch123> yeah, there's no guarantee that it will be the same for all audiences
09:36:57  <DorpsGek_III> [OpenTTD/bananas-api] TrueBrain closed issue #43: Publishing updates fails https://git.io/Jft8K
09:37:13  <frosch123> just for gh it's a the same url for all users
09:38:05  <TrueBrain> owh, it is based on the application-id, ofc
09:38:08  <TrueBrain> euh
09:38:13  <TrueBrain> yeah, lets add it to config in that case
09:40:10  <frosch123> however, does the frontend know how the user logged in?
09:41:12  <frosch123> currently the front-end does not need to know what audiences are available
09:42:18  <TrueBrain> well, no, currently it is forced to one :)
09:42:19  <TrueBrain> by config
09:42:27  <TrueBrain> the frontend tells the API which audience to use
09:42:33  <TrueBrain> so yes, the frontend does know that :)
09:43:11  <frosch123> oh, i forgot that
09:43:21  <frosch123> all good then :)
09:43:47  <frosch123> i can display all the buttons "Login to xyz" then
09:44:10  <TrueBrain> that was the plan; a page when you click Manager (and you are not logged in) which shows all the buttons you can use to login
09:44:16  <TrueBrain> so no longer hard-code the frontend :)
09:45:06  <frosch123> i was so used to the developer login page of the api, that i thought the api would decide which flow to use :)
10:04:25  <DorpsGek_III> [OpenTTD/bananas-api] TrueBrain opened pull request #46: Add: /config endpoints with info about user-audiences, licenses and branches https://git.io/JftPf
10:04:39  <TrueBrain> frosch123: will you make the changes to the frontend, or do you want me to do so? :) (just to avoid us both doing it :P)
10:05:24  <DorpsGek_III> [OpenTTD/bananas-api] TrueBrain updated pull request #46: Add: /config endpoints with info about user-audiences, licenses and branches https://git.io/JftPf
10:09:05  <DorpsGek_III> [OpenTTD/bananas-api] TrueBrain updated pull request #46: Add: /config endpoints with info about user-audiences, licenses and branches https://git.io/JftPf
10:09:14  <TrueBrain> bye OpenTTD login flow .. you will never exist ..
10:12:17  *** tokai has joined #openttd
10:12:17  *** ChanServ sets mode: +v tokai
10:19:10  *** tokai|noir has quit IRC
10:53:03  <frosch123> TrueBrain: i would do it, unless you already started now :)
10:53:50  <andythenorth> so nml release or rewrite changelog? o_O
10:54:00  <frosch123> i like the changelog
10:54:17  <frosch123> no idea whether anything else is blocking the release
10:55:31  <TrueBrain> Go for it frosch123 ; I am enjoying a lunch :D
10:55:56  * andythenorth wants lunch
10:56:11  <andythenorth> nml changelog has requested changes, so can't be merged
10:56:35  <andythenorth> https://github.com/OpenTTD/nml/pull/104
10:56:55  <DorpsGek_III> [OpenTTD/nml] frosch123 approved pull request #104: Doc: Changelog for 0.5.0 release https://git.io/JftXH
10:57:17  <andythenorth> merge is blocked though
10:57:23  <andythenorth> (thanks also)
10:58:23  <DorpsGek_III> [OpenTTD/nml] frosch123 dismissed a review for pull request #104: Doc: Changelog for 0.5.0 release https://git.io/Jftkw
10:58:30  <andythenorth> looks like github bug
10:58:39  <andythenorth> or just hard problems :P
11:00:28  <DorpsGek_III> [OpenTTD/nml] andythenorth merged pull request #104: Doc: Changelog for 0.5.0 release https://git.io/JftvS
11:02:23  <andythenorth> we release nml from master?
11:02:33  <DorpsGek_III> [OpenTTD/bananas-api] frosch123 commented on pull request #46: Add: /config endpoints with info about user-audiences, licenses and branches https://git.io/JftXA
11:02:35  <andythenorth> or instructions are wrong, and I should create a 0.5 branch?
11:02:42  * andythenorth looking at the past
11:02:50  <frosch123> from master
11:03:00  <frosch123> noone is going to maintain a branch
11:03:12  <frosch123> nml 0.2 was branched because 0.3 was incompatible
11:03:35  <frosch123> so if anything, one would create a 0.4 branch from whenever we broke stuff
11:03:52  <frosch123> but then, noone is going to add anything to 0.4, so the tags are good enough
11:04:02  <andythenorth> I have tagged 0.5.0
11:05:50  * andythenorth downloads zip
11:05:58  <andythenorth> README.md is wrong in the published zip
11:06:10  <andythenorth> Last updated:    2019-04-02
11:06:10  <andythenorth> Release version: 0.4.5
11:06:37  <andythenorth> hard-coded
11:06:38  <frosch123> ah, we removed that silly date from ottd :)
11:06:56  <andythenorth> oof
11:07:13  <frosch123> make a pr to remoe that line from master
11:07:18  <frosch123> don't care for 0.5 :)
11:08:18  <frosch123> blathijs: nml release is done
11:08:50  <DorpsGek_III> [OpenTTD/nml] andythenorth opened pull request #106: Doc: remove hard-coded release date and version number from README https://git.io/Jft1m
11:09:12  <DorpsGek_III> [OpenTTD/nml] frosch123 approved pull request #106: Doc: remove hard-coded release date and version number from README https://git.io/Jft1Y
11:09:20  <andythenorth> I just have to a few more docs then I can be free of nml for a bit :)
11:09:38  <andythenorth> great tool, easy to work on the code, PITA to maintain the docs :)
11:10:04  <andythenorth> nml is like a happy dog that occasionally brings home rotting dead birds
11:10:47  <frosch123> isn't that a cat thing?
11:11:01  <frosch123> or is the difference that cats bring fresh ones?
11:11:24  <andythenorth> maybe a dog that rolls in cow pats
11:11:38  <andythenorth> great dog, but sometimes you have to clean the house and it smells for days
11:16:47  <frosch123> blathijs: the bundles are on https://pypi.org/project/nml/0.5.0/ and https://github.com/OpenTTD/nml/releases/tag/0.5.0
11:22:08  <TrueBrain> gratz guys in releasing NML :)
11:25:16  <DorpsGek_III> [OpenTTD/bananas-api] TrueBrain updated pull request #46: Add: /config endpoints with info about user-audiences, licenses and branches https://git.io/JftPf
11:28:46  <DorpsGek_III> [OpenTTD/bananas-api] TrueBrain updated pull request #46: Add: /config endpoints with info about user-audiences, licenses and branches https://git.io/JftPf
11:28:50  <TrueBrain> did not know 'assert' was a keyword
11:29:53  *** gelignite has joined #openttd
11:30:51  <frosch123> assert all(license in LICENSES for license in License) <- haha, a great line to illustrate case sensitivity
11:31:07  <DorpsGek_III> [OpenTTD/bananas-api] frosch123 approved pull request #46: Add: /config endpoints with info about user-audiences, licenses and branches https://git.io/Jft1Q
11:32:33  <TrueBrain> frosch123: I very much enjoyed writing that line, I have to say :)
11:34:37  <DorpsGek_III> [OpenTTD/bananas-server] TrueBrain opened issue #8: Content-id are not deterministic over runs https://git.io/Jft1F
11:36:31  * andythenorth todo: finish NML docs, do a forum release announcement
11:39:47  <DorpsGek_III> [OpenTTD/bananas-api] TrueBrain closed issue #42: API endpoint to get URL for OAuth access review https://git.io/JftWr
11:39:47  <DorpsGek_III> [OpenTTD/bananas-api] TrueBrain closed issue #32: API endpoints to get list of licenses and branches https://git.io/Jfktr
11:39:51  <DorpsGek_III> [OpenTTD/bananas-api] TrueBrain merged pull request #46: Add: /config endpoints with info about user-audiences, licenses and branches https://git.io/JftPf
11:42:38  <mcbanhas> Guys, anyone knows if it's possible to change text alignment via a config file or something similar? I can't find anything of the sort in the .lng file documentation.
11:43:00  <mcbanhas> I would like to experiment with left-aligned tooltips
11:43:41  <frosch123> no, you need to compile the source for that
11:43:56  <frosch123> it would also change all tooltips
11:44:07  <mcbanhas> It's for testing purposes
11:44:15  <mcbanhas> So that's fine
11:44:16  <andythenorth> it's witty that our tooltips are long enough to require alignment :)
11:44:18  <DorpsGek_III> [OpenTTD/OpenTTD] nielsmh commented on issue #8098: The transfer value in "Transfer: £[number]" can be negative. https://git.io/Jftgp
11:44:34  <mcbanhas> How can I do it?
11:45:13  <frosch123> can you compile?
11:45:23  <mcbanhas> Yes
11:45:53  <frosch123> misc_gui.cpp:735  remove the SA_CENTER
11:47:37  <DorpsGek_III> [OpenTTD/nml] andythenorth closed issue #43: Release nml 0.5.x to provide NotRoadTypes support for nml authors https://git.io/Je8v7
11:48:36  * andythenorth needs to find the link to NML Windows binaries 
11:48:43  <andythenorth> for forum users
11:48:56  <andythenorth> many of whom won't be able to pip
11:49:14  <mcbanhas> Alright thanks, I'll have a look.
11:49:58  <mcbanhas> Essentially, I want to see how modern tooltip formatting would look in-game.
11:50:01  <andythenorth> ok so Windows binary is here https://github.com/OpenTTD/nml/actions/runs/88193567
11:50:07  <andythenorth> is that a safe place to link to?
11:50:18  <frosch123> i expected it on the release page
11:50:18  <andythenorth> safe / sustainable
11:50:29  <frosch123> maybe LordAro or TrueBrain know how that is supposed to work
11:50:51  <frosch123> does it need manual adding? or was it supposed to add stuff on its own?
11:50:56  <andythenorth> I'd have to upload the artefact manually to releases
11:51:06  <andythenorth> I'm surprised if that's the intended design
11:51:51  <TrueBrain> andythenorth: never link to an action directly; they expire afte a few days
11:52:21  <andythenorth> ok, so for today, download the artefact and attach to the release manually?
11:52:27  <TrueBrain> I don't know much about the rest; just that the windows whl is published on https://pypi.org/project/nml/#files
11:52:34  <TrueBrain> so "pip install nml" will work, also on Windows
11:52:50  <TrueBrain> you have to ask LordAro about anything else :D
11:53:31  <TrueBrain> just know Actions can also publish files in releases, if that is helpful in any way
11:54:04  <andythenorth> maybe that's intended, or it was just an oversight
11:54:13  <andythenorth> I've done it manually for today
11:54:29  <DorpsGek_III> [OpenTTD/OpenTTD] nielsmh commented on issue #8099: UI elements (incl. news message) do not resize properly, causing graphical glitches https://git.io/JftoL
11:54:52  <TrueBrain> most people also attach the whls to the release page btw
11:55:40  <TrueBrain> BaNaNaS now consumes the most amount of resources on AWS :P Also in terms of deployment size :D
11:55:47  <TrueBrain> we might have overdone this a bit 15 years ago :P
11:57:18  <andythenorth> one day I automate converting changelog lists to phpbb format :x https://www.tt-forums.net/viewtopic.php?f=68&t=86972&p=1231367#p1231367
11:57:43  <andythenorth> ouch time for €1 / download TrueBrain
11:57:50  <andythenorth> or flat pass, monthly
11:58:56  <TrueBrain> to pay for my time, sure :P
12:00:19  <frosch123> andythenorth: do you take bets how many people will reply?
12:00:33  <andythenorth> 3
12:00:58  <andythenorth> do we take bets that turns into the general NML questions thread?
12:01:23  <frosch123> haha :)
12:16:15  <andythenorth> thanks nielsm
12:17:44  <planetmaker> :) nice work, guys :) tyvm
12:18:00  <DorpsGek_III> [OpenTTD/OpenTTD] James103 opened issue #8100: Inconsistent target framerate (30 vs 33 fps) https://git.io/JftD2
12:18:33  *** glx has joined #openttd
12:18:33  *** ChanServ sets mode: +v glx
12:18:42  <heffer> Updates for nml on Fedora built an pushed. Now they get to ferment in updates-testing for a week.
12:21:08  <Samu> michi_cc, caching via AIList consumes too much RAM after all
12:21:20  <Samu> just discovered a few hours ago
12:21:41  <frosch123> thanks :)
12:22:32  <DorpsGek_III> [OpenTTD/OpenTTD] nielsmh commented on issue #8100: Inconsistent target framerate (30 vs 33 fps) https://git.io/JftD2
12:22:59  <Samu> I'm currently testing with a priority queue that is reversed. it prioritizes worst paths first
12:23:25  <Samu> quickly reaches 12 GB RAM in a minute
12:24:02  <DorpsGek_III> [OpenTTD/OpenTTD] James103 commented on issue #8100: Inconsistent target framerate (30 vs 33 fps) https://git.io/JftD2
12:26:42  <michi_cc> Samu: I made a PR anyway, the code still seems overly expensive to me no matter if it helps your current thing.
12:27:19  <michi_cc> Somebody would need to approve them though, otherwise they just sit there.
12:27:48  <Samu> oh snap, this crashed 1.10.1
12:27:53  <Samu> i need the pdb thingy
12:29:16  <Samu> Unhandled exception at 0x00007FF66CF44C66 in openttd.exe: 0xC0000005: Access violation writing location 0x000000E2C63FFFF0. occurred
12:31:03  <glx> it's on the download page
12:31:13  <glx> as always
12:31:21  <DorpsGek_III> [OpenTTD/OpenTTD] nielsmh commented on issue #8100: Inconsistent target framerate (30 vs 33 fps) https://git.io/JftD2
12:31:27  <Samu> >	openttd.exe!SQInstance::`scalar deleting destructor'(unsigned int)	C++
12:31:32  <Samu> crashed here
12:32:07  <Samu> the only thing listed in the call stack
12:32:19  <glx> stack overflow then
12:32:49  <glx> probably the usual memory allocation issue
12:33:14  <Samu> I guess it was caused by prioritizing worst paths first :p
12:33:42  <Samu> it zig-zags paths the most it can, it's like playing snake, covering the whole map without repeating
12:34:05  <Samu> too many open nodes at once?
12:34:11  <glx> crazy idea for a pathfinder
12:34:31  <Samu> yeah, i was testing the extreme
12:34:36  <Samu> see how much ram it would use
12:38:21  <DorpsGek_III> [OpenTTD/OpenTTD] nielsmh commented on issue #8100: Inconsistent target framerate (30 vs 33 fps) https://git.io/JftD2
12:39:10  <Samu> d:\a\s\srcrdparty\squirrel\squirrel\sqstate.cpp - says I dont have this file?
12:39:26  <Samu> well, not in that path, but...
12:39:52  <glx> your source is in 1.10.1 tag ?
12:40:07  <Samu> nop
12:40:17  <Samu> Unhandled exception thrown: write access violation.
12:40:18  <Samu> o._unVal.pTable->**** was 0xA28E8AFFD8. occurred
12:40:37  <Samu> openttd.exe!SQSharedState::MarkObject(SQObjectPtr & o, SQCollectable * * chain) Line 245
12:40:37  <Samu> 	at d:\a\s\srcrdparty\squirrel\squirrel\sqstate.cpp(245)
12:40:40  *** Progman has quit IRC
12:40:43  <glx> you need to "git switch 1.10.1"
12:41:07  <glx> yes that's the typical memory fail
12:41:10  <Samu> this time it was an array
12:41:16  <Samu> a squirrel array
12:41:32  <glx> garbage colletction issue as always
12:41:58  <Samu> it crashed in like 30 seconds, seems that the reversed priority is good at picking crashes
12:42:18  <glx> yes but it's unfixable
12:42:53  <glx> well a solution probably exists but...
12:43:00  <Samu> gonna test one more time, now with the original method, which is... i dunno what to call it
12:45:44  <Samu> crashed in 00:00:46
12:45:46  <Samu> seconds
12:45:56  <DorpsGek_III> [OpenTTD/OpenTTD] James103 commented on issue #8100: Empty game runs too slow on Windows https://git.io/JftD2
12:46:24  <Samu> >	openttd.exe!SQSharedState::MarkObject(SQObjectPtr & o, SQCollectable * * chain) Line 245	C++
12:46:27  <Samu> same error
12:47:32  <nielsm> glx: I'm stumped on that slowness issue, any ideas?
12:51:10  <Samu> i get the usual 0.98x-0.99x
12:51:23  <Samu> 32ish fps
12:51:43  <DorpsGek_III> [OpenTTD/OpenTTD] nielsmh commented on issue #8100: Empty game runs too slow on Windows https://git.io/JftD2
12:52:46  <nielsm> it sounds like something wrong with the system timer on his system, I'm not sure what could be happening otherwise
12:53:03  *** WormnestAndroid has quit IRC
12:53:05  <Samu> cpu running at low clocks?
12:53:12  <Samu> let me try power save mode here
12:53:16  *** WormnestAndroid has joined #openttd
12:53:24  <Samu> if a bulldozer could save power...
12:56:20  <Samu> 0.98x at 1.37Ghz
12:56:24  <Samu> still fine
12:56:34  <DorpsGek_III> [OpenTTD/OpenTTD] James103 commented on issue #8100: Empty game runs too slow on Windows https://git.io/JftD2
12:59:45  <Samu> I thought intel was good
12:59:51  <Samu> those results are odd
13:00:18  <TrueBrain> okay ... I think the server should now auto-reload on changes to the BaNaNaS GitHub ..
13:00:20  <TrueBrain> fingers crossed
13:04:59  <TrueBrain> the 1.10 title screen really is a nice title screen; too bad of the maglev, but otherwise .. lot of things to spot and see :)
13:05:37  <glx> Samu: it was, now it's amd ryzen
13:06:09  *** syr has quit IRC
13:07:33  <Samu> https://i.imgur.com/l62hu4L.png
13:07:45  <TrueBrain> YES! It works :D If you now change something no https://bananas.staging.openttd.org/, you can see that via the staging content-service ingame after ~15 minutes :D
13:07:50  <TrueBrain> AUTOMAGIC :D
13:07:52  <Samu> 0.98x, 32.xx fps
13:07:58  <TrueBrain> okay, really happy with that :)
13:08:02  <TrueBrain> frosch123: what else was on the list of things to do?
13:08:44  *** syr has joined #openttd
13:09:07  <Samu> wow, it almost looks like the finished product
13:12:46  <frosch123> write a nice paragraph about how to get access to old logins?
13:13:13  <frosch123> from the issues, the most noticeable ones (since they worked with old bananas) are "upload zip/tar" and "download link"
13:16:06  *** syr has quit IRC
13:16:09  <TrueBrain> besides the paragraph about old logins, is there anything blocking us from release now?
13:16:58  <TrueBrain> bah; CDK fixed something that broke my configuration .. this happens sadly more than I would like that to happen ..
13:17:19  <planetmaker> right, so how to login indeed? :D
13:17:38  <frosch123> TrueBrain: no, just some kind of announcement :)
13:17:57  <TrueBrain> frosch123: good; guess we need to let some people test and see if they can break it
13:18:07  <TrueBrain> I have some infra things to check out, likecurrently the installer will break if we switch over :P
13:18:28  <frosch123> TrueBrain: i wasn't able to test the blacklist. it looked like devauth allows everything
13:18:40  <TrueBrain> euh ... yes ... I never wrote that part :D
13:18:44  <TrueBrain> good point :P
13:18:57  <TrueBrain> it reads the blacklist, it just never checks against it
13:19:17  <frosch123> in that case: i also could update packages owned by others
13:19:19  <TrueBrain> so my list: unbreak CDK, write paragraph, prepare installer CDN, fix blacklist issue, validate we are migrating everything correctly, add upload zip/tar :)
13:19:26  <TrueBrain> that should be prevented
13:19:31  <TrueBrain> there is code that should disallow that
13:19:40  <TrueBrain> (a blacklisted package doesn't really exist; that is where it is different)
13:20:08  <TrueBrain> session["errors"].append("You do not have permission to upload a new version for this package.")
13:20:37  <glx> TrueBrain: people will of course break it, the question is "will sentry catch the breaks ?" ;)
13:20:54  <TrueBrain> glx: sentry is picking bugs up nicely atm, but yeah :D
13:21:04  <TrueBrain> I put some effort in sentry reporting correctly, so I hope so :P
13:21:54  *** Yexo_ has joined #openttd
13:21:56  <Samu> if i put stuff in bananas staging, will it go to finalized product too?
13:22:02  <planetmaker> o_O
13:22:23  <planetmaker> that is a name I haven't seen in a decade
13:22:25  *** Yexo is now known as Guest23408
13:22:25  *** Yexo_ is now known as Yexo
13:22:31  <TrueBrain> Samu: absolutely not. It is staging. Data on staging never moves to production
13:22:50  <Yexo> Afternoon all :)
13:22:58  <planetmaker> moin!
13:23:44  <Samu> hi
13:23:50  <frosch123> hoi :)
13:24:48  <Yexo> Yes, got busy with work, life. Didn´t make the time to keep up
13:25:01  <Yexo> Good to OpenTTD is still very much alive
13:25:08  <frosch123> TrueBrain: ok, ownership check works. just blacklist then
13:25:31  <TrueBrain> frosch123: ack
13:25:47  <TrueBrain> hi Yexo :) "alive" might be a bit overstating, but surviving :P :P :P
13:26:33  <Yexo> ¨Not completely abandoned¨ at least
13:27:24  <frosch123> planetmaker: https://bananas.staging.openttd.org/ <- do you know that url? (not sure what your login question was about)
13:27:47  <planetmaker> yes, I saw that URL
13:27:54  <planetmaker> and referred to that, yes
13:28:24  <frosch123> there are two login links on the frontpage?
13:28:33  <frosch123> actually 3
13:29:02  <planetmaker> it has... one? "login via github"
13:29:12  <planetmaker> and TOS. And "upload updates or new content"
13:29:14  <frosch123> "manager" and "upload ..." also works
13:29:33  <planetmaker> I loged in via github fine, yes
13:30:42  <planetmaker> So that's not something which contains the real packages, so most of it is empty, yes?
13:30:51  *** syr has joined #openttd
13:30:57  <frosch123> no, it's the sandbox
13:31:03  <milek7> probably intended, but logout doesn't quite works
13:31:07  <frosch123> for testing, people are supposed to upload shit :)
13:31:15  <planetmaker> he, k
13:31:29  <planetmaker> yeah, logout is... interesting. I am always logged-in again when I click either link
13:31:32  <milek7> clicking manager logs immadiately with same account
13:31:36  <frosch123> milek7: it works, but as long as you are logged in to github, it won't ask you again
13:31:43  <glx> that's how github works
13:31:50  <planetmaker> ah, k
13:31:51  <TrueBrain> it is on the todo to add an additional page there :)
13:31:56  <frosch123> milek7: but we want to insert another confirmation page inbetween
13:31:59  <glx> same with most oauth providers
13:34:39  <frosch123> ohoh, i tried to add an old scenario, and it says "junk at end of file"
13:35:42  <TrueBrain> there are a few out there yes, I believe 4
13:35:51  <TrueBrain> which aren't "master" scenarios :)
13:35:55  <TrueBrain> so gratz on finding one of them :P
13:35:57  <mcbanhas> https://www.gamingonlinux.com/uploads/articles/article_media/7915127711583314979gol1.gif OpenRA has implemented smooth zooming. Can we pls have too? :)
13:36:08  <frosch123> TrueBrain: no, i took one from my personal test folder
13:36:22  <TrueBrain> mcbanhas: if you are willing to put in the effort for that, sure
13:36:28  <TrueBrain> frosch123: awh .. :P
13:36:38  <nielsm> mcbanhas: first needs the video drivers to be replaced with something hardware accelerated
13:36:49  <frosch123> TrueBrain: i cannot reproduce your "file is gone after delete+reupload"
13:37:32  <Samu> that's Dune 2000
13:37:35  <Samu> i miss that game
13:37:38  <TrueBrain> not sure how you pieced that together, but what I did was: I uploaded some files, killed the API, restarted the API, uploaded the files again
13:37:41  <mcbanhas> At least I tried :)
13:37:55  <mcbanhas> Samu, you know it's free nowadays right?
13:37:58  <TrueBrain> from that point on, they were "uploaded", just never added
13:38:21  <frosch123> ok, i did not kill the api
13:38:25  <frosch123> just deleted and added them
13:38:44  <TrueBrain> you should have asked for steps to reproduce :P I haven't even tried the delete buttons :D
13:39:11  <frosch123> but if you kill the api, the upload token is invalid, isn't it?
13:39:14  <frosch123> so nothing works
13:39:20  <TrueBrain> basically, it happens when the API did not clean up the files, so "tusd" sees them again, and the tus-js-client knows in the localstorage the hash it used
13:39:23  <TrueBrain> that combination breaks it
13:39:31  <TrueBrain> yes, so you start a new session
13:39:33  <TrueBrain> you upload the files
13:39:36  <TrueBrain> and they simply never arrive
13:39:46  <TrueBrain> tus doesn't check metadata to be equal
13:39:51  <TrueBrain> it simply looks at filenames
13:40:29  <frosch123> i can even start a new upload with new token, and it also works
13:40:36  <TrueBrain> and for some reason, on the tusd-side, the hooks are not being triggered
13:41:25  <TrueBrain> the situation you have to create: when you kill the API, "ls data/" should still show files
13:41:43  <TrueBrain> after that, starting the API, and uploading the same file, hit validate, file is no longer there
13:41:49  <frosch123> well, i'll file this under low-prio then :)
13:41:58  <TrueBrain> no, you have to add the setting I showed you in the js client
13:42:02  <TrueBrain> the one they say: SET THIS TO TRUE
13:42:04  <TrueBrain> that solves it :)
13:42:19  <TrueBrain> the "removeFingerprintOnSuccess "
13:42:37  <TrueBrain> they themself say their default is bullshit
13:44:16  <frosch123> ok, i'll just enable it
13:44:25  <TrueBrain> tnx
13:44:38  <TrueBrain> tusd has some implementation issues, to say the least
13:44:54  <TrueBrain> it really should just send the hooks, and it would have been fine .. it doesn't :(
13:45:18  <TrueBrain> but it is also the piece of software that first says: I AM DONE! And after there: HERE IS A CHUNK OF DATA
13:45:23  <TrueBrain> because the callbacks .. are implemented wrong
13:48:42  <TrueBrain> btw, frosch123 , another way the above problem could happen is if someone logged out and logged in. But the latest API solves that by removing all files on log out too. Otherwise that was the easier way to reproduce :D
13:49:09  <glx> "finished ! PS: forgot to send the data, here they are"
13:54:35  <Samu> watch my methods: https://pastebin.com/raw/Vn7HGyAq
13:55:38  <Samu> can I do something to improve the speed of method 2 or 0?
13:55:55  <Samu> by speed, i mean, less ticks needed
13:56:35  <Samu> I'm still torn on this :(
13:58:55  <Samu> is it possible for the  "continue"  inside the while loop or foreach loop to apply to the loop outside of it?
13:59:31  <Samu> that would avoid the bool, and break to continue
14:00:18  <Samu> oh, there's no continue inside, my bad
14:01:12  <Samu> i wanted to "double continue"
14:01:25  <Samu> or break/continue, hmm
14:02:51  <Samu> there's no such thing as a break + continue in 1 command?
14:06:38  <Samu> while { while { do_stuff; break_continue; } } ?
14:07:51  <milek7> goto
14:08:03  <milek7> but considier regrouping into functions, and return from them
14:09:14  <TrueBrain> frosch123: https://gist.github.com/TrueBrain/8c61a86bc2b09ebd613d555c81766be6 a start
14:10:29  <Samu> going to try function
14:15:57  <frosch123> sorry for the confusion, but on forums i have no 123 :)
14:16:07  <frosch123> maybe add a "sorry for the inconvenience"
14:16:11  <frosch123> the creator also did that
14:17:06  <Samu> nop, no improvement, same number of ticks
14:17:23  <frosch123> it's a bit complicated, maybe it's easier with a pm/email content in a box, where the author needs to fill in some <bla>
14:17:35  <frosch123> s/content/template/
14:19:07  <FLHerne> mcbanhas: Pixel art at OTTD scale looks really bad if you smooth-scale it
14:20:02  <FLHerne> All the details are hinted at with a handful of pixels
14:21:48  <mcbanhas> FLHerne, not with this type of smooth scaling methinks.
14:23:01  <FLHerne> I've tried it, it does
14:23:19  <milek7> account migration cannot be done automatically?
14:23:23  <FLHerne> Might work if you have one of these really high-DPI screens
14:24:10  <mcbanhas> That's strange, because the sprite scale shouldn't be much different from what you see in RA1 units. Though the buildings and terrain are more hi-res, yes.
14:24:13  <FLHerne> But with a normal display, if you do anti-aliasing the details all get blurred, and if you don't it distorts the shapes and makes arbitrary thigs too big
14:24:39  <TrueBrain> frosch123: updated
14:24:40  <mcbanhas> Yeah, but I don't think they're doing AA here.
14:25:11  <mcbanhas> I'm not a technical guy, but they call it pixel doubling in the changelog.
14:25:52  <FLHerne> mcbanhas: Pixel doubling gets you 2x steps, which is what we already have :P
14:26:16  <FLHerne> There are algorithms to do it in a more heuristic way than simply doubling every pixel, like HQ2x
14:26:17  <mcbanhas> It doesn't look like AA in the game though.
14:26:54  <milek7> whether hq2x actually looks better is debatable
14:26:56  <FLHerne> But those are mostly tuned for 2D Nintendo-like games and work very badly on OTTD sprites
14:27:01  <mcbanhas> I'd say try it yourself. But it's just surreal how well it scales.
14:27:33  <frosch123> TrueBrain: looks good. i prefer option one a lot, so let's hope people pick that one :)
14:28:00  <TrueBrain> lets add that to the text
14:28:03  <TrueBrain> :D
14:28:25  <milek7> that openra fractional scaling is in some patch?
14:28:34  <TrueBrain> "This content-type + unique-id is blacklisted."
14:28:37  <TrueBrain> k, that works now
14:28:49  *** Progman has joined #openttd
14:29:24  <milek7> started dune2000 and it works in 2x steps like in ttd
14:29:52  <DorpsGek_III> [OpenTTD/bananas-api] TrueBrain opened pull request #47: Fix: don't allow uploading blacklisted content https://git.io/Jft9d
14:30:01  <TrueBrain> frosch123: you HTML-ify that migration page?
14:30:43  <mcbanhas> milek7, you gotta download the latest beta
14:30:49  <mcbanhas> you probably got the stable
14:30:50  <frosch123> yes
14:31:09  <FLHerne> mcbanhas: Hard to judge from a low-resolution gif
14:31:15  <FLHerne> mcbanhas: But yes, it's antialiasing https://github.com/OpenRA/OpenRA/commit/860117daf956a7bea662af60ff96dae8163d8760
14:31:32  <mcbanhas> Alright, I stand corrected then :)
14:31:56  <mcbanhas> Also, did you have a chance to try the new .lng file?
14:32:21  <FLHerne> It looks like it's doing something a bit smarter than a plain bilinear filter though
14:32:28  <LordAro> good morning
14:32:30  * FLHerne reads the linked blog posts
14:32:33  <FLHerne> Not yet
14:32:54  <TrueBrain> LordAro: what continent did you move to?
14:33:39  <DorpsGek_III> [OpenTTD/bananas-api] TrueBrain updated pull request #47: Fix: don't allow uploading blacklisted content https://git.io/Jft9d
14:33:51  <FLHerne> https://csantosbh.wordpress.com/2014/02/05/automatically-detecting-the-texture-filter-threshold-for-pixelated-magnifications/ and
14:33:56  <FLHerne> https://csantosbh.wordpress.com/2014/01/25/manual-texture-filtering-for-pixelated-games-in-webgl/
14:34:11  <FLHerne> (those should be in the opposite order, sorry)
14:34:22  <LordAro> andythenorth: frosch123: TrueBrain: nml windows binary - the pypi package is still just the standard python "module", the purely standalone executable is, as you've found, separate, and needs manually uploading *somewhere* (automatically attaching stuff to the GH release page was more difficult than it was worth). But still, if you've got python installed, you don't need the standalone executable
14:34:59  <TrueBrain> https://github.com/actions/upload-release-asset
14:35:42  <mcbanhas> So they're doing some trickery then?
14:35:46  <LordAro> TrueBrain: assert indeed a keyword (like print used to be in py2), but you still need to make sure you're not using it for control flow, as (like you might expect) they get removed if you run python optimisations. I noticed a few of these in nml...
14:36:12  <TrueBrain> owh, he is backlog-reading .. this can take a while :P
14:36:39  <LordAro> Samu: d:\a\s is the path where the "official" binaries are built, so the debug symbols you get for those will point to that path. you can (obviously) ignore that bit of the path and just look at the bit relative to src/
14:36:51  <LordAro> omg, an Yexo
14:37:04  <TrueBrain> this is pretty funny :P
14:37:08  <TrueBrain> liek a rerun of a movie
14:38:02  <LordAro> and scrollback complete!
14:38:04  <LordAro> TrueBrain: :p
14:38:09  <TrueBrain> :D
14:38:14  <LordAro> TrueBrain: yeah, i saw that action, decided i couldn't be bothered
14:38:16  <TrueBrain> time-sync reached
14:38:28  <milek7> FLHerne: https://milek7.pl/.stuff/rascaling/
14:38:36  <LordAro> TrueBrain: i did a cycle, so had to reread the last 5 hours :p
14:38:36  <milek7> looks average, maybe on higher dpi it is passable
14:38:41  <TrueBrain> so you have the tools to fully automate things, and you give up just before you reached the endline
14:38:53  <Yexo> o/ LordAro
14:38:54  <LordAro> TrueBrain: perhaps
14:38:59  <Yexo> Glad to see you (and others) keep NML alive
14:39:10  <TrueBrain> LordAro: :D
14:39:17  <LordAro> i'm mostly surprised that the pypi release actually worked
14:39:23  <TrueBrain> so am I :P
14:39:30  <TrueBrain> but really happy you managed to do that :)
14:39:32  <LordAro> Yexo: feel like finishing station support? :p
14:39:35  * LordAro runs
14:39:38  * TrueBrain feels less dependent on now :D
14:40:14  <Yexo> LordAro: perhaps. Just getting a feel back for how stuff works
14:40:25  <TrueBrain> your left thumb moves like this
14:40:30  <TrueBrain> and your right .. well .. similar
14:40:31  <Yexo> OpenTTD code itself hasn´t changed too much at least
14:40:45  <LordAro> quick, someone merge cmake and change everything
14:41:14  <TrueBrain> and I am waiting for my cert to be issues by AWS ..
14:41:16  <TrueBrain> it is a bit slow
14:41:30  <TrueBrain> with a bit I mean .. it often doesn't happen before the timeout happens
14:43:09  <DorpsGek_III> [OpenTTD/OpenTTD] LordAro commented on pull request #8097: Codechange: [Script] Improve copying a list into another empty list. https://git.io/JftHJ
14:46:37  <planetmaker> :) Yeah, creation of NML was an awesome thing you pulled-off, Yexo :)
14:47:13  <planetmaker> it's incredible of how much boost that gave to newgrf creation
14:47:35  <planetmaker> certainly > 50% market share :)
14:48:36  <glx> and today we officially broke all existing industry related .nml ;)
14:48:56  <glx> (well 0.4.5 still exists for them)
14:49:01  <planetmaker> well. ^^
14:49:09  <Yexo> Glad to hear that, but that wouldn have happened without maintenance/updates
14:49:15  <Yexo> 0.5 is a breaking change?
14:49:18  <planetmaker> NML did that before that old versions for old code
14:49:41  <glx> syntax change for more cargo support
14:49:42  <planetmaker> yes
14:52:08  <DorpsGek_III> [OpenTTD/OpenTTD] LordAro commented on pull request #8091: Add: [Script] Native priority queue; useful e.g. for pathfinders. https://git.io/JftHc
14:53:51  <TrueBrain> y
14:53:57  <TrueBrain> wrong window :D
15:09:51  <DorpsGek_III> [OpenTTD/OpenTTD] SamuXarick commented on pull request #8097: Codechange: [Script] Improve copying a list into another empty list. https://git.io/JftHH
15:13:33  <DorpsGek_III> [OpenTTD/nml] LordAro merged pull request #106: Doc: remove hard-coded release date and version number from README https://git.io/Jft1m
15:14:00  <DorpsGek_III> [OpenTTD/nml] LordAro dismissed a review for pull request #103: Blacken & Flake8 nml https://git.io/JfLGq
15:16:39  <DorpsGek_III> [OpenTTD/OpenTTD] glx22 commented on pull request #8097: Codechange: [Script] Improve copying a list into another empty list. https://git.io/JftQf
15:21:18  <DorpsGek_III> [OpenTTD/OpenTTD] LordAro approved pull request #8097: Codechange: [Script] Improve copying a list into another empty list. https://git.io/JftQm
15:23:05  <andythenorth> LordAro maybe I should update the nml readme release instructions?  ("Upload the windows binary to github manually")
15:23:33  <DorpsGek_III> [OpenTTD/bananas-frontend-web] frosch123 opened pull request #30: PR of the day https://git.io/JftQO
15:23:35  <LordAro> andythenorth: possibly
15:24:44  <DorpsGek_III> [OpenTTD/OpenTTD] SamuXarick commented on pull request #8091: Add: [Script] Native priority queue; useful e.g. for pathfinders. https://git.io/JftQs
15:25:40  <DorpsGek_III> [OpenTTD/bananas-api] frosch123 approved pull request #47: Fix: don't allow uploading blacklisted content https://git.io/JftQZ
15:26:34  <frosch123> bbl
15:33:02  <DorpsGek_III> [OpenTTD/bananas-frontend-web] TrueBrain approved pull request #30: PR of the day https://git.io/JftQu
15:33:19  <DorpsGek_III> [OpenTTD/bananas-frontend-web] frosch123 dismissed a review for pull request #30: PR of the day https://git.io/JftQu
15:33:20  <DorpsGek_III> [OpenTTD/bananas-frontend-web] frosch123 updated pull request #30: PR of the day https://git.io/JftQO
15:33:31  <TrueBrain> frosch123: deployment will fail, but I will fix that after :)
15:33:44  <frosch123> i fixed the "auto-fill" commit
15:33:56  <frosch123> the button enable/disable failed
15:34:09  <DorpsGek_III> [OpenTTD/bananas-frontend-web] TrueBrain approved pull request #30: PR of the day https://git.io/JftQ2
15:34:24  <TrueBrain> I don't have a real way to judge these commits, so I am just approving them :P
15:34:34  <TrueBrain> "it doesn't look wrong" :P
15:34:50  <DorpsGek_III> [OpenTTD/bananas-api] TrueBrain merged pull request #47: Fix: don't allow uploading blacklisted content https://git.io/Jft9d
15:35:28  <LordAro> dbg: [net] injecting: 000acfdb; 48; ff; 000000; 00000000; 00000000; 00000036; "zul" (CmdRenamePresident)
15:35:33  <LordAro> this is no desync only zul
15:37:00  <glx> <andythenorth> LordAro maybe I should update the nml readme release instructions?  ("Upload the windows binary to github manually") <-- but the upload part can be handled via github API calls IIRC
15:37:27  <andythenorth> I am about to PR some tweaks 1 min
15:38:24  <DorpsGek_III> [OpenTTD/nml] andythenorth opened pull request #107: Doc: more acccurate release process in README https://git.io/JftQP
15:38:58  <andythenorth> if we can automate the Windows binary it would be better, but it's not the end of the world
15:39:35  <glx> https://github.com/actions/upload-release-asset <-- didn't really check but this should work
15:40:05  <DorpsGek_III> [OpenTTD/nml] andythenorth updated pull request #107: Doc: more acccurate release process in README https://git.io/JftQP
15:40:18  <andythenorth> glx be interesting to try
15:40:30  * andythenorth has hopes of restoring what Coop Bundles did, but with GH Actions
15:40:35  <andythenorth> build + publish grfs
15:41:40  <LordAro> glx: i did look at that at the time, but decided not to bother
15:41:53  <LordAro> there was some weirdness about whether it actually wanted to create the release object as well
15:41:58  <LordAro> so i got scared off
15:42:30  <DorpsGek_III> [OpenTTD/bananas-frontend-web] frosch123 merged pull request #30: PR of the day https://git.io/JftQO
15:43:46  <frosch123> TrueBrain: you can do your magic :)
15:48:01  <TrueBrain> waiting for it to fail deployen, before I can upload the code that makes it succeed :D Waiting waiting waiting .....
15:49:48  <TrueBrain> hmm .. I forgot we also have scenarios .. what to do with those .. I guess they are also for the installer, so lets do that
15:50:38  <frosch123> huh?
15:51:02  <TrueBrain> I was checking what files we host
15:51:07  <TrueBrain> and I found the "scenarios" folder again
15:51:11  <TrueBrain> totally forgot about that tbh
15:51:20  <frosch123> isn't that a thing of ottd < 0.4 ?
15:51:40  <TrueBrain> there is a 0.4.6, 0.4.7, 0.4.8, 0.5.0 version
15:51:53  <frosch123> glx: does windows installer stil offer to install scenarios?
15:51:54  <TrueBrain> no clue if anything ever uses it,tbh
15:52:10  <TrueBrain> it does not frosch123
15:52:30  <glx> only opengfx and friends
15:53:21  <TrueBrain> lets check the logs if they are ever accessed ..
15:53:33  <TrueBrain> in the last 90 days, 0 times
15:55:07  <TrueBrain> frosch123: login looks a lot better now, awesome
15:56:01  <TrueBrain> okay, given nobody ever uses those files, I will let them be forgotten in the way they were 10 minutes ago :P
15:56:21  <Eddi|zuHause> archeologists in 100 years will discover them
15:57:08  <LordAro> dbg: [net] sync check: 000ad0c5; 00; mismatch expected {b65c131c, 005de115}, got {8549d8e1, 9077ce26}
15:57:11  <LordAro> oho.
16:01:22  <FLHerne> michi_cc: With the append-to-empty patch, what happens if the other list is modified afterward?
16:01:47  <michi_cc> FLHerne: Nothing. It's a copy.
16:01:51  <FLHerne> (can that happen? I'm not familiar with Squirrel's containers, just a thought)
16:02:08  <michi_cc> ScriptList is our container, not Squirrel's.
16:02:41  <michi_cc> The contents is entirely managed on the C++ side.
16:04:36  <FLHerne> Ok, I see
16:06:36  *** adikt has quit IRC
16:09:39  <TrueBrain> okay .. installer is prep'd on new infra, bananas I double-checked that we have all the URLs, and is running on staging correctly ..
16:09:45  <DorpsGek_III> [OpenTTD/OpenTTD] michicc merged pull request #8097: Codechange: [Script] Improve copying a list into another empty list. https://git.io/Jftl6
16:10:25  <TrueBrain> to me it sounds like tomorrow I can launch it in production on AWS, wire the whole shit up, and redirect traffic .. but I have to say: there are so many things going on with binaries .. it became a real maze over the last 15 years .. I hope I have everything in view :D
16:10:28  <TrueBrain> worst case .. something breaks :P
16:12:07  <TrueBrain> is there any reason not to push the new BaNaNaS live tomorrow? ( LordAro , nielsm , frosch123 , glx , who-ever-has-input-sorry-if-I-forgot-to-mention-you)
16:12:40  <DorpsGek_III> [OpenTTD/OpenTTD] michicc commented on pull request #8091: Add: [Script] Native priority queue; useful e.g. for pathfinders. https://git.io/Jft7g
16:12:55  <glx> probably the only way to really test it
16:13:03  <TrueBrain> exactly ;)
16:13:14  <TrueBrain> and pretty sure there will be a screw-up somewhere or what-ever
16:16:03  <LordAro> can it be put back if everything falls over?
16:17:21  <TrueBrain> till the 30th, yes
16:17:28  <TrueBrain> after that .. well, still yes, but in degraded capacity
16:17:54  <TrueBrain> personally I am more worried about integrity .. not sure anyone really tested if what they upload also shows up ingame
16:18:01  <TrueBrain> for me that is hard to judge, other than: it works :D
16:34:20  *** Flygon has quit IRC
16:36:00  <LordAro> has anyone else tried to reproduce the desync in #8093 ? i've followed the docs in desync.md (not done it before), but no mismatch
16:36:10  <LordAro> (the one above was due to me starting the wrong savegame)
16:58:29  <frosch123> TrueBrain: i have a question left: when some dude sends some email to transfer accounts, how do i match the email to some account?
16:59:02  <frosch123> other than that, i have by bets on the first 3 complains, but no real blockers :)
16:59:37  <glx> usually the email doesn't match IIRC
17:00:46  <TrueBrain> frosch123: let me pick up the first of the emails, I guess .. but basically: login to the LDAP :)
17:00:52  <glx> and the user doesn't remember username or password either
17:03:26  * glx is looking at actions/upload-release-asset for nml, I think the easiest way is to change the workflow trigger
17:06:11  <frosch123> LordAro: what did you do exactly, if you replay everything, and get no mismatch, then that is good
17:09:29  <frosch123> TrueBrain: did someone already write a script to add users to the bananas data?
17:09:47  <TrueBrain> I did not
17:11:25  <LordAro> frosch123: i ran with the full log (nearly, had to remove the titlegame from the log), and then truncated it down until i was past the point where the client was disconnected due to desync
17:12:12  <frosch123> that means your replay matches the server, which is a good sign, as it means that you have enough data
17:12:25  <frosch123> next check when the desyncing player joined
17:12:38  <frosch123> and replay starting with the same just before/after the join
17:12:58  <frosch123> s/same/save/
17:13:51  <frosch123> hmm, or did you mean that with "truncate"?
17:13:55  <frosch123> truncate from font?
17:17:02  <blathijs> frosch123: Awesome!
17:17:21  <DorpsGek_III> [OpenTTD/bananas-api] frosch123 opened issue #48: Validation warning when no readme.txt and/or changelog.txt was uploaded https://git.io/Jftdg
17:18:08  <frosch123> blathijs: i only delivered the message :) no work from me involved
17:18:48  <LordAro> frosch123: yeah, done that
17:18:56  <LordAro> truncated to the individual save: lines
17:19:00  <LordAro> from the top
17:19:12  <LordAro> "truncate from font" ?
17:19:22  <blathijs> LordAro: andythenorth: Congrats on the nml release, I'll be packaging it this week :-)
17:19:30  <frosch123> front :)
17:20:05  <LordAro> thought so :p
17:20:17  <LordAro> blathijs: still not really me, i just did the release process :p
17:20:29  <frosch123> LordAro: two options left: (1) replay from start and let it make a save at exactly the join tick. (2) compare the savegames with the savegame-diff tools, sometimes they differ without the desync being detected
17:21:12  <frosch123> after that i would claim the user was using a patched client
17:21:29  <andythenorth> :)
17:21:30  <LordAro> hmm
17:21:40  <LordAro> how can i force a save on join tick?
17:21:51  <LordAro> (and where are savegame diff tools?)
17:24:04  <frosch123> LordAro: for making a save: openttd.cpp:1364
17:24:34  <frosch123> aren't the diff tools linked from desync.md?
17:24:55  <frosch123> (they are really shitty, and probably should be rewritten in python using the new bananas savegame code)
17:25:13  <frosch123> https://github.com/OpenTTD/OpenTTD/blob/master/docs/desync.md#33-comparing-savegames
17:25:27  <LordAro> right
17:26:30  <frosch123> LordAro: oh, option (3), the desync depends on the OS/compiler
17:28:03  <Samu> just experimented with walking backwards to check for conflicting bridges, ticks went from 307 to 374, it regressed :(
17:28:03  <frosch123> stuff like signed/unsigned enums
17:28:03  <LordAro> ohno
17:28:20  <Samu> the advantage is that it retains compatibility with graph.aystar-6
17:28:32  <LordAro> kj:b#
17:28:38  <LordAro> er
17:28:46  <LordAro> that's why my terminal didn't do anything
17:28:59  <frosch123> ctrl+q ?
17:29:45  <frosch123> "ctrl+q" is a close second to "ESC q!"
17:34:16  *** snail_UES_ has joined #openttd
17:36:31  <blathijs> andythenorth: LordAro: Any plans to also publish nml at https://bundles.openttdcoop.org/nml/releases/ as before? If not, maybe a redirect or note should be uploaded under "LATEST" there?
17:37:48  <blathijs> The Debian package checks that URL for new versions, but I can easily change that to check github instead, of course.
17:38:20  <blathijs> Also, the README still says "Release version: 0.4.5"
17:38:30  <Samu> michi_cc, have you worked on implementing graph aystar back in openttd? I was actually wondering what kind of performance there was to gain
17:38:54  <blathijs> Oh never mind the latter, git master has the 0.4.5 reference removed already I see.
17:39:08  <LordAro> blathijs: not sure about devzone, probably needs a planetmaker to fix
17:39:51  <blathijs> Samu: What do you mean by "back"? Was the A* pathfinder removed at some point?
17:40:17  <frosch123> blathijs: remove the devzone url :) it won't sustain
17:40:22  <blathijs> frosch123: Ok
17:40:47  <Samu> reading the docs in devzone indicates it was in openttd before
17:41:09  <LordAro> probably related to the original AI
17:42:25  <Samu> it was an old version, version 1
17:42:30  <blathijs> Samu: I implemented the first A* pathfinder in OpenTTD many years ago for vehicles (called NPF for New Pathfinde, 2004 or something I think), and I think aystart might have been a generalization of that for use with AIs? I think TrueBrain originally built the Aystar AI implementation, maybe I also worked on that.
17:44:56  <Samu> make it version 6 compatible
17:45:41  <TrueBrain> the original AI did not have A* :P
17:46:09  <glx> the original AI was silly (but better than TT)
17:46:59  <TrueBrain> blathijs: indeed, something like that. I think in the end we both worked together to make a generalized version of A* in OpenTTD, but the scripting language never had direct access to that, as it would mean the game could stall for too long. So the Squirrel version is a near-copy of the OpenTTD variant (well, which is just A*)
17:47:12  <TrueBrain> it was always a very delibrate choice not to let AI use the built-in pathfinders
17:47:19  <TrueBrain> the AI would stall the game constantly
17:47:26  <TrueBrain> was considered very unacceptable ;)
17:47:53  <LordAro> ah, yes
17:47:56  <Samu> oh, I guess that's why
17:47:58  <LordAro> that would be a good reason :p
17:48:06  <TrueBrain> I tried to tell that in the PR :)
17:48:14  <TrueBrain> you need additional ways to promise an AI doesn't kill performance
17:48:19  <TrueBrain> in Squirrel, they are bound to ticks
17:48:21  <TrueBrain> no way around it
17:49:02  <TrueBrain> adding NoAI was already a bit of: wtf will performance do
17:49:10  <TrueBrain> so back then it was more an issue then it would be now
17:52:21  <Samu> I was imagining... in squirrel the pathfinder would provide the callback functions to graph.aystar, kinda like it does now, but a* would be governed by openttd, would that kill performance?
17:52:59  <LordAro> yes
17:53:05  <mcbanhas> Guys how do I mark an article as a draft on the wiki? The {{Draft article}} tag doesn't seem to be working on this version of mediawiki.
17:53:43  <Yexo> Currently the AI can be stopped at any point during pathfinding by the squirrel interpreter. However it is not allowed to stop a script while in a callback from native code
17:53:43  <LordAro> "{{Draft article}}" sounds like something that's been added by whichever mediawiki instance you were using
17:53:55  <LordAro> but the wiki is very very old
17:54:14  <Yexo> Which means that the native pathfinder code would have to include code that can stop pause your AI until the next tick
17:54:28  <mcbanhas> So can I just save a page that's incomplete?
17:54:34  <LordAro> yes
17:54:37  <mcbanhas> Ok cool
17:57:12  <andythenorth> mcbanhas just stick DRAFT somewhere near the top
17:57:23  <andythenorth> if no better way is found
17:59:21  <LordAro> frosch123: ok, so the desync disconnect occurred at 000ad0ef. I compared the dumps of 000ad100 from save that duck provided, and the one that i just generated - no differences
17:59:56  <LordAro> wait hang on, i need to start the game from 000ad053
18:00:22  <DorpsGek_III> [OpenTTD/bananas-api] TrueBrain opened pull request #49: Minor fixes and zip/tar/tar.gz support https://git.io/JftF4
18:00:25  <TrueBrain> there you go frosch123 , specially for you :)
18:01:04  <TrueBrain> you need to press auto-fill after uploading for frontend-web for now
18:01:31  <DorpsGek_III> [OpenTTD/bananas-api] TrueBrain updated pull request #49: Minor fixes and zip/tar/tar.gz support https://git.io/JftF4
18:01:32  <TrueBrain> and the mandatory: did you forget black again?!
18:01:44  <TrueBrain> so happy with GitHub Actions :D
18:04:03  <DorpsGek_III> [OpenTTD/bananas-api] TrueBrain updated pull request #49: Minor fixes and zip/tar/tar.gz support https://git.io/JftF4
18:07:00  <frosch123> "theze" :)
18:08:06  <DorpsGek_III> [OpenTTD/bananas-frontend-web] TrueBrain opened pull request #31: Fix: minor tuning on the migration text https://git.io/JftFz
18:08:19  <TrueBrain> where did I type that ...
18:08:44  <frosch123> last commit message
18:09:00  <TrueBrain> fail whale
18:09:02  <DorpsGek_III> [OpenTTD/bananas-api] TrueBrain updated pull request #49: Minor fixes and zip/tar/tar.gz support https://git.io/JftF4
18:09:37  <mcbanhas> https://wiki.openttd.org/Manual_of_style
18:09:42  <TrueBrain> it is funny how the interface of TarInfo and ZipInfo are JUST not the same
18:09:46  <TrueBrain> fucking tards
18:09:58  <LordAro> TrueBrain: there's a lot of the python stdlib like that :p
18:10:01  <mcbanhas> Just so you know what I've been up to. It still neds a lot of work and all the examples still need to be put in.
18:10:28  <mcbanhas> But hopefully it will see a lot more polish in the next few days.
18:10:44  <LordAro> frosch123: i'm getting the impression that saving the game on a specific tick value (date_fract) doesn't work so well - the desync stuff definitely appears to be assuming that i'm starting on 0 (rather than 9)
18:10:54  <mcbanhas> I know it's just a document, but hopefully having this will make proofreading for UI texts much easier.
18:18:06  <DorpsGek_III> [OpenTTD/bananas-api] frosch123 commented on pull request #49: Minor fixes and zip/tar/tar.gz support https://git.io/JftF6
18:19:12  <frosch123> LordAro: why would it care about the tick?
18:19:40  <LordAro> frosch123: i'm not sure, but commands.log is getting skipped
18:20:07  <LordAro> i'm just reversing 9 ticks anyway, nothing else apparently happened in between
18:21:07  <DorpsGek_III> [OpenTTD/bananas-api] TrueBrain commented on pull request #49: Minor fixes and zip/tar/tar.gz support https://git.io/JftFX
18:21:42  <frosch123> LordAro: assuming the desync is caused by pathfinder caches, the commands do not matter
18:21:53  <DorpsGek_III> [OpenTTD/bananas-api] TrueBrain updated pull request #49: Minor fixes and zip/tar/tar.gz support https://git.io/JftF4
18:23:44  <DorpsGek_III> [OpenTTD/bananas-api] frosch123 approved pull request #49: Minor fixes and zip/tar/tar.gz support https://git.io/JftFS
18:27:07  *** mcbanhas_ has joined #openttd
18:29:33  *** gelignite_ has joined #openttd
18:33:17  *** mcbanhas has quit IRC
18:35:10  <DorpsGek_III> [OpenTTD/bananas-api] TrueBrain merged pull request #49: Minor fixes and zip/tar/tar.gz support https://git.io/JftF4
18:35:57  *** gelignite has quit IRC
18:45:32  <LordAro> dbg: [net] sync check: 000ad0ef; 00; mismatch expected {33188c8c, 1d7b69f5}, got {007a3060, 2361519c}
18:45:35  <LordAro> aha
18:45:39  <LordAro> and i think this is genuine this time :)
18:46:22  <LordAro> no idea what it means, but...
18:49:15  <LordAro> and yeah, got a difference in savegame dumps at ad0e0
18:51:16  <Samu> are you hunting desyncs?
18:51:26  <Samu> tedious job
18:54:28  <frosch123> what chunk? vehicles?
18:54:46  <LordAro> MAP2
18:55:36  <LordAro> also MAPE
18:56:23  <DorpsGek_III> [OpenTTD/bananas-frontend-web] TrueBrain updated pull request #31: Fix: minor tuning on the migration text https://git.io/JftFz
18:56:25  <TrueBrain> frosch123: feel free to improve the wording of both commits :)
18:56:50  <LordAro>        00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00  |        00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
18:56:54  <LordAro> wait, these are identical
18:56:56  <LordAro> silly diff
18:57:00  <LordAro> not MAPE
18:58:44  <frosch123> map array is unusual for a desync
19:05:12  <LordAro> i'll write up what i've got in the bug
19:05:20  <LordAro> but feel free to recheck my work :p
19:08:02  <DorpsGek_III> [OpenTTD/nml] FLHerne updated pull request #70: Binop cleanup https://git.io/Jey9d
19:10:35  <DorpsGek_III> [OpenTTD/bananas-frontend-web] frosch123 approved pull request #31: Fix: minor tuning on the migration text https://git.io/JftNL
19:11:29  <DorpsGek_III> [OpenTTD/nml] FLHerne commented on pull request #70: Binop cleanup https://git.io/JftN3
19:14:26  <DorpsGek_III> [OpenTTD/bananas-frontend-web] TrueBrain merged pull request #31: Fix: minor tuning on the migration text https://git.io/JftFz
19:15:30  <TrueBrain> Except for a web download url it seems to at least do what the current version does
19:15:40  <TrueBrain> Nice :)
19:16:18  <DorpsGek_III> [OpenTTD/nml] FLHerne commented on pull request #70: Binop cleanup https://git.io/JftNR
19:16:23  <DorpsGek_III> [OpenTTD/OpenTTD] LordAro commented on issue #8093: Regular server desynchronisations from clients https://git.io/JfkmV
19:22:03  <frosch123> aw, i broke it
19:22:17  <frosch123> waiting for sentry
19:25:54  <TrueBrain> you broke something I am sure I fixed already :P Let see
19:26:28  <TrueBrain> hmm .. you pressed delete on something :)
19:27:01  <TrueBrain> ah, yes ofc
19:27:21  <Samu> MAP2 chunk?
19:27:29  <TrueBrain> I did not check removing of files, clearly :D
19:29:00  <mcbanhas_> https://i.imgur.com/T3vhgZA.png frosch123 if I take down the SA_CENTER parameter like you told me, it unfortunately does not compensate for the top or right margins if the text is small. Which values do I have to tweak to make this possible?
19:30:30  <DorpsGek_III> [OpenTTD/bananas-api] TrueBrain opened pull request #50: Fix: there is no .info file for extracted files https://git.io/JftAv
19:30:40  <TrueBrain> frosch123: it is a silly mistake :D
19:32:07  *** mcbanhas_ has quit IRC
19:32:31  *** mcbanhas has joined #openttd
19:32:52  <mcbanhas> In a nutshell, here's what I' trying to do https://i.imgur.com/wB9LQWY.png
19:34:21  <mcbanhas> These are the tooltips from CK2. Tooltips are left aligned in this game, but always reserve a margin for smaller instances to appear centered.
19:37:01  <frosch123> mcbanhas: use SA_VERT_CENTER then
19:37:29  <mcbanhas> Alright,let me try
19:38:45  <LordAro> (or SA_HOR_CENTER :p )
19:39:21  <DorpsGek_III> [OpenTTD/bananas-api] frosch123 commented on pull request #50: Fix: there is no .info file for extracted files https://git.io/JftAC
19:42:08  <DorpsGek_III> [OpenTTD/bananas-api] TrueBrain updated pull request #50: Fix: there is no .info file for extracted files https://git.io/JftAv
19:42:10  <TrueBrain> frosch123: during testing that never happened, but it can't hurt to prevent it :D
19:45:27  <DorpsGek_III> [OpenTTD/nml] LordAro approved pull request #102: Fix #100: follow PEP440 when generating version https://git.io/JftAV
19:49:35  <DorpsGek_III> [OpenTTD/nml] LordAro approved pull request #107: Doc: more acccurate release process in README https://git.io/JftAD
19:49:42  <DorpsGek_III> [OpenTTD/nml] LordAro merged pull request #107: Doc: more acccurate release process in README https://git.io/JftQP
19:50:41  *** gnu_jj_ has joined #openttd
19:51:19  *** gnu_jj__ has joined #openttd
19:55:33  <DorpsGek_III> [OpenTTD/OpenTTD] JGRennison commented on issue #8093: Regular server desynchronisations from clients https://git.io/JfkmV
19:55:56  *** gnu_jj has quit IRC
19:59:02  *** gnu_jj_ has quit IRC
19:59:25  <DorpsGek_III> [OpenTTD/bananas-api] frosch123 approved pull request #50: Fix: there is no .info file for extracted files https://git.io/JftxT
20:00:16  *** gnu_jj__ has quit IRC
20:01:11  <mcbanhas> https://i.imgur.com/2Telfwz.png frosch123 this still doesn't fix it entirely. The box is essentially still behaving as if the text was still centered and leaves a way too big margin on the right. The max it should ideally go should be the length of the longest line.
20:02:33  <mcbanhas> You can see also on the small tooltip, the right margin is a bit bigger.
20:02:33  <frosch123> mcbanhas: ottd's layout capabilities are very limited
20:02:42  <frosch123> it firs decides for a width of the box, then breaks the lines
20:03:54  <DorpsGek_III> [OpenTTD/bananas-api] TrueBrain merged pull request #50: Fix: there is no .info file for extracted files https://git.io/JftAv
20:04:19  <mcbanhas> I don't suppose that has an easy fix.
20:04:25  <frosch123> nope :)
20:04:56  <mcbanhas> Would it require a serious overhaul or anything of the kind?
20:04:57  <glx> https://github.com/OpenTTD/nml/compare/master...glx22:upload <-- in theory this should work
20:05:12  <frosch123> LordAro: aw, i was betting for ships/rv pathfinder caches, but it looks like it's trains again :/
20:05:17  *** gnu_jj has joined #openttd
20:05:18  <glx> but not very easy to test
20:06:00  <LordAro> glx: that's why there's a nml-test package on test.pypi.org :p
20:06:24  <LordAro> actually, i can give you a token for it, if you like
20:06:34  <LordAro> you can add the secret to your repo
20:06:55  <glx> means I need to merge it in my repo to trigger a release
20:07:34  <LordAro> or make a separate branch that also removes the master-only release behaviour
20:07:57  <DorpsGek_III> [OpenTTD/nml] Yexo commented on pull request #70: Binop cleanup https://git.io/Jftxn
20:07:59  <glx> the trigger is now on release, not push
20:08:23  <LordAro> well, you can change that :p
20:08:42  <glx> but I need it to easily get the upload url
20:08:44  <DorpsGek_III> [OpenTTD/bananas-api] frosch123 closed issue #22:  Allow uploading .zip/.tar instead of individual files https://git.io/Jfktf
20:10:39  <mcbanhas> Wait a moment, can I adjust the minimum margin manually? I see it is 1px on the left side. Is this controlled by line 730?
20:11:31  <TrueBrain> glx : why zip the executable? That feels weird :p
20:11:48  <DorpsGek_III> [OpenTTD/nml] FLHerne commented on pull request #70: Binop cleanup https://git.io/Jftxu
20:11:49  <frosch123> mcbanhas: sure, it's jsut code
20:12:15  <glx> TrueBrain: so it's always nmlc.exe once extracted
20:12:18  * mcbanhas is not a programmer
20:12:30  <mcbanhas> not even a pro-gamer
20:12:56  <TrueBrain> Feels weird .. maybe add license and readme in zip too?
20:13:31  <TrueBrain> Otherwise it looks nice :)
20:13:32  <glx> ho good idea
20:13:58  <glx> yeah just hard to test as always with github actions ;)
20:17:37  <TrueBrain> Especially if you start using non-git-hooks indeed
20:17:50  <TrueBrain> There are simulation tools for it .. never tried them
20:18:12  <TrueBrain> I try to stay within the git domain :D
20:19:33  <andythenorth> oof it's nearly Monday :|
20:19:39  <andythenorth> usually I am fine with Monday
20:20:36  * andythenorth has not had a day off since Jan 2nd, except public holidays
20:23:13  <frosch123> TrueBrain: i guess i can break it again, be deleting the ".tar". but i guess we do not care
20:28:48  <DorpsGek_III> [OpenTTD/bananas-api] frosch123 opened issue #51: Uploading an update discards all meta data except "version" https://git.io/Jftxb
20:33:50  <TrueBrain> frosch123 : define break? It should give a 400 on the API
20:34:06  <andythenorth> hmm NRT bug report in forums :|
20:34:17  <andythenorth> only 15 types available
20:34:54  <andythenorth> oof, docs say more
20:35:09  * andythenorth tactically ignores this for today :D
20:38:13  <frosch123> TrueBrain: ok, all good then :)
20:40:02  <TrueBrain> It is the front end that is using obsolete info :p
20:43:24  <TrueBrain> hmm .. frosch123  .. can a version have another name than the main package?
20:43:37  <TrueBrain> feels a bit weird :P
20:43:38  <TrueBrain> but okay
20:43:42  <andythenorth> wait wait wait wait wtf
20:43:46  <TrueBrain> by spec, you currently should be able
20:43:48  <frosch123> we allowed editing it :)
20:43:49  <andythenorth> yexo just commented in GH
20:44:12  <andythenorth> @seen yexo
20:44:12  <DorpsGek> andythenorth: yexo was last seen in #openttd 2 hours, 49 minutes, and 58 seconds ago: <Yexo> Which means that the native pathfinder code would have to include code that can stop pause your AI until the next tick
20:44:33  <frosch123> andythenorth: you missed something :p
20:45:05  <andythenorth> what next Dalestan?
20:45:12  <andythenorth> Belugas?
20:45:19  <frosch123> dalestan was spotted on factorio forums
20:45:31  <frosch123> he explained to planetmaker how to play the game properly, or something
20:45:58  <LordAro> @seen bjarni
20:45:58  <DorpsGek> LordAro: bjarni was last seen in #openttd 8 years, 29 weeks, 1 day, 20 hours, 26 minutes, and 51 seconds ago: <Bjarni> heh
20:46:15  <TrueBrain> @seen Celestar
20:46:16  <DorpsGek> TrueBrain: Celestar was last seen in #openttd 2 years, 15 weeks, 4 days, 7 hours, 50 minutes, and 1 second ago: <Celestar> lo
20:46:22  <TrueBrain> @seen MiHaMiX
20:46:22  <DorpsGek> TrueBrain: I have not seen MiHaMiX.
20:46:24  <TrueBrain> ha!
20:46:30  <frosch123> @seen mihamek
20:46:30  <DorpsGek> frosch123: I have not seen mihamek.
20:46:38  <frosch123> how many other spellings?
20:46:56  <TrueBrain> @seen ludde
20:46:56  <DorpsGek> TrueBrain: ludde was last seen in #openttd 7 years, 41 weeks, 4 days, 13 hours, 43 minutes, and 34 seconds ago: <ludde> hej
20:47:10  <LordAro> how far back does the DorpsGek history go?
20:48:04  <TrueBrain> at least 8 years and 29 weeks :P
20:48:08  <andythenorth> this was last time I saw yexo :) 2012 https://github.com/andythenorth/chips/commit/e65fd31a088ba85d1ac53515ceffd13651171875
20:48:15  <FLHerne> According to recent empirical evidence, at least 8 years, 29 weeks, 1 day, 20 hours, 29 minutes and a few seconds :P
20:48:39  <FLHerne> TrueBrain: jinx
20:48:48  <TrueBrain> LordAro: #openttd.20070403.log
20:49:08  <TrueBrain> who is a good bot? DorpsGek is a good bot! WHO WANTS A COOKIE?! DorpsGek should visit www.amazon.com
20:49:27  <frosch123> @seen MiHaMeK
20:49:27  <DorpsGek> frosch123: I have not seen MiHaMeK.
20:49:35  <frosch123> lies, he has been here in 2008
20:50:39  <frosch123> DorpsGek: http://webster.openttdcoop.org/index.php?channel=openttd&date=1201651200#1201718829
20:51:13  <frosch123> LordAro: those logs start 2006-02-25
20:51:47  <LordAro> it parses logs to see when last seen?
20:51:51  <LordAro> that seems... inefficient
20:52:19  <frosch123> no idea, but i do it like that :p
20:52:53  <TrueBrain> it did record MiHaMeK in another channel ..
20:53:23  <DorpsGek_III> [OpenTTD/OpenTTD] JGRennison commented on issue #8093: Regular server desynchronisations from clients https://git.io/JfkmV
20:53:29  <TrueBrain> in 2013
20:54:13  <TrueBrain> weird, there is an entry of him in the database
20:54:20  <TrueBrain> @seen MiHaMeK
20:54:20  <DorpsGek> TrueBrain: I have not seen MiHaMeK.
20:54:27  <TrueBrain> I .. am looking .. at the entry ...
20:54:29  <TrueBrain> wth
20:54:33  <frosch123> @seen miha*
20:54:33  <DorpsGek> frosch123: I haven't seen anyone matching miha*.
20:54:33  <LordAro> lol
20:54:43  <TrueBrain> #openttd,MiHaMiX,1199645917.23,<MiHaMiX> hmm
20:54:45  <TrueBrain> it is right there!
20:54:50  <frosch123> @seen mih*
20:54:50  <DorpsGek> frosch123: miho was last seen in #openttd 5 years, 25 weeks, 0 days, 23 hours, 21 minutes, and 5 seconds ago: <miho> yeah, I'll give it try
20:55:12  <frosch123> @seen *hami*
20:55:12  <DorpsGek> frosch123: WHamilton was last seen in #openttd 1 year, 39 weeks, 0 days, 18 hours, 15 minutes, and 55 seconds ago: <WHamilton> forgot i closed it earlier, thanks
20:55:31  <frosch123> well, now i feel like samu
20:56:49  <DorpsGek_III> [OpenTTD/bananas-api] TrueBrain opened pull request #52: Fix #51: set the name / description / url / tags when uploading a new version https://git.io/Jftpg
20:57:08  <TrueBrain> :D
20:57:24  <TrueBrain> I am happy so far the bugs you keep finding are easy
20:59:39  <frosch123> is there a difference in raw_data and session for new packages?
20:59:52  <TrueBrain> raw_data is the processed session info
20:59:56  <TrueBrain> so it went through the validator
21:00:05  <TrueBrain> just to catch any remaining glitches
21:01:36  <DorpsGek_III> [OpenTTD/bananas-api] frosch123 approved pull request #52: Fix #51: set the name / description / url / tags when uploading a new version https://git.io/Jftpy
21:02:45  <frosch123> so, news post tomorrow :)
21:03:45  <TrueBrain> :D
21:04:38  <frosch123> LordAro: wasn't there some samu/ai discussion about attaching wagons to trains in depots? was anything changed?
21:05:00  <LordAro> frosch123: i don't believe i ended up changing anything, i just changed documentation
21:05:15  <nielsm> could it be related to auto-refit on purchase?
21:05:42  <LordAro> https://github.com/OpenTTD/OpenTTD/pull/8002
21:08:12  *** sla_ro|master has quit IRC
21:08:44  <DorpsGek_III> [OpenTTD/bananas-api] TrueBrain merged pull request #52: Fix #51: set the name / description / url / tags when uploading a new version https://git.io/Jftpg
21:08:45  <DorpsGek_III> [OpenTTD/bananas-api] TrueBrain closed issue #51: Uploading an update discards all meta data except "version" https://git.io/Jftxb
21:12:29  *** tokai has quit IRC
21:15:12  <frosch123> nielsm: the GroupStatistics stuff in the build+refit looks fishy for sure
21:15:48  <frosch123> the OrderBackup in there may even cause a desync
21:15:54  *** snail_UES_ has quit IRC
21:15:55  <Samu> what are we talking about? I was mentioned
21:15:59  <Samu> :o
21:16:14  *** snail_UES_ has joined #openttd
21:16:59  *** snail_UES_ has joined #openttd
21:17:37  <LordAro> nielsm: frosch123: put something in the issue, to make sure you're not overlapping with JGR
21:19:57  <Samu> is that it? https://github.com/OpenTTD/OpenTTD/pull/7919#pullrequestreview-340119230
21:20:04  <DorpsGek_III> [OpenTTD/OpenTTD] nielsmh commented on issue #8093: Regular server desynchronisations from clients https://git.io/JfkmV
21:20:24  <DorpsGek_III> [OpenTTD/OpenTTD] frosch123 commented on issue #8093: Regular server desynchronisations from clients https://git.io/JfkmV
21:27:09  *** Samu has quit IRC
21:31:25  <andythenorth> hurrah https://www.tt-forums.net/viewtopic.php?p=1231375#p1231375
21:31:35  <andythenorth> I suspect those clown shoes are mine
21:32:44  <andythenorth> hmm
21:33:24  <andythenorth> oops https://github.com/OpenTTD/nml/blob/master/nml/actions/action0.py#L207
21:33:36  <TrueBrain> counting from 0, I assume? :D
21:33:50  <TrueBrain> so that is right; 16, 0 .. 15 :)
21:34:17  <andythenorth> "Roadtypes and Tramtypes share a pool of 64 IDs. Either feature can use as many IDs as needed (i.e. it is not limited to 32 per type), but both features combined are limited to 64 in total"
21:34:48  <TrueBrain> bboooooeeeeee
21:34:54  <andythenorth> 'oops'
21:35:11  <andythenorth> 0.5.1 soon then
21:36:57  <frosch123> andythenorth: looks like 64 is also a lie
21:36:59  <frosch123> it's only 63
21:37:44  <frosch123> i guess one id is used for "no road/tram" present
21:41:00  <andythenorth> nfo and nml wiki pages are wrong then :)
21:41:03  * andythenorth will fix, not now
21:49:27  *** frosch123 has quit IRC
21:51:07  *** nielsm has quit IRC
22:18:13  *** Progman has quit IRC
22:19:17  *** andythenorth has quit IRC
22:23:39  *** Yexo has quit IRC
22:33:41  *** Wolf01 has quit IRC
22:47:18  <FLHerne> I don't suppose anyone knows why there's a NOTHASBIT operator?
22:47:36  <FLHerne> As opposed to just... NOT(HASBIT())
23:01:56  <FLHerne> I guess because action7 supports that
23:08:06  *** gelignite_ has quit IRC
23:38:38  <Eddi|zuHause> hysterical raisins?

Powered by YARRSTE version: svn-trunk