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

Recipes with more than 9 base components crashes the server in Smart Mode #50

Open
ZetDude opened this issue Aug 28, 2020 · 2 comments
Open

Comments

@ZetDude
Copy link

ZetDude commented Aug 28, 2020

After some encounters i've narrowed down the bug in a test environment. When making a craft, logistics bridge automatically breaks down all recipes to their base components. I created a relatively simple albeit maybe over-the-top test setup that crafts a Signalum Upgrade Kit from Thermal Foundation that ends up with more than 9 things in list of ingredients
. At some point in AE2's process it tries to put these ingredients in a vanilla crafting grid in some way, which leads to an
ArrayIndexOutOfBoundsException.

I've found that this can be fixed by setting the bridge to "Simple Mode", however looking at some of the other issues it seems that switching to simple mode is a fix to some other problems. What is it that makes "Smart Mode" "smart" then? And why is it the default option in its current state?

@ZetDude
Copy link
Author

ZetDude commented Aug 29, 2020

I've now realized that Simple Mode results in a lot of "Can't keep up" messages that cause apparent server lag despite the ticks per second being 20, i suspect it has something to do with #49 in some way

@tom5454
Copy link
Owner

tom5454 commented Aug 30, 2020

Please right click the AE Bridge with a stick to create a profiler report, and please post it here.

tom5454 added a commit that referenced this issue Aug 30, 2020
Fixes #51
Fixes crash #50
#49 #35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants