Minetest logo

IRC log for #minetest-dev, 2021-06-30

| Channels | #minetest-dev index | Today | | Google Search | Plaintext

All times shown according to UTC.

Time Nick Message
00:03 MTDiscord <exe_virus> Do I sense sarcasm? Cause yeah almost every invention has been invented like 10 times before it becomes publically available or widely known as "invented" and several more before the first official invention. This is extremely common in just about all hardware and software
00:09 behalebabo joined #minetest-dev
01:00 MTDiscord <Warr1024> #11405 is ready, anyone wanna give it a review?
01:00 ShadowBot https://github.com/minetest/minetest/issues/11405 -- Add API for mods to hook liquid transformation events by Warr1024
02:18 v-rob joined #minetest-dev
03:16 hecks joined #minetest-dev
03:59 tekakutli joined #minetest-dev
04:12 v-rob joined #minetest-dev
04:25 v-rob joined #minetest-dev
05:29 YuGiOhJCJ joined #minetest-dev
07:32 sfan5 did you check the performance impact?
07:35 specing_ joined #minetest-dev
07:46 queria joined #minetest-dev
08:15 ssieb joined #minetest-dev
08:24 ivanbu joined #minetest-dev
08:32 ivanbu joined #minetest-dev
08:35 MTDiscord joined #minetest-dev
09:17 Fixer joined #minetest-dev
09:20 entuland joined #minetest-dev
09:36 tekakutli joined #minetest-dev
09:41 tekakutli joined #minetest-dev
10:23 calcul0n_ joined #minetest-dev
11:08 behalebabo joined #minetest-dev
11:47 MTDiscord <Warr1024> I couldn't feel any performance impact on my machine, and I'm on an 8 year old thinkpad so I'd think I should if anyone would.  If you have a more thorough way to test it, I'd like to know.
11:47 MTDiscord <Warr1024> I did do some basic benchmarking of the hook in use and found that each node of liquid transformed cost about 1 microsecond, given NodeCore's logic, which is probably a bit heavy.
11:49 MTDiscord <Warr1024> Basically I wired it into NodeCore's "on_nodeupdate" system, which looks up the position hash in an index for optics, and also does a get_node at that position to look up the node by name in the crafting system index.
11:50 MTDiscord <Warr1024> I was able to get up to about 800 nodes to consume a little over 1ms every 10 seconds.
13:18 calcul0n__ joined #minetest-dev
14:26 appguru joined #minetest-dev
14:40 Fixer joined #minetest-dev
15:12 sfan5 pushing http://sprunge.us/FlFx6G?diff in 5m
15:13 Hiligan joined #minetest-dev
15:38 Extex joined #minetest-dev
17:54 queria joined #minetest-dev
18:24 Krock will merge #11382 #11398 #11387 and #11395 in 15 minutes
18:24 ShadowBot https://github.com/minetest/minetest/issues/11382 -- Inventory: Make addList() consistent by SmallJoker
18:25 ShadowBot https://github.com/minetest/minetest/issues/11398 -- Auth API: Error when accessed prior to ServerEnv init by SmallJoker
18:25 ShadowBot https://github.com/minetest/minetest/issues/11387 -- run on_grant and on_revoke callbacks after privs change by AFCMS
18:25 ShadowBot https://github.com/minetest/minetest/issues/11395 -- Remove unsupported video drivers by hecktest
18:34 v-rob joined #minetest-dev
18:39 Krock merging
18:42 Krock done
19:01 kilbith joined #minetest-dev
19:01 kilbith https://paste.ubuntu.com/p/6H2tsH3k4G/
19:01 kilbith the fuck is this
19:03 sfan5 a bug I guess
19:07 kilbith 062fd21 has not been tested for sure
19:08 kilbith joined #minetest-dev
19:08 Krock excuse me what
19:08 Krock I tested this for sure
19:08 sfan5 with MTG?
19:08 Krock yeah
19:08 sfan5 ¯\_(ツ)_/¯
19:08 Krock rebuilding minetest to ensure
19:09 kilbith so where is the auth function in creative?
19:09 sfan5 check_player_privs
19:10 sfan5 it should never reach there however, at load time it's only called with name=""
19:10 kilbith ah, hmm
19:10 sfan5 ...unless the builtin creative_is_enabled forgets to check for this
19:10 kilbith joined #minetest-dev
19:10 Krock he left
19:10 Krock nvm
19:11 sfan5 hm no builtin has no creative priv
19:11 Krock compiling...
19:12 kilbith sorry, bad connection
19:13 Krock well idk. cannot reproduce in vanilla MTG
19:15 sfan5 shouldn't the error have the full call stack shown?
19:16 Krock yes. it should look like this: https://pastebin.com/raw/pt4vmsaX
19:17 Krock if that error is valid, then is_creative_enabled might be overwritten by another mod which makes use of auth functions before they're ready
19:19 kilbith well it probably comes from i3 which doesn't check if name == ""
19:20 kilbith I now added it and the bug's gone
19:31 AliasAlreadyTake joined #minetest-dev
19:37 specing_ joined #minetest-dev
20:15 MTDiscord <josiah_wi> sfan5 I can redo the commit I accidentally clobbered if you'd like.
20:15 sfan5 it's not about the commit
20:15 sfan5 you just need those changes or it's broken
20:17 MTDiscord <josiah_wi> I understand. I will make a new commit with those changes.
20:18 MTDiscord <josiah_wi> I do see why the changes are necessary. :-)
20:31 AliasAlreadyTake joined #minetest-dev
20:44 sfan5 ubuntu 16.04 does not like the enum inside a map
20:44 sfan5 https://gitlab.com/minetest/minetest/-/jobs/1390128628
20:44 sfan5 not sure we should care
20:45 sfan5 yeah support for it ended in april
21:29 MTDiscord <josiah_wi> I'm happy to report that my PR is passing CI and seems pretty solid.
21:44 ssieb joined #minetest-dev
22:09 freshreplicant[m joined #minetest-dev
22:20 wsor4035 joined #minetest-dev
23:44 Alias2 joined #minetest-dev

| Channels | #minetest-dev index | Today | | Google Search | Plaintext