TeamJM / journeymap

JourneyMap mod for Minecraft
http://journeymap.info
129 stars 10 forks source link

[1.21] [Bug]: Severe lag for some #752

Closed NielsPilgaard closed 2 months ago

NielsPilgaard commented 3 months ago

What happened?

One of our players was experiencing severe lag, ran a spark profile, and it pointed towards Journeymap being the culprit. Removing Journeymap alleviates the lag.

Original issue: https://github.com/EnigmaticaModpacks/Enigmatica10/issues/20

Mod Loader

NeoForge

Minecraft Version

1.21.x

Version

6.0.0

Relevant log output

Spark profile: https://spark.lucko.me/4kEqk7gmBH

Latest.log: https://gist.github.com/NielsPilgaard/171c52dd933f07e6ef91cb1f0ad90560

Journeymap log: https://gist.github.com/NielsPilgaard/11c3f9226a11ba07af0c30d814bb75ac

Log appears to come from this code: https://github.com/oshi/oshi/blob/master/oshi-core/src/main/java/oshi/util/platform/windows/WmiQueryHandler.java#L140

mysticdrew commented 3 months ago

Looks like there is a lot of lag by fpsreducer lol image

mysticdrew commented 3 months ago

Log appears to come from this code: https://github.com/oshi/oshi/blob/master/oshi-core/src/main/java/oshi/util/platform/windows/WmiQueryHandler.java#L140

I don't understand what this is in regards to?

Also you should be on beta.14 as we did make some fairly significant performance improvements.

NielsPilgaard commented 3 months ago

Log appears to come from this code: https://github.com/oshi/oshi/blob/master/oshi-core/src/main/java/oshi/util/platform/windows/WmiQueryHandler.java#L140

I don't understand what this is in regards to?

Also you should be on beta.14 as we did make some fairly significant performance improvements.

I was guessing you used that library, thought a link to the source would save you some time.

Right, we have updated to beta.14. I'll ask the player to do the same.

mysticdrew commented 3 months ago

Nope, no idea what that's from, either mc or nf

mysticdrew commented 3 months ago

Looks like the lag is from log spam trying to register textures. Likely a mod interaction or somthing. But fpsreducer is taking the most rendering time, should try removing that.

NielsPilgaard commented 3 months ago

FPSReducer just decreases frames when the window is unfocused or paused, it shouldn't do anything during active play.

mysticdrew commented 3 months ago

FPSReducer just decreases frames when the window is unfocused or paused, it shouldn't do anything during active play.

aah okay, it just has the most rendering time in the spark profile.

NielsPilgaard commented 3 months ago

That is a bit concerning, I'll keep my eye on it, thanks 😊

I've asked the player to update, we can hope that'll fix the texture registration issue.

mysticdrew commented 3 months ago

if not, let me know we can see if we can find a solution.

NielsPilgaard commented 3 months ago

Same issue I'm afraid, here are a bunch of new logs: https://github.com/EnigmaticaModpacks/Enigmatica10/issues/20#issuecomment-2211847839

mysticdrew commented 2 months ago

1.21-6.0.0-beta.15 Is publishing with the fix. However, we have to write some custom logic for the creeper reloaded mod to generate those icons, they do some funny stuff. This will come at a future date.

NielsPilgaard commented 2 months ago

Sounds good, thank you!