Time Nick Message
03:47 MTDiscord shouldnt we be allowed to compare vectors to other numbers using the "<, <=, >=, >" operators? shouldn't it just automatically do vector.length?
03:47 MTDiscord I mean, we already have other mathematical operators for them, I fell like it'd make sense.
08:14 MTDiscord No. I've been down that route and it's a poor idea. The only reasonable comparison to be implemented on vectors is (in)equality.
08:17 MTDiscord What should a comparison v < w yield? You say it should compare the lengths as in len(v) < len(w). Other perfectly valid definitions include lexicographic ordering or "all components of v must be smaller than the respective components of w".
08:20 MTDiscord And all of them have their applications. Lexicographic ordering is useful if you want to have a sorted data structure (sorted set / sort a list - although TBF Lua's table.sort doesn't seem to be respecting metatables). Lexicographic is the only ordering that won't yield an "invalid comparison function" error here. Comparing all components is useful for cuboid checks - to check whether p is within a cuboid defined by min and max, you
08:20 MTDiscord simply do p >= min and p <= max. And what you have proposed is useful for comparing lengths.
08:21 MTDiscord All of them break basic assumptions made about comparison functions though.
08:23 MTDiscord For lengths: not (v < w or v > w) should be the same as v == w, right? If a "number" (vector in this case) is neither smaller nor larger than another vector, the two vectors must be equal. Wrong if lengths are compared: There are a total of eight vectors with the same length (all ways to flip the signs).
08:24 MTDiscord Or similarly, if v > 0 then -v < 0, right? Again not the case with lengths: Both v and -v will be > 0 (where 0 is the null vector)
08:26 MTDiscord Similar arguments can be made for the other funcs
08:26 MTDiscord Mathematicians thus say that vector spaces are not arrangable
08:26 MTDiscord There is no comparison function that behaves the same way we expect a comparison function on numbers to behave
08:53 sfan5 merging #12781, #12751, #12353 soon (let's see if we'll have to revert the last one one day)
08:53 ShadowBot https://github.com/minetest/minetest/issues/12781 -- Fix a trivial TODO in localplayer.h by Desour
08:53 ShadowBot https://github.com/minetest/minetest/issues/12751 -- Replace XOR with complement for bitflip by ndren
08:53 ShadowBot https://github.com/minetest/minetest/issues/12353 -- Fix formula used for acceleration by appgurueu
09:56 Zughy[m] 75 PRs, a brand new record. ?
10:17 MTDiscord Props to the core devs and the triager!
13:00 sfan5 rubenwardy: when you build the android build remember this https://github.com/minetest/minetest/issues/12788
13:04 jwmhjwmh Any chance you could consider approving #12611 in the next meeting? VoxelManip bulk methods are about twice as fast when using the FFI.
13:04 ShadowBot https://github.com/minetest/minetest/issues/12611 -- Implement some VoxelManip functionality safely using LuaJIT's FFI library by TurkeyMcMac
14:10 kilbith typo here: https://github.com/minetest/minetest/blob/master/doc/lua_api.txt#L4814
14:10 kilbith /deleteobjects -> /clearobjects
22:20 Zughy[m] what should we do with #11342? Op looks like gone, it's still a draft but theoretically is ready
22:20 ShadowBot https://github.com/minetest/minetest/issues/11342 -- Serverlist enhancements by ShadowNinja
22:27 MTDiscord @ShadowNinja
22:27 kilbith a demo of the camera API that can grab a scene node from a mouse click: https://www.youtube.com/watch?v=VED5EEcYxTo
22:37 Zughy[m] that's awesome
22:42 sfan5 Zughy[m]: wait and leave it open
22:42 sfan5 I resurrected a PR from 2019, imo much needed #12790
22:42 ShadowBot https://github.com/minetest/minetest/issues/12790 -- Take geographic distance into account for server list ordering by sfan5
22:45 Zughy[m] and also a filter
22:45 Zughy[m] in the future it'd be nice to have icons instead of text, but nice
22:45 sfan5 the text there is just debug
22:45 sfan5 but I supposed it could be represented with icons
23:13 x2048 PR for bloom has landed: #12791
23:13 ShadowBot https://github.com/minetest/minetest/issues/12791 -- Bloom by x2048