KitKait2001 / roamers

0 stars 0 forks source link

[1.18.2] TPS concern - more details below #1

Open Cixon opened 1 year ago

Cixon commented 1 year ago

Firstly, wonderful work on porting this to 1.18.2. I and many others have been waiting and I must admit its amazing in all its glory. I must acknowledge however, that because this is the very first release to a new version that things will not be perfect... so,

Screenshot (401)

Because there's plenty of tasks that the AI must complete, I'm concerned with TPS. I run a pretty fair amount of mods to combat this issue alone so to see that there's a consistent spike that wasn't present before is bothersome.

Mods I use: Canary (main mod that helps TPS) Entity Collision FPS fix by Corgi Taco Radon Rubidium by Asek3

Cixon commented 1 year ago

debug.log

Debug log above on Line 14,015 shows there's a substantial tick issue with the client being 12205ms behind or 12 whole seconds which isn't very pleasant.

DeshiDesu commented 1 year ago

Same issue here. Using canary as well

gardenappl commented 1 year ago

I ran a quick profile using Spark and supposedly 50% of my server tick time was spent on PlayerLikeCharacter.checkDepression() which called PlayerLikeCharacter.checkIfConfined(). I went into the config and disabled depression (and claustrophobia, just in case), and at least for now performance seems much better.

Cixon commented 1 year ago

I ran a quick profile using Spark and supposedly 50% of my server tick time was spent on PlayerLikeCharacter.checkDepression() which called PlayerLikeCharacter.checkIfConfined(). I went into the config and disabled depression (and claustrophobia, just in case), and at least for now performance seems much better.

wonderful tip! thank you

gardenappl commented 1 year ago

wonderful tip! thank you

Take this with a grain of salt beacuse I really didn't do much testing at all. I ended up removing the mod pretty quickly. (it did not seem to integrate too well with TerraFirmaCraft and I was too lazy to configure it otherwise)