Skip to content

Conversation

@jurrejelle
Copy link
Contributor

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

@jurrejelle jurrejelle requested a review from a team as a code owner October 2, 2025 08:00
@jurrejelle jurrejelle added type: bugfix General bug fixes 1.20.1 Release: Patch - 0.0.X Smaller changes that either are bug fixes or very minor tweaks. labels Oct 2, 2025
@github-actions github-actions bot added the Tests: Passed Game Tests have passed on this PR label Oct 2, 2025
@jurrejelle
Copy link
Contributor Author

Wait to merge until we get the OK from ghosti if it actually fixes the lag they cause

@jurrejelle
Copy link
Contributor Author

  • Update, this isn't the slow part, the slow part is
image image

Back to the drawing board, idek if we can fix this in any reasonable way. Will need to discuss

@jurrejelle jurrejelle marked this pull request as draft October 2, 2025 10:16
@a-a-GiTHuB-a-a
Copy link
Contributor

a-a-GiTHuB-a-a commented Oct 3, 2025

why is getStorageService underlined? like, what's the issue it's complaining about?

@jurrejelle
Copy link
Contributor Author

why is getStorageService underlined? like, what's the issue it's complaining about?

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

@github-actions github-actions bot added Tests: Failed Game Tests have failed on this PR Tests: Passed Game Tests have passed on this PR and removed Tests: Passed Game Tests have passed on this PR Tests: Failed Game Tests have failed on this PR labels Oct 4, 2025
@jurrejelle jurrejelle marked this pull request as ready for review October 4, 2025 20:02

private void syncListToHandler() {
int index = 0;
for (var entry : topItems) {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See prior comment - also can this code be deduplicated?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not in a clean way I could find, because of the usage of the slots, the fact that both lists dont have a shared superclass, etc

@DilithiumThoride DilithiumThoride added the Do Not Merge DO NOT MERGE THIS PR YET! label Nov 8, 2025
@jurrejelle jurrejelle removed the Do Not Merge DO NOT MERGE THIS PR YET! label Jan 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

1.20.1 Release: Patch - 0.0.X Smaller changes that either are bug fixes or very minor tweaks. Tests: Passed Game Tests have passed on this PR type: bugfix General bug fixes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants