SpongePowered / SpongeForge

A Forge mod that implements SpongeAPI
http://www.spongepowered.org/
MIT License
1.14k stars 306 forks source link

Console error - Error while creating CraftingInventoryLensImpl for brad16840.common.ContainerStack #2031

Closed ProsperCraft closed 6 years ago

ProsperCraft commented 6 years ago

I am currently running

Issue Description

The stacktrace attached shows up in console. I don't know what the players were doing ingame, or if it has an effect on them.

https://gist.github.com/ProsperCraft/fd9de4e6d99a87595888e9cd1a55b3d7

Faithcaio commented 6 years ago

This error wont affect gameplay. But Sponge will not be able to provide the CraftingInventory for that Container.

ProsperCraft commented 6 years ago

I found how to reproduce it, you install this mod- https://minecraft.curseforge.com/projects/minetweaker-recipemaker

Then type /mtrm

Then you get this - https://gist.github.com/ProsperCraft/b8855967ca5e60a55c44cf8e15dab64c

Faithcaio commented 6 years ago

Did not think that a crafting inventory without output makes sense. But here it is...

PoroUsedSnax commented 6 years ago

I have been noticing this error too (for brad16840). I'm fairly certain it is related to the Backpacks! mod. And more specifically a crafting window that is opened by right clicking a crafting table in the players inventory, opening a custom GUI.

I have reported it to the developer, but thought I would mention it here as well as I found this issue.

Here is more information for it, same as given to brad: Error: https://gist.github.com/PoroUsedSnax/cf2f874d67f94d58f3ccb3bf75e56378

Versions: Minecraft: 1.12.2 SpongeAPI: 7.1.0-SNAPSHOT-06016f2 SpongeForge: 1.12.2-2611-7.1.0-BETA-2990 Minecraft Forge: 14.23.2.2611 Backpacks!: 3.4.4

Whilst this doesn't seem to affect gameplay as explained, I am again fairly certain that is does actually cause lag issues for the server (Eventually getting to constants of 4-7 TPS) After asking players to try and avoid using it, the TPS has been keeping at it's usual 20, as it should be. When this lag kicks in, I start to notice (Using LagGoggles) that sponge's CPU usage (I think that's what the percentage is) sky rockets, to the point where it could be showing between 300-600% and possible even more, as shown in this image (Bottom right pane): 2018-03-29_18 29 51

gabizou commented 6 years ago

The problem with marking the Sponge listener as "hot" code is that it's not doing anything special except processing the tick scheduler for plugins (so naturally sponge gets the massive hit for having to run all the scheduled tasks in that listener. If possible to associate warm roast or another VM hotspot timings reader would help as the image above doesn't provide any useful information to find the source of lag in this context.

But other than that, that's room for another discussion/issue as the topic at hand is for the error with a crafting inventory lens.

PoroUsedSnax commented 6 years ago

My error is the same CraftingInventoryLens one that this issue was originally made for, the Error while creating CraftingInventoryLensImpl for brad16840.common.ContainerStack one, it's just that our lag issues seem to be related to this error, but I can start a new issue if you'd like, not sure what else it would be called though. I can sort out a warmroast for you to take a look at.

EDIT: brad has now fixed this with his coming latest update, so from my point of view this is free to be closed.