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

Added Water Pump #2694

Merged
merged 23 commits into from
Jul 18, 2024
Merged

Added Water Pump #2694

merged 23 commits into from
Jul 18, 2024

Conversation

evgengoldwar
Copy link
Contributor

Water Pump :
image
image

Tier 1 :
image

Tier 2 :
image

@chochem chochem added the Affects Balance Change affecting balance. Requires admin approval before merging. label Jun 29, 2024
@chochem
Copy link
Member

chochem commented Jun 29, 2024

did you discuss this with the dev team at all? please dont make big changes like this to early without any prior discussion.

here some obvious starting question:
why?
how much does it make? (tooltip is very unclear)
how much fuel/what type of fuel does it take?

@evgengoldwar
Copy link
Contributor Author

did you discuss this with the dev team at all? please dont make big changes like this to early without any prior discussion.

here some obvious starting question: why? how much does it make? (tooltip is very unclear) how much fuel/what type of fuel does it take?

Dream asked to add to the existing Steam Washer as it requires a lot of water.
The water extraction formula is written in the tooltip. Humidity and Tiers are written in the wailla on the controller.
It requires nothing to run.

@chochem
Copy link
Member

chochem commented Jun 29, 2024

This is not a steam washer.
As I said the tooltip is totally unclear. (no time frame! 40% humidity is 40 or 0.4?)
A pump that runs on nothing is nonsense. We already have a water pump and it uses fuel.

@chochem
Copy link
Member

chochem commented Jun 29, 2024

to add to those questions: why would humidity affect a water pump?

@Reflex18
Copy link
Contributor

Is this a replacement to the Stirling dynamo and an upgrade to the railcraft water tanks?

@evgengoldwar
Copy link
Contributor Author

This is not a steam washer. As I said the tooltip is totally unclear. (no time frame! 40% humidity is 40 or 0.4?) A pump that runs on nothing is nonsense. We already have a water pump and it uses fuel.

40% = 40 As already said in wailla the humidity of the biome is deduced

@Dream-Master
Copy link
Member

The new washer need tons of water so a pump is a nice idea and needed. About blance when can discuss.

@evgengoldwar
Copy link
Contributor Author

to add to those questions: why would humidity affect a water pump?

It's a mix of Railcraft Water tank and Pump from Impact.

@chochem
Copy link
Member

chochem commented Jun 29, 2024

Is this a replacement to the Stirling dynamo and an upgrade to the railcraft water tanks?

replacing the stirling pump with something nicer looking (at roughly the same mechanics) could be a cool thing yea. that is not what this is sadly.

@chochem
Copy link
Member

chochem commented Jun 29, 2024

This is not a steam washer. As I said the tooltip is totally unclear. (no time frame! 40% humidity is 40 or 0.4?) A pump that runs on nothing is nonsense. We already have a water pump and it uses fuel.

40% = 40 As already said in wailla the humidity of the biome is deduced

you are not saying anything if you dont say the time frame. for th railcraft tanks its 8 ticks. but for all I know you mean per second or per tick instead.

@chochem
Copy link
Member

chochem commented Jun 29, 2024

The new washer need tons of water so a pump is a nice idea and needed. About blance when can discuss.

we have a pump already dream

@evgengoldwar
Copy link
Contributor Author

This is not a steam washer. As I said the tooltip is totally unclear. (no time frame! 40% humidity is 40 or 0.4?) A pump that runs on nothing is nonsense. We already have a water pump and it uses fuel.

40% = 40 As already said in wailla the humidity of the biome is deduced

you are not saying anything if you dont say the time frame. for th railcraft tanks its 8 ticks. but for all I know you mean per second or per tick instead.

per second

Copy link
Member

@chochem chochem left a comment

Choose a reason for hiding this comment

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

thanks for answering some questions. Though these discussion should really happen in advance and not on the PR.

I propose we close this PR.

We already have a water pump which is better (excepty for looks). Specifically:

  • a water pump based on humidity makes no sense.
  • a water pump without power/fuel of any kind makes no sense.
  • adding some bronze to a water tank to make it produce (literally) 200x as much makes no sense.

If the reason for this was the new steam washer, there is a much simpler solution. Reduce the water consumption. It acts as a simple washer only, right? Meaning no byprodructs. You can simple wash 3 stacks of material with 1000L water in a cauldron, so low water usage for simple washing has precedent in the game.

Regarding water pumps, I could see us using a multiblock structure like this as a replacement for the existing (stirling) water pump but with its existing mechanics (based on fuel not humidity), just to make it look nicer. Outside of looks the existing water pump seems fine.

@Dream-Master
Copy link
Member

The steam washer has byproducts. Its not a simple washer.
The stirling Water pump from bartworks was a duct tape solution and never a very good way to generate water.
reduce the amount of water make the single block washer completly useless and break balance.
About how this pump have to generate/pump water we can discuss.

@dahgul333
Copy link
Contributor

thanks for answering some questions. Though these discussion should really happen in advance and not on the PR.

I propose we close this PR.

We already have a water pump which is better (excepty for looks). Specifically:

* a water pump based on humidity makes no sense.

* a water pump without power/fuel of any kind makes no sense.

