Times are UTC Toggle Colours
01:26:58 *** Wormnest has quit IRC 01:31:57 *** WormnestAndroid has quit IRC 02:00:54 *** WormnestAndroid has joined #openttd 02:01:39 *** WormnestAndroid has joined #openttd 02:02:04 *** Wormnest has joined #openttd 02:26:03 *** WormnestAndroid has quit IRC 02:54:18 *** Wormnest has quit IRC 03:10:03 *** WormnestAndroid has joined #openttd 03:18:08 *** WormnestAndroid has quit IRC 03:20:56 *** WormnestAndroid has joined #openttd 03:28:58 *** WormnestAndroid has quit IRC 03:31:58 *** D-HUND has joined #openttd 03:32:12 *** Wormnest has joined #openttd 03:35:27 *** debdog has quit IRC 03:37:07 *** WormnestAndroid has joined #openttd 03:41:25 *** D-HUND is now known as debdog 03:45:08 *** WormnestAndroid has quit IRC 03:46:31 *** WormnestAndroid has joined #openttd 03:46:43 *** WormnestAndroid has joined #openttd 04:02:58 *** WormnestAndroid has quit IRC 04:03:24 *** WormnestAndroid has joined #openttd 04:14:18 *** TROILUS has quit IRC 04:14:19 *** TROILUS7 has joined #openttd 04:14:20 *** TROILUS7 is now known as TROILUS 04:18:29 *** TROILUS has quit IRC 04:18:30 *** TROILUS9 has joined #openttd 04:18:30 *** TROILUS9 is now known as TROILUS 04:25:27 *** Wormnest has quit IRC 04:27:11 *** Wormnest has joined #openttd 04:27:48 *** WormnestAndroid has quit IRC 04:27:56 *** WormnestAndroid has joined #openttd 04:28:22 *** Wormnest has quit IRC 04:36:15 *** keikoz has joined #openttd 05:42:04 *** Flygon has joined #openttd 07:25:32 *** sla_ro|master has joined #openttd 08:31:09 *** HerzogDeXtEr has joined #openttd 09:40:23 <DorpsGek> [OpenTTD/OpenTTD] merni-ns opened issue #10530: [Crash]: Game crashes when closing the window if it is narrower than the confirmation message https://github.com/OpenTTD/OpenTTD/issues/10530 09:45:45 <DorpsGek> [OpenTTD/OpenTTD] LordAro commented on issue #10530: [Crash]: Game crashes when closing the window if it is narrower than the confirmation message https://github.com/OpenTTD/OpenTTD/issues/10530 10:02:00 <petern> Ah the old min > max clamp. 10:05:47 <andythenorth> anyone used? https://www.html-tidy.org/ 10:05:57 <andythenorth> I was looking for black, but for html, apparently that might be it 10:08:29 *** Westie has quit IRC 10:15:16 <pickpacket> glx[d]: how come a plane is 3 vehicles? 10:15:45 <pickpacket> 64k head vehicles sounds like more than anyone needs, though 10:17:21 <petern> Body, shadow and (for helicoptors) rotor. 10:17:40 <pickpacket> Oh 10:18:15 <petern> Body and shadow is the magic that allows a plane to carry passengers and mail. 10:18:27 <pickpacket> i assumed the rotor was just an animation 10:18:39 <pickpacket> the shadow carries goods? 10:19:15 <pickpacket> *cargo 10:25:55 <andythenorth> yes 10:26:00 <andythenorth> ships need a wake 😛 10:26:04 <andythenorth> for the same outcome 10:26:16 <andythenorth> or actually...not just inventing amusing hacks 10:28:44 <LordAro> andythenorth: ships are dead? 10:28:48 *** Merni has joined #openttd 10:28:48 <Merni> andythenorth: how about "exhaust fumes" as a vehicle for buses to carry mail then 10:34:22 <LordAro> petern: so can helicopters have 3 cargoes? :p 10:34:53 <LordAro> and has anyone brought up the fact that rotors don't cast a shadow? 10:34:59 <petern> "Yes" but the code doesn't do it. 10:35:37 <petern> Fast spinning objects don't cast much of a shadow anyway, when it's at rest you can pretend it's aligned with the body 🙂 10:35:47 <LordAro> :D 10:46:25 <petern> pickpacket: It is an animation that's attached to a vehicle. 10:47:11 <petern> Vehicle smoke and sparking effects are also vehicles, but they don't live very long. 10:47:33 <petern> So any cargo would be lost... 10:54:11 <pickpacket> ah 10:57:52 <pickpacket> petern: but what do you mean by "Body and shadow is the magic that allows a plane to carry passengers and mail."? 10:58:14 <petern> Vehicles can only carry one cargo type. 10:58:42 <petern> Planes are able to carry both passengers and mail, because they have both a body vehicle and the shadow vehicle. 10:59:41 <pickpacket> oh. That's pretty cool 11:02:27 <DorpsGek> [OpenTTD/OpenTTD] SamuXarick updated pull request #10330: Codechange: Rework 'start_date' parameter for AIs as a game setting https://github.com/OpenTTD/OpenTTD/pull/10330 11:10:05 <Xarick> wow, i don't know how to rebase 10373 11:10:16 <Xarick> squirrel_helper is different 11:10:21 <Xarick> too different 11:28:10 <andythenorth> ship shadows? 11:28:13 <andythenorth> train shadows? 11:28:28 <andythenorth> "imagine the pathfinding CPU impact" 11:36:13 <pickpacket> if I have a train with a passenger-carrying engine and two goods vans, how do I refit it so that I keep the passenger compartment and have two different cargoes in the vans? 11:51:07 *** gelignite has joined #openttd 11:51:11 <DorpsGek> [OpenTTD/OpenTTD] SamuXarick updated pull request #10373: Allow Scripts to convert returned uint32 values to int64 https://github.com/OpenTTD/OpenTTD/pull/10373 11:52:48 <DorpsGek> [OpenTTD/OpenTTD] SamuXarick updated pull request #10376: Fix: [Script] Ensure building/removing rail lines can't be backwards and allow building/removing rail lines coming from or heading to the void https://github.com/OpenTTD/OpenTTD/pull/10376 11:59:10 <glx[d]> pickpacket: You can't, refitting is global 11:59:41 <glx[d]> Though you can do in multiple steps 12:00:07 <Xarick> holy mother of conflicts 12:00:21 <glx[d]> Refit ebgin+wagons, detach wagons, refit engine, reattach wagons 12:02:14 <DorpsGek> [OpenTTD/OpenTTD] SamuXarick updated pull request #10411: Expose more functions to game scripts https://github.com/OpenTTD/OpenTTD/pull/10411 12:02:59 <Xarick> about 20 conflicts on that one 12:03:08 <Rubidium> Xarick: then you haven't seen my latest PR yet ;) 12:05:26 <petern> glx[d]: Doesn't refit allow to select which wagons to refit? Or is that only for cargo subtype? 12:06:24 <glx[d]> Can't remember, I don't play enough 🤣 12:06:58 <glx[d]> But could be some magic ctrl 12:07:14 <Xarick> i don't understand what changed <https://github.com/OpenTTD/OpenTTD/pull/10458/conflicts> 12:07:28 <petern> https://cdn.discordapp.com/attachments/1008473233844097104/1080823687571320892/image.png 12:07:52 <petern> Not even magic ctrl key 🙂 12:10:07 <Xarick> you mean, any tiny bit of change in that code block will trigger a conflict? 12:10:13 <Xarick> omg 12:11:11 <Xarick> and it was me again 12:11:14 <Xarick> lol, funny 12:11:32 <Xarick> I cause my own conflicts 12:12:27 <petern> If it's a very minor change, VS Code gives you an option to combine changes easily enough. 12:12:44 <glx[d]> <https://github.com/OpenTTD/OpenTTD/commit/fde7028a956e9e56dd9ba66ba8b0d9e2781fb531> 12:13:25 <glx[d]> Yes it was you, a single change in a to be adde/removed block will conflict 12:14:18 <glx[d]> But this one is easy as it's just a removal 12:15:15 <Xarick> i think im closing that PR 12:15:28 <Xarick> it's been 2 weeks or so and nothing about it 12:19:24 <DorpsGek> [OpenTTD/OpenTTD] SamuXarick closed pull request #10458: Codechange: Move AI settings back to ai_gui https://github.com/OpenTTD/OpenTTD/pull/10458 12:20:03 <DorpsGek> [OpenTTD/OpenTTD] SamuXarick opened pull request #10531: Codechange: Cache ScriptConfig for the slot in constructor https://github.com/OpenTTD/OpenTTD/pull/10531 12:23:53 <petern> Auto-andy 12:26:31 <DorpsGek> [OpenTTD/OpenTTD] SamuXarick updated pull request #8492: Fix: Don't allow towns to terraform certain floodable tiles https://github.com/OpenTTD/OpenTTD/pull/8492 12:26:40 <Xarick> all rebases done! 12:28:44 <pickpacket> petern: oh! Cool! 12:53:31 <glx[d]> Forgot to build before push in #8492 ? 13:11:46 <Xarick> oh 8492 failed 13:13:56 <Xarick> oh i see, water.h was removed 13:14:02 <Xarick> need to be included again 13:15:28 <Xarick> maybe I'll do what TallTyler suggested if now I am requiring to add includes 13:15:35 <Xarick> move it to town 13:18:25 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler commented on pull request #10531: Codechange: Cache ScriptConfig for the slot in constructor https://github.com/OpenTTD/OpenTTD/pull/10531#pullrequestreview-1321793973 13:36:37 <DorpsGek> [OpenTTD/OpenTTD] SamuXarick updated pull request #8492: Fix: Don't allow towns to terraform certain floodable tiles https://github.com/OpenTTD/OpenTTD/pull/8492 13:40:35 <andythenorth> petern: I stopped closing things, it was a bit like 2048, once you've got a high score, you can stop playing 13:40:56 <andythenorth> I reduced about 900 FS to about 100 GH issues, nobody else is going to beat that 🙂 13:41:02 <andythenorth> greatest of all time etc 13:42:50 <DorpsGek> [OpenTTD/OpenTTD] SamuXarick commented on pull request #8492: Fix: Don't allow towns to terraform certain floodable tiles https://github.com/OpenTTD/OpenTTD/pull/8492#pullrequestreview-1321846485 13:44:56 *** Flygon has quit IRC 13:44:56 *** APTX has quit IRC 13:44:56 *** birdjj has quit IRC 13:45:52 *** Flygon has joined #openttd 13:45:52 *** APTX has joined #openttd 13:45:52 *** birdjj has joined #openttd 13:46:56 <DorpsGek> [OpenTTD/OpenTTD] SamuXarick commented on pull request #10531: Codechange: Cache ScriptConfig for the slot in constructor https://github.com/OpenTTD/OpenTTD/pull/10531#pullrequestreview-1321853724 14:00:26 <DorpsGek> [OpenTTD/OpenTTD] SamuXarick commented on pull request #10531: Codechange: Cache ScriptConfig for the slot in constructor https://github.com/OpenTTD/OpenTTD/pull/10531#pullrequestreview-1321878487 14:03:07 <DorpsGek> [OpenTTD/OpenTTD] glx22 commented on pull request #10531: Codechange: Cache ScriptConfig for the slot in constructor https://github.com/OpenTTD/OpenTTD/pull/10531#pullrequestreview-1321883305 14:13:41 <Xarick> how can there be more roadstop id than station id? 14:13:47 <Xarick> did i screw up? 14:16:50 <Xarick> oh! each roadstop tile counts as an index? 14:16:52 <Xarick> wow 14:17:06 <Xarick> very interesting 14:22:02 <petern> You can have more roadstops than stations. 14:22:40 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler commented on pull request #10531: Codechange: Cache ScriptConfig for the slot in constructor https://github.com/OpenTTD/OpenTTD/pull/10531#pullrequestreview-1321921123 14:22:57 <petern> If each roadstop tile used the same ID then it wouldn't need an ID at all. 14:29:45 *** nielsm has joined #openttd 14:51:43 <audigexJon> petern: Now you’re thinking in MongoDB 15:13:57 <Xarick> the AI/GS split is such a headache 15:15:04 <Xarick> i can't rebase <https://github.com/SamuXarick/OpenTTD/tree/max_no_competitors-%3D-15-v33-r27931.patch> 15:15:50 <Xarick> it was a gui change to the AI/GS window when they were together 15:16:14 <Xarick> now they're split 15:16:33 <Xarick> a big chunk on ai_gui is also moved 15:16:42 <Xarick> terrible for rebasing 15:33:54 <Xarick> okay, all that's left to rebase is the big chunk ai_gui.cpp 😦 the other mini file changes are don 16:17:46 *** TROILUS6 has joined #openttd 16:18:59 *** TROILUS has quit IRC 16:19:00 *** TROILUS6 is now known as TROILUS 17:14:12 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 updated pull request #10529: Add precondition checks for deity/company mode https://github.com/OpenTTD/OpenTTD/pull/10529 17:15:00 * Rubidium wonders how many GS functions currently crash under company mode in an invalid company 17:18:49 <LordAro> "many" 17:20:01 <Rubidium> at least the whole GSEngine looks suspect 17:20:40 *** Wormnest has joined #openttd 17:23:12 *** tokai|noir has joined #openttd 17:23:12 *** ChanServ sets mode: +v tokai|noir 17:29:59 *** tokai has quit IRC 17:36:45 <Rubidium> yes, it will CTD... but testing it is a PITA as a company needs to be gone first 17:38:56 <glx[d]> dedicated server ? 17:47:16 <andythenorth> can clean servers? 17:47:35 * andythenorth has no idea what cleaning a company actually does 17:54:38 <Rubidium> glx[d]: I rather trigger it directly from a regression test or something 17:55:17 <andythenorth> hmm 17:55:24 <andythenorth> can GS actually invalidate companies? 17:55:38 <andythenorth> maybe it should be able to 😛 17:55:42 <glx[d]> IIRC it's possible to switch to invalid company with company mode 17:55:52 <andythenorth> "if you don't make your goal by this date, your company will be closed" 17:56:30 <andythenorth> no GSCompany.Remove() https://docs.openttd.org/gs-api/classGSCompany.html 18:00:37 <Rubidium> glx[d]: an invalid company would make it INVALID_COMPANY, and HasBit(company_avail (i.e. -1), 0xFF) returns false, where it would return true for companies with ID 0..15 18:02:36 <Rubidium> and that doesn't crash 18:03:08 <andythenorth> did I imagine there's some GS for actual regression testing? 18:03:24 <andythenorth> someone linked a repo maybe? 18:06:49 *** Wolf01 has joined #openttd 18:10:47 <glx[d]> there's no regression testing for GS 18:11:50 <glx[d]> theorically a GS could test all AI stuff, but many parts are not yet allowed to GS 18:12:57 <glx[d]> Rubidium: Oh I see, companymode is not updated on company disparition 18:37:11 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 opened pull request #10532: Fix: deity should see all signs, AIs should not see signs of deity https://github.com/OpenTTD/OpenTTD/pull/10532 18:41:52 *** virtualrandomnumber has joined #openttd 18:42:39 *** virtualrandomnumber has quit IRC 18:50:47 <DorpsGek> [OpenTTD/OpenTTD] DorpsGek pushed 1 commits to master https://github.com/OpenTTD/OpenTTD/commit/58aa7692820733f20ae6911147c8806df3f08e17 18:50:48 <DorpsGek> - Update: Translations from eints (by translators) 18:51:23 <glx[d]> funny how only VS warns when it should be an error everywhere 18:51:29 <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on pull request #10532: Fix: deity should see all signs, AIs should not see signs of deity https://github.com/OpenTTD/OpenTTD/pull/10532#issuecomment-1452386796 18:56:39 <DorpsGek> [OpenTTD/OpenTTD] glx22 commented on pull request #10532: Fix: deity should see all signs, AIs should not see signs of deity https://github.com/OpenTTD/OpenTTD/pull/10532#issuecomment-1452396557 18:57:19 <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on pull request #10532: Fix: deity should see all signs, AIs should not see signs of deity https://github.com/OpenTTD/OpenTTD/pull/10532#issuecomment-1452397878 19:05:08 *** Wormnest has quit IRC 19:15:06 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 commented on pull request #10532: Fix: deity should see all signs, AIs should not see signs of deity https://github.com/OpenTTD/OpenTTD/pull/10532#issuecomment-1452423565 19:18:40 <DorpsGek> [OpenTTD/OpenTTD] frosch123 commented on pull request #10532: Fix: deity should see all signs, AIs should not see signs of deity https://github.com/OpenTTD/OpenTTD/pull/10532#issuecomment-1452427179 19:19:08 *** Flygon has quit IRC 19:23:47 <DorpsGek> [OpenTTD/BaNaNaS] frosch123 opened pull request #129: Change: migrate OpenTTD user Ssabi to GitHub user Ssabi84 https://github.com/OpenTTD/BaNaNaS/pull/129 19:30:17 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 commented on pull request #10532: Fix: deity should see all signs, AIs should not see signs of deity https://github.com/OpenTTD/OpenTTD/pull/10532#issuecomment-1452439342 19:30:20 <DorpsGek> [OpenTTD/BaNaNaS] TrueBrain approved pull request #129: Change: migrate OpenTTD user Ssabi to GitHub user Ssabi84 https://github.com/OpenTTD/BaNaNaS/pull/129#pullrequestreview-1322487434 19:30:33 <DorpsGek> [OpenTTD/BaNaNaS] TrueBrain approved pull request #125: Change: add 'qamil95' and 'Brickblock1' as co-authors to Polish Train Set https://github.com/OpenTTD/BaNaNaS/pull/125#pullrequestreview-1322487755 19:34:07 *** Wormnest has joined #openttd 19:50:55 <DorpsGek> [OpenTTD/BaNaNaS] frosch123 merged pull request #129: Change: migrate OpenTTD user Ssabi to GitHub user Ssabi84 https://github.com/OpenTTD/BaNaNaS/pull/129 20:02:04 *** sla_ro|master has quit IRC 20:09:03 <DorpsGek> [OpenTTD/BaNaNaS] frosch123 commented on pull request #125: Change: add 'qamil95' and 'Brickblock1' as co-authors to Polish Train Set https://github.com/OpenTTD/BaNaNaS/pull/125#issuecomment-1452481536 20:09:13 <DorpsGek> [OpenTTD/BaNaNaS] frosch123 merged pull request #125: Change: add 'qamil95' and 'Brickblock1' as co-authors to Polish Train Set https://github.com/OpenTTD/BaNaNaS/pull/125 20:14:55 <DorpsGek> [OpenTTD/OpenTTD] glx22 commented on pull request #10529: Add precondition checks for deity/company mode https://github.com/OpenTTD/OpenTTD/pull/10529#pullrequestreview-1322544566 20:16:07 <Xarick> Rect tr = r.Shrink(WidgetDimensions::scaled.matrix); 20:16:23 <Xarick> how do i draw icons next to text, 😦 20:17:09 <glx[d]> icon position = text position + text bounding box 20:17:24 <Xarick> this rect changes ... 😦 20:19:05 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 commented on pull request #10529: Add precondition checks for deity/company mode https://github.com/OpenTTD/OpenTTD/pull/10529#pullrequestreview-1322549691 20:26:46 <Xarick> what are the replacements for WD_MATRIX_RIGHT WD_MATRIX_LEFT WD_FRAMERECT_LEFT WD_FRAMERECT_RIGHT? 20:27:27 <petern> WidgetDimensions::scaled.matrix and WidgetDimensions::scaled.framerect 20:27:36 <Xarick> thx 20:28:00 <petern> Draw your icon at tr.WithWidth(sprite_width, rtl) 20:28:15 <petern> Draw your text at tr.Indent(sprite_width + WidgetDimensions::scaled.hsep_normal, rtl) 20:44:07 <Xarick> https://cdn.discordapp.com/attachments/1008473233844097104/1080953706184462446/image.png 20:44:24 <Xarick> doesn't look aligned enough 20:44:30 <petern> Almost. 20:45:09 <Xarick> text needs to go up some pixels 20:45:12 <Xarick> and icon too 20:45:28 <Xarick> author, version i didn't touch 20:47:37 <petern> In master the Available AIs list is correctly aligned, so go back and see what you added... 20:48:49 <Xarick> i added y offset 20:48:57 <Xarick> guess I dont need it 20:58:14 <Xarick> <https://gist.github.com/SamuXarick/57b79067c5635d894c02f5aebc0fe5bb> does this look good? 21:04:22 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 updated pull request #10529: Add precondition checks for deity/company mode https://github.com/OpenTTD/OpenTTD/pull/10529 21:41:23 <andythenorth> lol grf variable to read sign at a specific tile index? when? 21:41:33 * andythenorth reading the AI-GS sign library 21:42:45 <Xarick> question, if I remove a String from english.txt, I don't need to remove it from the other languages, right? 21:43:09 <Xarick> it will do it automaticaly? 21:43:22 <petern> https://cdn.discordapp.com/attachments/1008473233844097104/1080968619967643718/image.png 21:43:22 <petern> Choo choo? 21:45:30 <Rubidium> Xarick: the translator will do it 21:45:51 <andythenorth> petern: is that a priority merge though? 21:46:09 <petern> Not a block signal in sight 21:46:43 <TrueBrain> I wonder what happens if a car tries to cross 21:46:58 <andythenorth> which signal grf is that on the left? 21:47:07 <andythenorth> it's a single yellow with red edges? 21:47:43 <Xarick> should I care about GS config window? 21:47:59 <Xarick> i guess i should 21:48:05 <andythenorth> petern: how many hp / t? 21:48:11 <andythenorth> you'll need 13.0 to see that though 22:12:17 *** keikoz has quit IRC 22:24:22 *** nielsm has quit IRC 22:37:01 *** Wolf01 has quit IRC 22:53:11 *** gelignite has quit IRC 23:10:25 *** HerzogDeXtEr has quit IRC