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

Fixed Mufflers and Large Boiler Fuels. #1319

Merged
merged 3 commits into from
Feb 10, 2018

Conversation

draknyte1
Copy link
Collaborator

@draknyte1 draknyte1 commented Dec 7, 2017

Mufflers can no longer face downwards, into a machine.
This abuse allowed machines to check inside themselves for air.

Large Boilers cannot take Lava anymore, as originally intended.

"The Large Boiler can burn almost all forms of solid and liquid fuel, except lava." - https://ftb.gamepedia.com/Large_Boiler
@draknyte1 draknyte1 changed the title Fixed Muffler abuse. Fixed Mufflers and Large Boiler Fuels. Dec 7, 2017
@MauveCloud
Copy link
Collaborator

Mufflers can no longer face downwards, into a machine.
This abuse allowed machines to check inside themselves for air.

Hang on. That would only be checking inside the machine for machines that require the muffler to be at the top above a hollow section. A large boiler requires the muffler hatch in the bottom layer, at which point having it face down toward an air block would not be checking inside the machine, and should be allowed. A large gas/steam turbine or large plasma generator allows the muffler hatch to be on the top, side, or bottom. If placed at the bottom, facing downward would be okay, but facing upward should be forbidden, and if placed at the side, determining which way is toward the inside of the machine would be a little trickier.

@draknyte1
Copy link
Collaborator Author

No machine REQUIRES it facing downwards. Which is far more exploitable than any other direction.

@leagris
Copy link
Contributor

leagris commented Dec 7, 2017

@draknyte1 Multimachines are cuboid.
Should be easy enough to check the air block is not within the cuboid structure boundaries instead of hardcoding a directional check.

@MauveCloud
Copy link
Collaborator

That's not universally true, but the multiblock machines that aren't cuboids don't use muffler hatches, so that seems like it will work. However, draknyte1's proposed solution seems to change whether a muffler hatch is even place-able/wrench-able to face downward, whereas checking the cuboid would have to be added to the multiblock controller (since the muffler hatch by itself doesn't know where the structure boundaries are).

@leagris
Copy link
Contributor

leagris commented Dec 7, 2017

Then having the Controller perform muffler hatch orientation check during check machine and report incomplete structure if muffler is in wrong orientation or has output obstructed could be a valid alternative.
Afterall, the air block beside a muffler output is part of the multiblock structure.

@draknyte1
Copy link
Collaborator Author

Yeah, or, just leave the simple fix as it is.

Maint. Hatches can only face horizontal, it's never caused an issue. This won't either because there is zero legitimate use for facing them down.

@SapientMC
Copy link

SapientMC commented Dec 8, 2017

@draknyte1
I disagree with

zero legitimate use for facing them down.

2017-12-08_11 14 05
2017-12-08_11 14 31

Middle turbine consume two different fuel type with its own optimal flow, so both top center blocks are occupied by input hatches.

@draknyte1
Copy link
Collaborator Author

I'd rather change where hatches go on turbines than change the current status of the PR.
95% of cases where they face down are people exploiting them to face internal.

I'll fix that tomorrow or something.

@SapientMC
Copy link

The turbine has only 2 blocks of air inside. It's logical to check if the muffler is directed to one of them and that's it.

@OvermindDL1
Copy link

OvermindDL1 commented Dec 12, 2017

95% of cases where they face down are people exploiting them to face internal.

Not really, I placed them facing down underneath my machines in the majority of cases.

(I didn't actually think of facing them internally, I figured that'd already be checked for...)

@Blood-Asp
Copy link
Owner

I somehow mean to remember that back in GT4 muffler hatches only could be placed on the top side of turbines but not sure. And as i tought so i never checked if they can work on other sides. But facing down for an exhaust actually feels wrong to me...

Also about the comments to check if they face inside: The way they are currently implemented that would not be so easy. Actually can't thing about an easy way right now at all.

@draknyte1
Copy link
Collaborator Author

draknyte1 commented Dec 22, 2017

There is no easy way, simply put.
Mufflers facing down is invalid and I agree with your statement on the fact it feels wrong.
But hey, maybe Pollution is denser than air and falls downwards?

I left this as In Progress because I wanted to think of an alternative solution over the next few days.

@SapientMC
Copy link

I want to remind you that the exhaust pipe of all cars is located below.

@draknyte1
Copy link
Collaborator Author

And they face horizontally in all 16 countries I've visited.
Your point is invalid.

@SapientMC
Copy link

But none of them occupies a square meter of surface.

@leagris
Copy link
Contributor

leagris commented Dec 22, 2017

I am just sarcastic but:
Gregtech is known to place multiblock devices the most annoyingly possible.
So if it is convenient to have a muffler hatch at bottom of turbine, be sure it will be denied because Gregtech is not convenient.
Why do you think turbine Controller is at the most annoying place possible? 😆

@OvermindDL1
Copy link

But facing down for an exhaust actually feels wrong to me...

@Blood-Asp Well at work the emergency generator exhaust faces down to prevent rain from getting in to it.

And yes, pollution is denser than air.

@Blood-Asp Blood-Asp merged commit 92bcce8 into Blood-Asp:unstable Feb 10, 2018
Dream-Master referenced this pull request in GTNewHorizons/GT5-Unofficial Aug 19, 2018
Fixed Mufflers and Large Boiler Fuels. (#1319)
92bcce8
Blood-Asp/GT5-Unofficial@92bcce8
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants