Time |
Nick |
Message |
00:51 |
|
ANAND joined #minetest-dev |
00:55 |
|
fluxflux joined #minetest-dev |
01:00 |
|
lisac joined #minetest-dev |
01:12 |
|
nephele_ joined #minetest-dev |
02:59 |
|
Miner_48er joined #minetest-dev |
05:10 |
|
fdgsfg joined #minetest-dev |
05:13 |
|
jonadab joined #minetest-dev |
05:27 |
|
calcul0n joined #minetest-dev |
06:09 |
|
ANAND joined #minetest-dev |
06:39 |
|
erlehmann joined #minetest-dev |
07:37 |
|
NetherEran joined #minetest-dev |
07:52 |
|
Edgy1 joined #minetest-dev |
08:00 |
|
ShadowNinja joined #minetest-dev |
09:28 |
PGimeno |
skyliner_369: that doesn't change the fact that some graphics drivers have trouble with that size |
10:02 |
|
Beton joined #minetest-dev |
10:13 |
|
Beton joined #minetest-dev |
10:18 |
|
NetherEran joined #minetest-dev |
10:20 |
|
fluxflux_ joined #minetest-dev |
10:32 |
|
proller joined #minetest-dev |
10:35 |
|
TheEnemyStando[m left #minetest-dev |
11:12 |
|
Fixer joined #minetest-dev |
11:23 |
|
absurb joined #minetest-dev |
14:06 |
|
ANAND joined #minetest-dev |
14:32 |
|
NetherEran joined #minetest-dev |
14:55 |
|
ANAND joined #minetest-dev |
17:00 |
rubenwardy |
thoughts on #9943 ? |
17:00 |
ShadowBot |
https://github.com/minetest/minetest/issues/9943 -- Deprecate disabling secure.enable_security by rubenwardy |
17:01 |
sfan5 |
its sometimes convenient to disable it |
17:01 |
sfan5 |
but as long as the option doesn't go away I don't care I guess |
17:01 |
sfan5 |
having a big fat warning is certainly a good thing |
17:01 |
rubenwardy |
ok |
17:17 |
Krock |
Should the velocity be kept when the player is detached from an entity? |
17:18 |
Krock |
Like jumping off a cart would then make the player slide a few meters |
17:18 |
longerstaff13 |
how long is a meter in game? |
17:19 |
Krock |
one nominal node |
17:19 |
longerstaff13 |
ah ty |
17:19 |
Krock |
but don't let Wuzzzzy see this |
17:24 |
|
fluxflux joined #minetest-dev |
17:27 |
sfan5 |
review reminder: #9770 #9906 #9900 #9935 |
17:27 |
ShadowBot |
https://github.com/minetest/minetest/issues/9770 -- Add minetest.is_creative_enabled by Wuzzy2 |
17:27 |
ShadowBot |
https://github.com/minetest/minetest/issues/9906 -- Restore visual_scale support for nodeboxes (and allfaces) by numberZero |
17:27 |
ShadowBot |
https://github.com/minetest/minetest/issues/9900 -- Show Toast in UI thread and fix unpacking on Android 10 by ubulem |
17:27 |
ShadowBot |
https://github.com/minetest/minetest/issues/9935 -- [NO SQUASH] content_cao: Don't expire visuals when unnecessary + nametag cleanups by sfan5 |
17:30 |
Krock |
testing 9935 |
17:51 |
Krock |
dunno about 9900 |
17:51 |
Krock |
testing 9906 |
18:00 |
Krock |
9770: `get_player_creative` would technically fit more into the API naming scheme |
18:00 |
|
NetherEran joined #minetest-dev |
18:00 |
rubenwardy |
oh no |
18:00 |
Krock |
is_creative_enabled sounds like a server-wide function |
18:01 |
Krock |
hmm.. "is_area_protected" well, I see.. it's probably fine |
18:11 |
Krock |
I think #9954 should be discussed tomorrow in a meeting |
18:11 |
ShadowBot |
https://github.com/minetest/minetest/issues/9954 -- Lua API: Decrease incorrect parameter type error to warning by SmallJoker |
18:11 |
Krock |
the opinions seem to diverge a little bit |
18:13 |
Krock |
added to meeting notes. more PR and issue discussion topics are welcome |
19:08 |
PGimeno |
Krock: physically it would be more realistic if the velocity is kept, I think that's a good thing |
19:10 |
Krock |
PGimeno: oh right. already forgot about this idea again. Now that player movement speed can be added nicely without anticheat problems it would be quite easy to add |
19:10 |
Krock |
to add |
19:11 |
sfan5 |
hm now that 9935 is reviewed I discovered there might be bug in it |
19:15 |
Krock |
what's it? |
19:16 |
sfan5 |
updateNametag passes the scene node to addNametag(), but when the visual expires a new scene node is created but the nametag isn't recreated to change it |
19:16 |
sfan5 |
I can't get this to trigger any buggy behavior |
19:16 |
sfan5 |
though |
19:18 |
sfan5 |
oh, removeFromScene also deletes the nametag |
19:18 |
sfan5 |
then it's fine |
19:19 |
Krock |
so far no bug |
19:37 |
|
pmp-p joined #minetest-dev |
19:54 |
|
NetherEran joined #minetest-dev |
20:02 |
|
appguru joined #minetest-dev |
20:03 |
appguru |
sfan5: I'd like to improve the server list design (https://servers.minetest.net) |
20:03 |
appguru |
Would you be fine with dragging in Bootstrap or a similar framework as dependency? |
20:10 |
|
NetherEran joined #minetest-dev |
20:14 |
sfan5 |
that's not too big so it should be okay |
20:15 |
sfan5 |
the issue is that the css needs to work on foreign pages too (e.g. https://www.minetest.net/servers/) |
20:16 |
Krock |
might need to default the node.param2 value to 0 in set_node so that older maps don't break |
20:16 |
Krock |
https://forum.minetest.net/viewtopic.php?f=6&t=24830 |
20:18 |
sfan5 |
that code was added in 5.2.0 yeah |
20:27 |
sfan5 |
rubenwardy: are your concerns to #9572 addressed? |
20:27 |
ShadowBot |
https://github.com/minetest/minetest/issues/9572 -- Some vector functions useful for working with rotations by NetherEran |
20:28 |
sfan5 |
merging #9770, #8668, #9935, game#2689 in 15 minutes |
20:28 |
ShadowBot |
https://github.com/minetest/minetest/issues/9770 -- Add minetest.is_creative_enabled by Wuzzy2 |
20:28 |
ShadowBot |
https://github.com/minetest/minetest/issues/8668 -- disable_jump check for the player's feet by MoNTE48 |
20:28 |
ShadowBot |
https://github.com/minetest/minetest/issues/9935 -- [NO SQUASH] content_cao: Don't expire visuals when unnecessary + nametag cleanups by sfan5 |
20:28 |
rubenwardy |
yeah |
20:28 |
ShadowBot |
https://github.com/minetest/minetest_game/issues/2689 -- carts: Fix item collection by sfan5 |
20:38 |
PGimeno |
vector.rotate_around_axis(v, axis, angle) <-- I have yet to see this function used for something other than axis-aligned rotation (for which it's overkill) |
20:42 |
sfan5 |
maybe we should have a specialization of it then? |
20:47 |
sfan5 |
god fuck this garbage software |
20:47 |
sfan5 |
github created a merge commit again despite this being explicitly disallowed by repo settings |
20:49 |
sfan5 |
now the patch won't apply anymore ???????????ß |
20:50 |
Krock |
force push |
20:52 |
sfan5 |
I just tried to merge montes minetest into my local minetest_game repo |
20:52 |
sfan5 |
needless to say that wasn't successful |
20:55 |
sfan5 |
ok done merging now |
21:51 |
PGimeno |
vector.rotate_around_X/Y/Z(v, angle) is OK |
23:01 |
|
Miner_48er joined #minetest-dev |
23:17 |
|
paramat joined #minetest-dev |
23:17 |
paramat |
will merge trivial #9955 in 15 mins |
23:17 |
ShadowBot |
https://github.com/minetest/minetest/issues/9955 -- Devtest: Fix missing liquid properties causing crash by Wuzzy2 |
23:31 |
PGimeno |
#9572 - "-- Compute matrix multiplication: `matrix` * `v`" That's not what the code that follows is doing. The matrix should be properly rearranged. |
23:31 |
ShadowBot |
https://github.com/minetest/minetest/issues/9572 -- Some vector functions useful for working with rotations by NetherEran |
23:45 |
paramat |
ok will add a comment with that |
23:46 |
paramat |
merging 9955 |
23:47 |
|
nephele left #minetest-dev |
23:48 |
PGimeno |
vector.directions_to_rotation calls very expensive functions. It's probably simpler to construct a matrix (two normalizations, one cross product), and then to convert that matrix to Euler. |
23:48 |
PGimeno |
sorry, two cross products |
23:49 |
paramat |
will add your extra comments later |
23:50 |
PGimeno |
thanks, that was my last comment, I've finished reviewing |