Time Nick Message 11:12 kilbith I really don't like the 3D line PR; it's better to have a more generalist API to draw any kind of primitives, not only an ill-considered callback for a line 11:12 kilbith I don't know what you have on mind to accept that 11:12 kilbith yes, it's flashy but it doesn't cut it API wise 11:18 kilbith if I didn't care about API design I'd have bothered to submit e.g. this: https://www.youtube.com/watch?v=swfWAO7H_lA 11:18 kilbith if your design sucks, as flashy as it sounds, it ain't worth it 11:20 ROllerozxa #13081 has been fixed and should be ready for merge 11:20 ShadowBot https://github.com/minetest/minetest/issues/13081 -- Don't install Development Test by rollerozxa 13:49 MTDiscord Well, API for drawing other primitives can be added later. I think it isn't worth to bloat the PR up to the incredible extents since it will make it difficult to review and test it and therefore a less possibility to be merged. For now it is possible to use hacks and to render by means of them some other figures as rectangles, cylinders, circles, even boxes just adjusting in the correct way their type, width and positions. Also, I 13:49 MTDiscord think drawing lines is more priority and in demand than other primitives since it will rather be used a way more frequent by modders. 13:54 kilbith you didn't yet bring the foundation of a good API design 13:55 kilbith accepting your PR now implies that we'll have to get around a bad design in the future 14:05 kilbith also it doesn't make any sense that it uses its own API; Krock is 100% right to say that AO should be extended 14:51 nrz i also agree with Krock, as the creator of intermediaite UnitSAO between LuaEntitySAO and SAO 🙂 14:53 nrz anoying the MTDiscord bot is not talking like anyone but as Notice now, and matterbridge discard msg 😄 16:41 jwmhjwmh sfan5: rubenwardy: Do your approvals for #13081 still apply? 16:41 ShadowBot https://github.com/minetest/minetest/issues/13081 -- Don't install Development Test by rollerozxa 16:41 rubenwardy yes 16:43 MTDiscord @kilbith If you insist so on remaking the current API with capability to render other primitives, it means you know usecases of them, otherwise you'd not suggest. Which usecases would they have? Personally I consider adding rendering API for lines is quite enough in most situations like lasers, fishing rods, ropes, cables and etc and it doesn't make sense to add support for rendering something else. Otherwise, if a modder needs to 16:43 MTDiscord render something more complicative like triangle, sphere, cone he can use entities with mesh visual type. 16:48 MTDiscord Modding API should be minimalistic and add only what is really needed. What to add that for if it won't have a real usage? 17:00 lhofhansl x2048: FYI. I had the old culler enabled, and looking at the code that branch does not have the shortlist. 17:01 lhofhansl What is still happening is that sometime a block is not sent from the server. 17:01 lhofhansl Looking forward to a PR! :) 19:28 MTDiscord Merging #13081 in 5 minutes. 19:28 ShadowBot https://github.com/minetest/minetest/issues/13081 -- Don't install Development Test by rollerozxa 19:34 MTDiscord Merged 19:36 MTDiscord Can someone help with access to dev.minetest.net? It looks like I lost my password, but the reset button won't work. 19:55 jwmhjwmh Merging #13131 in 5m. 19:55 ShadowBot https://github.com/minetest/minetest/issues/13131 -- Always set globals in `__newindex` in strict.lua by TurkeyMcMac 20:31 MTDiscord Merging #13124 and #13064 in 5m 20:31 ShadowBot https://github.com/minetest/minetest/issues/13124 -- Remove mapblock cache for mesh generation. by x2048 20:31 ShadowBot https://github.com/minetest/minetest/issues/13064 -- DevTest: `chest_of_everything` mod: Add search, bag, improve formspec by Wuzzy2 20:40 MTDiscord Done 23:02 Zughy[m] sfan5: should I close this or not? According to your label #12954 23:02 ShadowBot https://github.com/minetest/minetest/issues/12954 -- [Fix] The trusted mods are prohibited from using IO functions. by snowyu