Growlith1223 / ArsMagica2

Ars Magica 2 Bug Tracker
65 stars 32 forks source link

Lectern Causing Lag #312

Open Jonarlin opened 7 years ago

Jonarlin commented 7 years ago

The lectern causes lots of lag due to lots of chunk updates (sound this using sampler). When a lectern is placed it drops from 60 fps to 40 in the general lectern area.

TheIcyOne commented 7 years ago

Version?

Jonarlin commented 7 years ago

Sorry, I forgot. On 1.5.0-18.

TheIcyOne commented 7 years ago

Technically duplicate of #311, in that case, but it is probably cleaner to keep the issues separated.

Haerar commented 7 years ago

Possible link - FPS also drops when I place the Inscription Table; drop was from 80 fps to 16fps in version 1.10.2 1.5.0-18. Chunk updates went from 0-1 (prior to placing) to an instant 35-36 when placing the table. Did not move during this time; so not caused by world loading.

Both of the above perfectly reproducible on my end; remove table, issue disappears, place table, issue reappears - doesn't matter which chunk it's placed in. With my (admittedly limited) modding knowledge, I suspect that a loop that both of these tileentities run through needs to have a modulo applied; to reduce chunk update rate.

TheIcyOne commented 7 years ago

I thought I'd already fixed that o.O

Growlith1223 commented 7 years ago

i may have left a chunk call in the code in that block. regardless, im unable to fix it for a few more days, until friday it won't be able to be fixed by me. the laptop im on is low-end so i doubt i could run ars magica on here lol. but once it's friday, it should be fixed that day or the next(depending on when the cable company arrives to add internet to the house.)

Haerar commented 7 years ago

Right, bit more information after some experimentation... the issue only arises when placed next to a bibliocraft table; so I'll remove those and we can consider my contributions null and void x.x oddly, bibliocraft's table and the inscription table only seem to dislike each other... placing anything else next to the other doesn't cause a problem. False alarm all! sorry!

Lectern still causing it's issues but I'll experiment with different materials/blocks near it and see if this has any bearing.

@TheIcyOne - can I ask what caused the inscription table issue originally? I am well-versed in java and have dabbled with minecraft modding; made a fun zombie mod once... wonder if I can spot anything perhaps?

EDIT: Any tileentity from AM2 and a bibliocraft table will cause issues next to each other; not sure why but still experimenting. Other transparent blocks from bibliocraft have no issue with AM2 tileentities, bibliocraft tables have no issue with tileentities from other mods (enderio, for example) , and any regular AM2 blocks (witchwood planks, for example) have no impact either way.

TheIcyOne commented 7 years ago

Original issue was just that a few update functions were being called unnecessarily several times a tick.

cool2412 commented 7 years ago

Is there a fix for this out? because my friends and I are running a server with Ars Magica 2 version 1.10.2-1.5.0-18 and My friend gets 108 FPS with the lectern removed but only 18 FPS when it is placed down.

TheIcyOne commented 7 years ago

Downgrade to the previous version, the issue should be new.

scottosx commented 6 years ago

I have verified that both the Lecturn and the Inscription table cause excessive chunk updates when placed. Both in 1.5.17b and 1.5.18. There are between 50 and 100 constant chunk updates. This is in a superflat world (quartz slab surface) with no other structures or blocks. I lose about 10fps.

Looking away from either of these items normalizes chunk updating. Anytime either of these items are in your render distance the problem occurs.

It sounds like a fix is known. Can this be prioritized? It is a significant perfomance issue that needs addressed as these are basic items needed to play the mod.

TheIcyOne commented 6 years ago

I have a feeling the fix has been committed, just waiting on a release.

D0gee commented 6 years ago

Can this be prioritized please? Having lectern placed drops my FPS from 100 to 10.

StalkerM8 commented 6 years ago

So anyone actually fix this? Its still relevant in 1.10.2. On a server running same versions