Skip to content
This repository has been archived by the owner on Mar 8, 2024. It is now read-only.

Fix MDT void protection #367

Merged
merged 3 commits into from
Nov 13, 2023

Conversation

MarchingCube
Copy link

@MarchingCube MarchingCube commented Nov 12, 2023

I've noticed that two MDTs (Mega Distillation Tower) had a fairly high tick time in my world. After a quick inspection it seemed to one tick despite batch mode being enabled. After a quick check it turned out that disabling void protection helped and multi didn't actually void anything (as it was connected to ME output hatch).

I had a similar setup for MVF (Mega Vacuum Freezer) that worked correctly so MDT seemed to be the problem.

What is happening is that MDT has special logic to output per layer, original code was not detecting any hatches so it would throttle production.

I've changed this logic to detect ME Output Hatch per layer, and if all layers have a hatch then assume this multi does not need to go through full void protection pass.

@bombcar bombcar requested a review from a team November 12, 2023 02:29
@miozune
Copy link
Member

miozune commented Nov 12, 2023

While this works as a temporary solution, I think giving void protection support for ME buses/hatches and throwing away canDumpItemToME / canDumpFluidToME would be the right way. It'll also solve GTNewHorizons/GT-New-Horizons-Modpack#14556.

@MarchingCube
Copy link
Author

While this works as a temporary solution, I think giving void protection support for ME buses/hatches and throwing away canDumpItemToME / canDumpFluidToME would be the right way. It'll also solve GTNewHorizons/GT-New-Horizons-Modpack#14556.

I agree that implementing a proper solution would be a better option, since another issue on my list is to check why a certain MCR is refusing to run due to missing output space (while also being connected to ME output bus/hatch combo).

For this issue I wanted to implement something that can be hotfixed for 2.4.X version since it was affecting my world. And current implementation is simply wrong for MDT. Do you think that we can proceed with this fix or should a broader fix be rolled out first? @miozune

Copy link
Member

@miozune miozune left a comment

Choose a reason for hiding this comment

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

Yeah, it's good enough solution for now I guess

@miozune miozune merged commit ce23379 into GTNewHorizons:master Nov 13, 2023
1 check passed
@MarchingCube MarchingCube deleted the fix-mdt-void-protection branch November 13, 2023 12:59
mitchej123 pushed a commit that referenced this pull request Mar 8, 2024
* Update buildscript.

* Implement custom MDT logic for dumping fluid to ME.

* Code cleanup.

Former-commit-id: 7871fca4a3371822cec3dc1d9adf190853bed005
mitchej123 pushed a commit that referenced this pull request Mar 8, 2024
* Update buildscript.

* Implement custom MDT logic for dumping fluid to ME.

* Code cleanup.

Former-commit-id: 37d1e00c283ef1ae8b03150212c3ca425b0ef3b5
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants