Skip to content

allow draining tank tower from an empty tank above - #359

Merged
boubou19 merged 6 commits into
GTNewHorizons:masterfrom
danyadev:certus-quartz-tank-drain-from-above
Dec 16, 2025
Merged

allow draining tank tower from an empty tank above#359
boubou19 merged 6 commits into
GTNewHorizons:masterfrom
danyadev:certus-quartz-tank-drain-from-above

Conversation

@danyadev

Copy link
Copy Markdown
Member

Now you can drain an empty tank above if there is fluid in one of the tanks below. Also Waila now shows full tower amount and capacity when hovering on empty or filled tanks. Also it now shows the capacity even if there's no fluid

drain.upper.tanks.mp4
image

@Dream-Master Dream-Master added the 🚧 Testing on Zeta Do not merge yet, testing this PR on Zeta (experimental build) label Nov 14, 2025
@DarkShadow44

Copy link
Copy Markdown

Has there been discussion on whether we want this? I've always taking it for granted that you can't extract fluid that is under the level your pipe is connected.

@danyadev

Copy link
Copy Markdown
Member Author

All other tanks (that support towering) work this way: bc tanks, iron tanks, openblocks tanks

it just looks like it's mostly working: you can fill tanks from the top, you could see fluid in waila even when the tank is empty but one of the lower tanks isn't (I removed that in a previous pr during a refactor), the only thing you can't do is to drain from the top

@boubou19
boubou19 enabled auto-merge (squash) December 16, 2025 19:06
@boubou19
boubou19 merged commit 95dad4d into GTNewHorizons:master Dec 16, 2025
1 check passed
@danyadev
danyadev deleted the certus-quartz-tank-drain-from-above branch December 16, 2025 20:36
@Dream-Master Dream-Master removed the 🚧 Testing on Zeta Do not merge yet, testing this PR on Zeta (experimental build) label Dec 16, 2025
@PLASMAchicken

Copy link
Copy Markdown
Member

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

Successfully merging this pull request may close these issues.

6 participants