Minetest logo

IRC log for #minetest-dev, 2023-02-27

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

All times shown according to UTC.

Time Nick Message
00:13 Zughy[m] meeting notice posted (next Sunday)
00:18 Desour joined #minetest-dev
00:40 Yad joined #minetest-dev
00:43 Yad_ joined #minetest-dev
01:02 Yad__ joined #minetest-dev
01:57 lhofhansl joined #minetest-dev
01:57 lhofhansl sfan5: If you're still around... Have a quick look at #13256... It's a quick addendum for #13255 for large viewing_ranges.
01:57 pgimeno https://github.com/minetest/minetest/pull/13256 -- Check if client has a block even if the server has unloaded it. by lhofhansl
01:57 pgimeno https://github.com/minetest/minetest/pull/13255 -- Don't expire blocks visible to the client. by lhofhansl
05:00 MTDiscord joined #minetest-dev
05:10 Yad_ joined #minetest-dev
05:42 calcul0n_ joined #minetest-dev
05:59 Yad__ joined #minetest-dev
06:28 lhofhansl joined #minetest-dev
08:13 Yad joined #minetest-dev
08:38 sugarbeet joined #minetest-dev
09:46 Warr1024 joined #minetest-dev
12:43 YuGiOhJCJ joined #minetest-dev
14:01 proller joined #minetest-dev
14:02 sugarbeet joined #minetest-dev
14:28 LandarVargan joined #minetest-dev
15:15 hmmmm joined #minetest-dev
15:18 appguru joined #minetest-dev
15:53 proller joined #minetest-dev
15:58 kilbith joined #minetest-dev
16:48 sfan5 can we get some approvals on #13245, #13243
16:48 pgimeno https://github.com/minetest/minetest/pull/13245 -- [no squash] Fix task ordering and more in Gradle Android build by sfan5
16:48 pgimeno https://github.com/minetest/minetest/pull/13243 -- Drop dependency on IrrCompileConfig by numberZero
16:59 lhofhansl joined #minetest-dev
17:00 lhofhansl We need to either revert #13255 or merge #13256.
17:00 pgimeno https://github.com/minetest/minetest/pull/13255 -- Don't expire blocks visible to the client. by lhofhansl
17:00 pgimeno https://github.com/minetest/minetest/pull/13256 -- Check if client has a block even if the server has unloaded it. by lhofhansl
17:17 kilbith that's exactly why we need to wait and think a bit more prior merging something
17:18 kilbith I know merging your PRs can cause you to secrete some dopamine but that's not always a smart move
17:18 kilbith (Lars, in all friendship ;)
17:20 lhofhansl Not helpful. We do that all the time. git makes revert easy. Or we fix forward.
17:20 lhofhansl This is another four line change for a corner case.
17:24 sfan5 more like a fix for a fix
17:40 lhofhansl I file a brainstorming issue for the serverside for large viewing_ranges. There are a bunch of more things we can do.
17:57 lhofhansl Ok. Merged.
18:10 lhofhansl Also #13259 I just filed. It add a log line that measure the time it took the server to update/send the complete map to the client. For large viewing_ranges this can be minutes... And hence the default server_unload_unused_data_timeout (29s) gets in the way of that.
18:10 pgimeno https://github.com/minetest/minetest/pull/13259 -- Measure server map send time. by lhofhansl
18:13 lhofhansl (and don't mind my spelling :) )
18:27 appguru joined #minetest-dev
18:55 sfan5 is there some kind of trend putting a full stop at the end of commit titles
18:58 MTDiscord <PrairieWind> wonders why he followed the rules as best he could in his first PR
19:27 lhofhansl sfan5: Referring to my commit(s)? Should I not do that? Fine either way.
19:27 sfan5 yes yours
19:28 sfan5 I don't think we have a rule for/against it but without would be more consistent with the rest
19:28 lhofhansl Noted.
19:30 lhofhansl Check out #13260. Put some numbers on it.
19:30 pgimeno https://github.com/minetest/minetest/pull/13260 -- Reduce server CPU consumed by occlusion culling. by lhofhansl
19:30 lhofhansl (And I'll remove the trailing . if/when this is merged)
21:38 appguru joined #minetest-dev
22:03 proller joined #minetest-dev
22:27 appguru joined #minetest-dev
22:47 rubenwardy merging #12367 in 10
22:47 pgimeno https://github.com/minetest/minetest/pull/12367 -- Add minetest.get_player_window_information() by rubenwardy
22:50 proller joined #minetest-dev
22:51 MTDiscord <Ronoaldo> That is awesome!
23:08 sfan5 reminder: can we get some approvals on #13245, #13243
23:08 pgimeno https://github.com/minetest/minetest/pull/13245 -- [no squash] Fix task ordering and more in Gradle Android build by sfan5
23:08 pgimeno https://github.com/minetest/minetest/pull/13243 -- Drop dependency on IrrCompileConfig by numberZero

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