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?