Bonsai Trees 3

Bonsai Trees 3

51M Downloads

Bonsai Trees Tick time issues

Darkere opened this issue ยท 10 comments

commented

I like using bonsai trees however the tick time on them is very severe.
My basic bonsai oak tree takes 0.6ms of server tick time. Someone else had even more severe issues with up to 5ms per tree.
This seems like a bug to me as other than graphics the only thing it does is produce an item every few seconds.

Bonsai Trees version: b117
Forge version:2760
**Screenshot **: here

commented

This is very hard on servers. I too do not want to remove it from Mystical Village but i may have no choice.

commented

I assume those barrels only accept one type of item?
That's a bug and actually allows abuse of bonsai trees to increase the chance to get a specific drop. While also increasing tick time.

Would you mind retesting this with the latest release from here?

commented

No those are crates from Actually additions they are just oversized chests. The person that made the screenshot also said he was emptying the crates into his me system through those conduits so they were emty. Will check on tick times with the new release right away.

commented

That's weird. What he says and what the screenshot is saying are two different things. Trees on Hopping Pots only stop at 100% if they can't drop their loot into the inventory below. All three trees in the screenshot are at 100%, but are not being auto-harvested. Is he sure the inventory below is empty or rather has room for all items that should get dropped?

commented

Yeah.. well regardless the tick time is high anyway.

I tested with the new version. In the pack (ATM3:Remix) there are no improvements.
https://i.imgur.com/FebpWnZ.png

But in my testpack that only has Actually Additions and Bonsai trees installed.
https://i.imgur.com/bBdwHHx.png

So it's a compatibility issue. .I'm gonna check tinkers next
Although with tinkers it has increased tick time it's nowhere near that of the full pack.
https://i.imgur.com/GZFoOVN.png

commented

It's not just the Hopping Bonsai that has tick issues, even the normal one does.

This is the normal pot while growing
[16:18:13] [main/INFO] [net.minecraft.client.gui.GuiNewChat]: [CHAT] 783 micro-s/t: TILE_ENTITY: {WORLD_ID=0, BLOCK_NAME=TileBonsaiPot, BLOCK_POS_X=-360, BLOCK_POS_Y=44, BLOCK_POS_Z=229, BLOCK_CLASS_NAME=org.dave.bonsaitrees.tile.TileBonsaiPot, NANOS=828074826}

This is the normal pot idle (after it's grown)
[16:23:30] [main/INFO] [net.minecraft.client.gui.GuiNewChat]: [CHAT] 555 micro-s/t: TILE_ENTITY: {WORLD_ID=0, BLOCK_NAME=TileBonsaiPot, BLOCK_POS_X=-360, BLOCK_POS_Y=44, BLOCK_POS_Z=229, BLOCK_CLASS_NAME=org.dave.bonsaitrees.tile.TileBonsaiPot, NANOS=596604433}

commented

Sorry, did not have time to look into this further.
I did not forget, though. Still on my todo list.

commented

Does build 130 or newer from here fix it? With the included fix a lot of the lookups that are happening when growing a tree are now being cached and only done once when the chunk is loaded or a sapling being planted.

commented

Well congrats that seems to have fixed it.

image

Thank you very much.

commented

Excellent! Thanks for the retest. Will release a new version on curse before the weekend is over.
Cheers!