Time Nick Message 00:19 MTDiscord uh some recent changes to irrlicht are causing the dreaded spinning camera bugs 00:48 MTDiscord four month old build -> still spinning 00:48 MTDiscord launching MT from a terminal on mac prevents this 10:20 Wuzzy hey sfan5, let's talk about 11110 i guess its easier in chat ... 10:21 Wuzzy so are you still in the review process or are you done? 10:21 Wuzzy (i agree with the current new comments, however) 10:22 sfan5 #11110 10:22 ShadowBot https://github.com/minetest/minetest/issues/11110 -- Fix number of tool uses being off by 1..32767 by Wuzzy2 10:22 sfan5 no thats all 10:22 sfan5 I didn't test the PR yet but I assume I'll find that it works 10:24 Wuzzy cool, thanks 10:29 MTDiscord I'm getting the impression that somewhen during the last 5 years, character.b3d lost it's normals. 10:32 MTDiscord https://github.com/minetest/minetest_game/commit/2e7b509a94dbbe2c407d47e9bd1ad1d97c403cfe#diff-bc34e796b0cb0c4371a3c073649334c4abfee8b0780758b41c6bb2725ae2771e removed the normals from character.b3d! Please revert! 10:33 MTDiscord I always found it suspicious how that commit reduced the size of character.b3d 10:41 MTDiscord https://github.com/minetest/minetest_game/pull/2901 10:44 sfan5 instead of creating a revert pull it would be better to create one that re-exports the model with normals 10:47 MTDiscord I am not good with exporting B3D models though 10:47 MTDiscord s/with/at 10:48 MTDiscord I'll try GreenXenith's exporter now (https://github.com/GreenXenith/io_scene_b3d), we'll see 11:09 MTDiscord Turns out the exporter apparently has some kind of config for exporting normals which is by default set to false. I just hardcoded true for now and got a fixed model with normals: 11:09 MTDiscord https://cdn.discordapp.com/attachments/747163566800633906/898165641004081194/character.b3d 11:17 MTDiscord There you go, game#2902 11:17 ShadowBot https://github.com/minetest/minetest_game/issues/2902 -- Readd character.b3d normals by appgurueu 14:10 appguru Krock: Can you provide the second approval? I'd like to see it merged quickly - a mod of mine needs those normals. 14:16 sfan5 are you sure your users use 5.5.0-dev 14:18 MTDiscord Well, only 5.4.x is missing the normals so far. And indeed, my mod might actually force users to use 5.5.0-dev as it will likely use the new dynamic media API. 14:21 MTDiscord Oh, and the PNG writing utility ;) 14:27 MTDiscord Question related to that: The only advantage I see in using the embedded [png modifier is that it would only be sent to players who actually receive the texture, while dynamic media is sent to either all clients, or single clients specified by the modder. On the other hand, the embedded modifier would probably be sent over the wire many times, whenever object properties / formspecs / whatever uses the texture is sent, plus there is some 14:27 MTDiscord overhead from the base64 encoding and texture parsing. Currently, both can't be removed from caches and stay there forever. 14:30 sfan5 you have understood why I was so against adding it 14:58 MTDiscord Lazy loading of dynamic media would be nice 15:02 celeron55 spinny loading icons everywhere 16:33 MTDiscord It looks like player:get_look_dir() is slightly off sometimes? 16:38 MTDiscord #11692 16:38 ShadowBot https://github.com/minetest/minetest/issues/11692 -- player:get_look_dir() is slightly off sometimes 17:53 pgimeno but view bobbing is client side, right? 17:58 pgimeno does that mean that the client is constantly sending updated view coordinates to the server? 18:01 Krock it does send yaw and pitch 18:02 Krock and the resolution is 1/100 acccording to https://github.com/minetest/minetest/blob/master/src/network/serverpackethandler.cpp#L481-L482 18:02 Krock this should really be a f32 ... 19:16 appguru Trivial partial view bobbing fix: #11695 19:16 ShadowBot https://github.com/minetest/minetest/issues/11695 -- Fix view bobbing not resetting when resting by appgurueu