Time Nick Message 12:36 sfan5 merging #11352 in 10m 12:36 ShadowBot https://github.com/minetest/minetest/issues/11352 -- Rework Settings to support arbitrary hierarchies by sfan5 13:22 sfan5 oops 15:32 rubenwardy I'm using the "Roadmap: Approval needed" to track PRs that aren't on the roadmap and need a core dev concept approval: https://github.com/minetest/minetest/pulls?q=is%3Apr+is%3Aopen+label%3A%22Roadmap%3A+Needs+approval%22+sort%3Acreated-asc 15:33 rubenwardy If you support it, and by support I mean you also plan on reviewing it (:D), then you can replace that label with "Supported by core dev" 15:39 VanessaE has anyone run into an issue where a new LBM doesn't always run on every block of just-loaded (but OLD) terrain, even after changing the LBM's name, if the `run_at_every_load = false` ? I just had to set this to true on one of my mods to force it to do its job.. 15:40 VanessaE (where by "OLD" I mean it's not newly-emerged, but old stuff that's been in the DB since before the LBM) 15:46 VanessaE -the 16:08 rubenwardy merging #11375 in 5 16:08 ShadowBot https://github.com/minetest/minetest/issues/11375 -- Perform some quality assurance for translation strings by Wuzzy2 16:34 sfan5 those are long 5 minutes 16:35 Krock hey rubenwardy 16:35 rubenwardy oh right 16:35 Krock quick jump into the TARDIS 16:35 rubenwardy we need a bot for this 16:36 Krock well, there's ShadowBot where a timer plugin could be added 16:39 celeron55 let's just make an irc bot that operates github so that we don't need to open our web browsers 16:39 celeron55 also who needs text editors, just implement ed in the bot 16:40 celeron55 it would be an interesting experiment for a smaller collaborative project actually 8) 16:40 sfan5 how about a bot that writes the code for us? 16:40 sfan5 it would be the obvious next step 16:42 celeron55 well, i mean, i can give you a concept approval but you better not ask help making it 16:42 Krock if I only could find the link to the AI-generated fake source code webpage 16:50 Krock https://doesnotexist.codes/ 16:50 Krock sorry it took a while 17:01 kilbith I found a critical regression related to item lists 17:05 kilbith okay that's what I suspected, this commit absolutely needs reverting as it causes regressions: https://github.com/minetest/minetest/commit/2db6b07d 17:06 Krock kilbith: have you checked today's new PRs yet? 17:06 kilbith get the i3 mod and try moving items onto the trash slots, or clicking the trash button, or clicking the compress button 17:06 kilbith * slot 17:06 kilbith Krock: yes 17:07 Krock then you know that it's already in the progress of being fixed 17:07 kilbith ah no I didn't see 11382 17:09 kilbith https://github.com/minetest/minetest/pull/10788#issuecomment-866410646 17:09 kilbith holy shit sfan5 what are you doing 17:09 kilbith you're much faster on merging your PRs 17:11 kilbith we have Lars giving support to the best compression algo in the world and none is giving a shit 17:21 Krock why are we doing double-compression? once for MapNode::deSerializeBulk and again for the node metadata? 17:21 Krock a mistake in the current design? 17:25 sfan5 there's no double compression right now 17:25 sfan5 or do you mean in the PR? 17:26 sfan5 (though there shouldn't be any there either) 17:27 Krock first: https://github.com/minetest/minetest/pull/10788/files#diff-958d818edcdc5e6ca5082d092312ddc89aeb2239ccfbdcc6ceb09ca25d3065a6R515 17:27 Krock second: https://github.com/minetest/minetest/pull/10788/files#diff-958d818edcdc5e6ca5082d092312ddc89aeb2239ccfbdcc6ceb09ca25d3065a6R533 17:27 Krock moment. this seems to be segmented; separate 17:28 sfan5 yes, there are two compressed streams in current mapblocks 19:39 sfan5 celeron55: the other PR has this demo video https://user-images.githubusercontent.com/17455197/120680769-0226bc00-c49b-11eb-95db-b351aa23c7ba.mp4 19:39 sfan5 but on second thought I have no idea if corals actually look like this