IRC logs for #openttd on OFTC at 2023-03-02
            
00:14:05 *** MaverickFischer_ has quit IRC ()
00:15:44 *** MaverickFischer has joined #openttd
00:43:54 *** HerzogDeXtEr has quit IRC (Read error: Connection reset by peer)
01:26:58 *** Wormnest has quit IRC (Ping timeout: 480 seconds)
01:31:57 *** WormnestAndroid has quit IRC (Ping timeout: 480 seconds)
02:00:54 *** WormnestAndroid has joined #openttd
02:00:56 *** WormnestAndroid has quit IRC (Read error: Connection reset by peer)
02:01:39 *** WormnestAndroid has joined #openttd
02:02:04 *** Wormnest has joined #openttd
02:26:03 *** WormnestAndroid has quit IRC (Ping timeout: 480 seconds)
02:54:18 *** Wormnest has quit IRC (Ping timeout: 480 seconds)
03:10:03 *** WormnestAndroid has joined #openttd
03:18:08 *** WormnestAndroid has quit IRC (Ping timeout: 480 seconds)
03:20:56 *** WormnestAndroid has joined #openttd
03:28:58 *** WormnestAndroid has quit IRC (Ping timeout: 480 seconds)
03:31:58 *** D-HUND has joined #openttd
03:32:12 *** Wormnest has joined #openttd
03:35:27 *** debdog has quit IRC (Ping timeout: 480 seconds)
03:37:06 *** WormnestAndroid has joined #openttd
03:41:25 *** D-HUND is now known as debdog
03:45:08 *** WormnestAndroid has quit IRC (Ping timeout: 480 seconds)
03:46:31 *** WormnestAndroid has joined #openttd
03:46:35 *** WormnestAndroid has quit IRC (Read error: Connection reset by peer)
03:46:43 *** WormnestAndroid has joined #openttd
04:02:58 *** WormnestAndroid has quit IRC (Ping timeout: 480 seconds)
04:03:24 *** WormnestAndroid has joined #openttd
04:14:18 *** TROILUS has quit IRC (Read error: Connection reset by peer)
04:14:19 *** TROILUS7 has joined #openttd
04:14:20 *** TROILUS7 is now known as TROILUS
04:18:29 *** TROILUS has quit IRC (Remote host closed the connection)
04:18:30 *** TROILUS9 has joined #openttd
04:18:30 *** TROILUS9 is now known as TROILUS
04:25:27 *** Wormnest has quit IRC (Ping timeout: 480 seconds)
04:27:11 *** Wormnest has joined #openttd
04:27:48 *** WormnestAndroid has quit IRC (Ping timeout: 480 seconds)
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:01:59 <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 (Ping timeout: 480 seconds)
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:14 <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:54 <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:52 <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:46 <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:11 <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:23 <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:30 <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:36 <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 (resistance.oftc.net coherence.oftc.net)
13:44:56 *** APTX has quit IRC (resistance.oftc.net coherence.oftc.net)
13:44:56 *** birdjj has quit IRC (resistance.oftc.net coherence.oftc.net)
13:45:52 *** Flygon has joined #openttd
13:45:52 *** APTX has joined #openttd
13:45:52 *** birdjj has joined #openttd
13:46:55 <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:06 <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 (Read error: Connection reset by peer)
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:00 <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 (Ping timeout: 480 seconds)
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:34 * 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:25 <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:53 *** 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:28 <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 (Ping timeout: 480 seconds)
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 (Read error: Connection reset by peer)
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:03 *** 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:27:59 <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:06 <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:21 <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:04 <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 (Ping timeout: 480 seconds)
22:24:22 *** nielsm has quit IRC (Ping timeout: 480 seconds)
22:37:01 *** Wolf01 has quit IRC (Quit: Once again the world is quick to bury me.)
22:53:11 *** gelignite has quit IRC (Quit: Stay safe!)
23:10:25 *** HerzogDeXtEr has quit IRC (Read error: Connection reset by peer)