-
Notifications
You must be signed in to change notification settings - Fork 329
Speedup ae2 stocking hatches #4009
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
base: 1.20.1
Are you sure you want to change the base?
Conversation
|
Wait to merge until we get the OK from ghosti if it actually fixes the lag they cause |
|
why is |
possible nullpointer exception if the previous call returns null, but it shouldn't at that point. Regardless, as per the last commit we're rewriting it to use an ae2 storagemonitor service rather than just querying mainnet all the time |
src/main/java/com/gregtechceu/gtceu/api/misc/StockingHatchList.java
Outdated
Show resolved
Hide resolved
src/main/java/com/gregtechceu/gtceu/api/misc/StockingHatchList.java
Outdated
Show resolved
Hide resolved
src/main/java/com/gregtechceu/gtceu/api/misc/StockingHatchList.java
Outdated
Show resolved
Hide resolved
src/main/java/com/gregtechceu/gtceu/api/misc/StockingHatchList.java
Outdated
Show resolved
Hide resolved
src/main/java/com/gregtechceu/gtceu/api/misc/StockingHatchList.java
Outdated
Show resolved
Hide resolved
src/main/java/com/gregtechceu/gtceu/integration/ae2/machine/MEStockingBusPartMachine.java
Outdated
Show resolved
Hide resolved
src/main/java/com/gregtechceu/gtceu/integration/ae2/machine/MEStockingHatchPartMachine.java
Outdated
Show resolved
Hide resolved
src/main/java/com/gregtechceu/gtceu/integration/ae2/machine/MEStockingHatchPartMachine.java
Outdated
Show resolved
Hide resolved
src/main/java/com/gregtechceu/gtceu/integration/ae2/machine/MEStockingHatchPartMachine.java
Outdated
Show resolved
Hide resolved
src/main/java/com/gregtechceu/gtceu/integration/ae2/machine/MEStockingBusPartMachine.java
Outdated
Show resolved
Hide resolved
|
It seems because of ae2 overhead it's significantly slower 😢 before: after: Notice how our AutoIO() goes from 15% usage to 1.1%, but the AE2 overhead seems to have gone up a ton and therefore the save went from 10 TPS to 3 TPS |


What
Removes a redundant check in stocking hatches
Implementation Details
We no longer simulate extracting for every item in the system, as this is expensive, and instead only simulate the 16 items we pick. This does mean inaccessable items will show up there, but that's user error, and they will show up as 0 amount.
Outcome
no longer 130us/t for a single stocking hatch