| Time |
Nick |
Message |
| 00:29 |
|
Lone_Wolf joined #minetest-dev |
| 01:23 |
|
NoctisLabs joined #minetest-dev |
| 01:48 |
|
Miner_48er joined #minetest-dev |
| 01:55 |
|
tuedel joined #minetest-dev |
| 02:00 |
|
tuedel joined #minetest-dev |
| 03:20 |
|
Ruslan1 joined #minetest-dev |
| 03:27 |
|
Wuzzy joined #minetest-dev |
| 03:36 |
|
questionguy joined #minetest-dev |
| 03:48 |
|
Foz joined #minetest-dev |
| 03:50 |
|
questionguy left #minetest-dev |
| 04:07 |
|
NoctisLabs joined #minetest-dev |
| 05:49 |
|
fluxflux joined #minetest-dev |
| 06:56 |
|
ensonic joined #minetest-dev |
| 08:38 |
|
ShadowNinja joined #minetest-dev |
| 08:53 |
|
NobleTechie left #minetest-dev |
| 08:54 |
|
NobleTechie joined #minetest-dev |
| 09:25 |
|
ensonic joined #minetest-dev |
| 09:52 |
|
ssieb joined #minetest-dev |
| 10:34 |
|
proller joined #minetest-dev |
| 10:53 |
|
Fixer joined #minetest-dev |
| 11:18 |
|
proller joined #minetest-dev |
| 11:30 |
|
tomraceror joined #minetest-dev |
| 12:06 |
|
kilbith joined #minetest-dev |
| 12:19 |
|
proller joined #minetest-dev |
| 12:32 |
|
Fixer joined #minetest-dev |
| 12:53 |
|
proller joined #minetest-dev |
| 12:53 |
|
Foz joined #minetest-dev |
| 12:53 |
|
pyrollo joined #minetest-dev |
| 12:53 |
|
p_gimeno joined #minetest-dev |
| 12:53 |
|
Edgy1 joined #minetest-dev |
| 12:53 |
|
Hijiri joined #minetest-dev |
| 12:57 |
|
tomraceror joined #minetest-dev |
| 13:03 |
|
Fixer_ joined #minetest-dev |
| 13:11 |
|
proller joined #minetest-dev |
| 13:32 |
|
Fixer joined #minetest-dev |
| 14:09 |
|
CrazyDave joined #minetest-dev |
| 14:44 |
|
tomraceror joined #minetest-dev |
| 15:42 |
|
Ruslan1 joined #minetest-dev |
| 15:44 |
|
Lunatrius joined #minetest-dev |
| 16:13 |
|
AkRa_ joined #minetest-dev |
| 16:21 |
|
Lone_Wolf joined #minetest-dev |
| 16:36 |
|
Wuzzy joined #minetest-dev |
| 17:25 |
|
Krock joined #minetest-dev |
| 17:34 |
|
ssieb joined #minetest-dev |
| 17:36 |
|
ensonic joined #minetest-dev |
| 17:43 |
|
fluxflux joined #minetest-dev |
| 18:08 |
Krock |
Not a blocking receive IIRC, but the server only processes very few data per step |
| 18:41 |
|
kilbith joined #minetest-dev |
| 19:06 |
Krock |
sfan5: any chance to make use of std::chrono in semaphore.cpp? That time calculation looks awful and attractive for bugs |
| 19:07 |
sfan5 |
not really since you need it in a struct timespec in the end |
| 19:11 |
Krock |
I just hope that time calculation is correct |
| 19:14 |
Krock |
okay. looks correct |
| 19:16 |
sfan5 |
merging #9115 in a few minutes |
| 19:16 |
ShadowBot |
https://github.com/minetest/minetest/issues/9115 -- Optimize semaphore wait with zero timeout on POSIX by sfan5 |
| 19:17 |
Krock |
> Note that this function can potentially wait infinitely |
| 19:18 |
Krock |
Does it? Semaphore in "m_signal.wait" (container.h) should actually prevent that |
| 19:18 |
sfan5 |
it does continue on certain events |
| 19:19 |
Krock |
right. added and removed peers |
| 19:20 |
Krock |
No, that's not a problem for now |
| 19:26 |
Krock |
... the local server runs at some weird non-linear speed (steps). anoying |
| 19:30 |
|
fwhcat joined #minetest-dev |
| 19:38 |
Krock |
sfan5: "immediately available". Are these packets already cached and ready to process? |
| 19:41 |
sfan5 |
yes |
| 19:41 |
sfan5 |
it's the packets that are already inside the m_event_queue |
| 19:41 |
Krock |
ah right. filled by ConnectionReceiveThread::receive() |
| 19:44 |
Krock |
nice work |
| 19:49 |
sfan5 |
now the issue reporter just needs to say that this fixes the issue |
| 19:50 |
|
ensonic joined #minetest-dev |
| 19:51 |
sfan5 |
thanks for reviewing |
| 19:54 |
sfan5 |
re dtime accuracy: what about making dedicated_server_loop increment dtime more often, but having AsyncRunStep() only do something when `m_step_dtime> dedicated_server_step`? |
| 19:56 |
Krock |
np. these were straight forward rather easy fixes |
| 19:58 |
Krock |
well, that would also be an option. I didn't want to rely more than necessary on delta time values. Sleep is not always so accurate after all |
| 20:02 |
Krock |
and currently the function mostly just returns instantly due to "if((dtime < 0.001) && !initial_step)" |
| 20:02 |
Krock |
caused by setting m_step_dtime to zero each step |
| 20:04 |
Krock |
rather than "m_step_dtime -= dtime;" it would mean "m_step_dtime -= fixed_server_step" |
| 20:04 |
sfan5 |
well then dtime would be exactly the fixed step again |
| 20:05 |
Krock |
yes, in any case. regardless how much time actually has passed. until AsyncRunStep is called again, m_step_dtime might already be incremented twice by dedicated_server_step |
| 20:10 |
Krock |
after all it comes down that it's inaccurate by +/- 1 server step. two steps which take 10ms might be run instantly after each other if there's a m_step_dtime increment after 1..9ms |
| 20:10 |
Krock |
given that a previous step took longer than usual |
| 20:44 |
kilbith |
it's good to be sfan5 |
| 20:44 |
kilbith |
your PRs only opened for a few hours |
| 20:45 |
kilbith |
if only it was like that for everything else ... |
| 20:45 |
sfan5 |
that's also due to the size of my PRs |
| 20:45 |
sfan5 |
but yes, there is an obvious advantage if the PR submitter is a coredev |
| 20:45 |
kilbith |
like the CSM PR |
| 20:46 |
kilbith |
caring more of your own PRs yes |
| 20:52 |
rubenwardy |
that was also a fix to a fairly critical bug |
| 20:53 |
rubenwardy |
wait, different PR |
| 20:53 |
sfan5 |
you're not wrong |
| 20:54 |
sfan5 |
well not really "critical", but some CSM things were badly broken too |
| 20:57 |
kilbith |
the other ppl fixes are obvously dubious yeah |
| 20:57 |
kilbith |
/s |
| 21:12 |
|
proller joined #minetest-dev |
| 21:34 |
|
proller joined #minetest-dev |
| 22:11 |
|
proller joined #minetest-dev |
| 22:54 |
|
longerstaff13 joined #minetest-dev |
| 23:11 |
|
Fixer joined #minetest-dev |
| 23:21 |
|
Ruslan1 joined #minetest-dev |
| 23:58 |
|
NoctisLabs joined #minetest-dev |