Time Nick Message 00:31 Megaf #4553 00:31 ShadowBot https://github.com/minetest/minetest/issues/4553 -- Areas where there are a lot of caves and tunnels underground harm FPS 11:17 _Megaf Zeno`: hey, would you like to give minetest a lot of your time and to a huge work on it? #4553 11:17 ShadowBot https://github.com/minetest/minetest/issues/4553 -- Areas where there are a lot of caves and tunnels underground harm FPS 12:00 Fixer _Megaf: one way is to rework culing 12:01 Fixer _Megaf: i remember someone wrote in wiki that making culing working on per node basis should help a lot 12:37 _Megaf Fixer: I always expected the things to be per node based 12:37 Fixer it is mapblock based right now iirc, 16x16x16 12:40 est31 culling per node basis is totally wasteful 12:40 est31 culling happens on the CPU and the cpu should not handle single nodes 12:40 est31 not at least in every single frame 12:45 _Megaf and I would be so happy if someone worked on this #1620 12:45 ShadowBot https://github.com/minetest/minetest/issues/1620 -- [Request] ETA/Information/Status for /clearobjects 12:46 _Megaf est31: any idea how Minecraft does that? 12:46 est31 _Megaf, dunno but i doubt they do per node based culling 12:56 _Megaf maybe culling is not the problem 12:57 _Megaf maybe the way things are rendered is the problem 13:14 _Megaf everamzah: I can confirm your bug https://github.com/minetest/minetest/issues/4559#issuecomment-250462257 13:18 _Megaf sfan5: How a keypress thing is affecting the server build? I thought that build was regularly checked 13:41 _Megaf est31: do you think the server build fail is compiler specific or not? 13:41 _Megaf I'm trying with clang now, to see if it will build or not 13:41 est31 _Megaf, its not compiler specific 13:45 _Megaf oh, that is caused by the new sfan5's keytest thing 13:45 _Megaf sfan5: you should've tested that... 13:49 est31 yes its caused by sfan5 13:58 sfan5 _Megaf: i though travis tested server builds ¯\_(ツ)_/¯ 14:03 FirePowi Hi everyone, I'm having trouble with a node to register, I tried many things with ^[transform … But it don't transform at all. Please, have you got some exemples on how to use transform ? 14:06 _Megaf sfan5: est31: Indeed, very same output with clang 3.8 14:11 red-001 do trivial changes only need one approval to merge? 14:11 est31 yes 14:13 sfan5 FirePowi: could you pastebin the code of what you've tried? 14:14 sfan5 also this channel is for coredev not modding 14:14 FirePowi Oh, sorry… 14:14 FirePowi tiles = {"streets_asphalt.png^streets_asphalt_side.png^[transformT6","streets_asphalt.png"}, 14:14 FirePowi (No need for pastebin, one line ^^) 14:15 FirePowi I tried transform:T6 / transform:FY transformFY,… 14:16 sfan5 ^[transformFY should be working 14:19 FirePowi Oh, well Thanks ! 14:19 FirePowi I just didn't choose the good tranformation :p 14:19 red-001 could some review #4546? 14:19 ShadowBot https://github.com/minetest/minetest/issues/4546 -- Add multiline support to minetest.colorize by red-001 14:21 sfan5 pushing in 5 mins http://sprunge.us/WeBB 14:21 sfan5 (^ _Megaf) 14:23 est31 sfan5, the patch should be improved 14:23 sfan5 how 14:23 est31 make it export UNITTEST_SRCS and UNITTEST_CLIENT_SRCS 14:24 sfan5 why 14:24 est31 then update src/CMakeLists 14:24 est31 .txt 14:24 est31 to add UNITTEST_CLIENT_SRCS to client_SRCS 14:24 red-001 can't #420 be closed? 14:24 ShadowBot https://github.com/minetest/minetest/issues/420 -- Call on_leaveplayer for all connected players on before shutdown 14:25 sfan5 oh that makes sense 14:25 est31 :) 14:31 sfan5 est31: how's this http://sprunge.us/dJNh?diff? 14:32 est31 +1 14:32 sfan5 ok, pushing 18:11 _Megaf red-001: https://github.com/minetest/minetest/pull/4554#issuecomment-250546663 :) 18:11 _Megaf #4554 18:11 ShadowBot https://github.com/minetest/minetest/issues/4554 -- Add /grantme command by red-001 18:12 _Megaf sfan5: paramat: ^ 18:19 sfan5 i already approved it 18:19 sfan5 it needs a second approval 18:21 red-001 Doesn't the trivial label mean only one approval is needed? 18:36 _Megaf sfan5: eh, that was an easy fix 18:39 sfan5 red-001: that only applies to "trivial" in the sense of lines changed 18:39 sfan5 (at least that's how i'd interpret the rule) 18:39 _Megaf sfan5: so, why you added the label trivial? 18:39 _Megaf I don't think it's a trivial change there 18:40 sfan5 because i thought that the changes are trivial in the sense of changed code logic 18:40 sfan5 i guess 18:40 sfan5 (it mostly moves code around) 18:52 _Megaf sfan5: the fix is merged to master already, isn't it? 18:52 _Megaf The one to fix the server compilation 18:52 sfan5 yes 18:53 _Megaf ok, will update my server then