Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[BUG] Massive lags in 1.18.2 #431

Open
Sabbelchen opened this issue Sep 22, 2024 · 2 comments
Open

[BUG] Massive lags in 1.18.2 #431

Sabbelchen opened this issue Sep 22, 2024 · 2 comments
Labels
bug Something is not working as intended. unverified The contents of this issue have not yet been verified.

Comments

@Sabbelchen
Copy link

Minecraft Version

1.18.2

Mod Version

8.1.29

Mod Loader

Forge

What environment are you running the mod in?

Dedicated Server

Issue Description

Hello,

we use a dedicated server for 5 people and we using a lot of Botany Pots. Now the server is extrem lagging because the mod is generating a lot of ticks. Without the Botany Pots everything is fine. I have used Spark with the same world but on a local server because I can't upload this mod on the actual live server at the moment.

Here you ca find the Spark Report

What can I do to stop the lags? There is no real good alternative to your mod so it would be really really sad to delete the mod.

@Sabbelchen Sabbelchen added bug Something is not working as intended. unverified The contents of this issue have not yet been verified. labels Sep 22, 2024
@MUYUTwilighter
Copy link

I think this issue have many similarities with #421

@MUYUTwilighter
Copy link

The server lag is probably due to tickly invoked pot.getInventory().update(); in BlockEntityBotanyPot, where the update method would iterate all the crop & soil recipes by invoking BotanyPotHelper.findSoil and BotanyPotHelper.findCrop in BotanyPotContainer.update.
I think there should be a check on whether the iteration is really necessary.
Here is my solution. https://github.com/MUYUTwilighter/BotanyPotsFix/blob/master/src/main/java/cool/muyucloud/botanypotsfix/mixin/BotanyPotContainerMixin.java#L48

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something is not working as intended. unverified The contents of this issue have not yet been verified.
Projects
None yet
Development

No branches or pull requests

2 participants