Time  Nick         Message
12:20 Krock        next iteration of attachment cleanup is incoming
12:27 rubenwardy   Yay
12:29 ANAND        Awesome!
12:31 Krock        actually I wanted to preserve velocity on detaching, but ended up doing this
12:31 Krock        ....
12:34 sfan5        same happened when I wanted to implement node particles into the API, I ended up cleaning up the internal structures first
12:35 Krock        #9968
12:35 ShadowBot    https://github.com/minetest/minetest/issues/9968 -- Move shared parameters sending to UnitSAO by SmallJoker
12:35 Krock        yes.. always the same story
12:41 Krock        wtf lint
12:41 Krock        NotLikeThis
13:10 PGimeno      these are all steps in the right direction, kudos!
13:14 sfan5        in case lua type checks are discussed later and I'm not here: I think it's okay to merge it now to avoid "breaking" too many mods but it should have a big and explicit warning that 1) this can cause lua errors later on 2) you need to fix it right now because it will become error in the future; then for 5.4.0-dev the checks should immediately be made fatal
13:14 sfan5        oh and the coercion should obviously be fixed
13:15 Krock        okay, thanks for the input
13:15 rubenwardy   maybe error stream it?
13:15 rubenwardy   heh
13:15 sfan5        "big and explicit warning" could include printing sufficient amount of !!!'s and/or printing to to errorstream
13:16 rubenwardy   shame we can't force all modders to read their warning streams
13:16 Krock        rubenwardy: yes.
13:16 Krock        errorstream should be fine.
13:16 Krock        about the coercion... I couldn't find any Lua function that allows checking whether a string can be converted to a number
13:17 Krock        tonumber() returns 0 on failure
13:17 rubenwardy   lua_isnumber
13:17 rubenwardy   " The function lua_isnumber accepts numbers and numerical strings, lua_isstring accepts strings and numbers"
13:17 Krock        uh.. ok
13:18 rubenwardy   lua_tonumber will convert a string to a number too
13:19 Krock        I looked at luaL_checknumber, but didn't know there was lua_isnumber
13:20 Krock        coercion is really weird. not sure why they added it
13:20 rubenwardy   checknumber uses "lua_tonumberx", which only works on numbers and not strings
13:21 Krock        exactly
13:21 rubenwardy   weird choice
14:09 rubenwardy   Anyone free for a meeting today?
14:10 * Krock      <-- this dude
17:07 Krock        If there's no-one else, let's do it next week then...
17:10 rubenwardy   sfan5, nore, sofar?
17:35 Krock        sfan55 already said they will not be here, and sofarr didn't attend in a while
17:57 rubenwardy   :(
17:57 rubenwardy   ok, another week then
20:27 skyliner_369 anyone have a list of planned/upcoming core features for the next release of Minetest?
20:32 Krock        mainly Android rework
20:32 Krock        new changelog goes here: https://dev.minetest.net/Changelog
20:32 Krock        nothing so far
20:35 skyliner_369 Ah... I'm kind of hoping for a proper mob/entity core.
20:35 Krock        see mobs_reo
20:37 skyliner_369 using that now. mobs redo is pretty good, but the AI is a bit silly still.
21:05 PGimeno      does mobs_redo use pathfinding? maybe it suddenly starts to work fine :)
21:09 sfan5        pathfinding was fixed already
21:23 PGimeno      I mean, if mobs redo uses pathfinding, since pathfinding was fixed, maybe mobs_redo begins working fine