* adding some bronze to a water tank to make it produce (literally) 200x as much makes no sense.

If the reason for this was the new steam washer, there is a much simpler solution. Reduce the water consumption. It acts as a simple washer only, right? Meaning no byprodructs. You can simple wash 3 stacks of material with 1000L water in a cauldron, so low water usage for simple washing has precedent in the game.

Regarding water pumps, I could see us using a multiblock structure like this as a replacement for the existing (stirling) water pump but with its existing mechanics (based on fuel not humidity), just to make it look nicer. Outside of looks the existing water pump seems fine.

  1. We alredy have humidiry based water source - Water Tank (If you want a near-realistic justification, the groundwater depends on humidity+-.
    2)instead of fuel, we pay with the cost of the structure
    3)I don't see any point in being greedy for water.

@Dream-Master
Copy link
Member

This pump use steam which need coal to work. About the pump rate and other balacing stuff we can discuss here or in Discord.

@chochem
Copy link
Member

chochem commented Jun 29, 2024

The steam washer has byproducts. Its not a simple washer. The stirling Water pump from bartworks was a duct tape solution and never a very good way to generate water. reduce the amount of water make the single block washer completly useless and break balance. About how this pump have to generate/pump water we can discuss.

why does it have byproducts? the steam macerator does not.

@chochem
Copy link
Member

chochem commented Jun 29, 2024

This pump use steam which need coal to work. About the pump rate and other balacing stuff we can discuss here or in Discord.

no. they say it uses nothing.

@ah-OOG-ah
Copy link
Member

The steam washer has byproducts. Its not a simple washer. The stirling Water pump from bartworks was a duct tape solution and never a very good way to generate water. reduce the amount of water make the single block washer completly useless and break balance. About how this pump have to generate/pump water we can discuss.

why does it have byproducts? the steam macerator does not.

Probably because unlike the macerator, there is an LV washer that gives byproducts. It's just slower and more expensive to run.

@Dream-Master
Copy link
Member

Pump need to used Steam so it need to be changed. About Byproducts. Lv macerator has no byproducts same as multi. Lv washer has byproducts same as the new multi

@chochem
Copy link
Member

chochem commented Jul 16, 2024

Steam based is a good approach. Apparently that was Dreams idea from the start, so that is great.
And at least not depending on skycheck but only humidity improves things a lot on that front for sure.

Outside of improving the tooltip, I guess a big discussion point left is balance. If I understand correctly this now produces 5000L/s water for 400L/s running cost. Well with the production scaled down proportionally at less than 100 humidity and doubled with steel tier though the steam consumption remains the same. (please correct me if wrong)
For comparison the passive railcraft water tank makes 25L/s water at 100 humidity and the coal/charcoal/similar based existing water pump makes 150L/s (I dont remember the fuel consumption rate).
So right now even just the bronze tier is like 200 water tanks (but with steam consumption!) or like 33 water pumps (and probably more efficient. That is a lot, and way more than you need in steam age, even with the washer. (and its highly unlikely you will run the washer a lot with max parallels anyway)

@Dream-Master Dream-Master requested a review from a team July 17, 2024 04:21
Copy link
Contributor

@NotAPenguin0 NotAPenguin0 left a comment

Choose a reason for hiding this comment

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

There are a few minor code quality nitpicks I'd like to see addressed. Some of these aren't necessary, but would improve maintainability a little and make it easier for others unfamiliar with the implementation to change this in the future.

There's also a few other things like trying to automatically determine the tier of the multi instead of having to set it with a screwdriver, but overall well done on the multi.

As for balancing, I think in it's current state it's mostly fine. Spamming the RC water siding tanks in Steam/Early LV is not very interesting in my opinion, so having a pump multiblock that's a bit stronger does make some sense. It would also increase the viability of the large boilers in LV/MV a bit, which I'm all for.

@NotAPenguin0
Copy link
Contributor

No more comments from me

@Dream-Master
Copy link
Member

if the values need to be changed we can do another pr

@Dream-Master Dream-Master merged commit 5e6ba0d into GTNewHorizons:master Jul 18, 2024
2 checks passed
evgengoldwar added a commit to evgengoldwar/GT5-Unofficial that referenced this pull request Jul 19, 2024
* Add Water Pump

* Added check humidity

* Apply spotless

* Added tier 2 WaterPump

* Added new textures and Fixed the display in nei

* Optimizing imports

* Fix tooltip

* Added craft for Output hatch ULV, Wooden Casing, Water Pump

* Apply spotless

* sa+update deps

* Delete the sky check method

* sa

* Rewrote WaterPump to SteamMultiBase.

* Added a steam hatch to the structure and changed the tooltip

* Spotless apply

* Changed VoidProtection and calculate water generation

* Remove star imports

* Rename Wooden Casing to Primitive Wooden Casing

* Remove enableMultiblock_WaterPump

* Rename static variables

* Change variables and tooltip

* Sa

---------

Co-authored-by: Dream Master <[email protected]>
@evgengoldwar evgengoldwar deleted the WaterPump branch September 5, 2024 08:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Affects Balance Change affecting balance. Requires admin approval before merging.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants