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

Preliminary BWR Functionality #1

Merged
merged 3 commits into from
Oct 15, 2024

Conversation

funions123
Copy link
Contributor

Adds the ability to pump distilled water directly into a reactor core and get steam.
Efficiency (configurable) currently set at 2x compared to LHE 5x.
Maximum output per reactor set at around 600 HU/s, based on the size of the output buffer tank.
Intended as an early, low-titanium way of harnessing nuclear power, since without the LHE the amount of titanium required for a full setup is very low. Consequently, it is less efficient and less power dense. However, you can still pull around 2A of EV from one of these with these numbers.

ISSUES: Reactor fluid ports should probably be configurable to auto-eject fluids, since pumps at HV/EV tier can't extract 200000 steam/s.

Adds the ability to pump distilled water directly into a reactor core and get steam.
Efficiency (configurable) currently set at 2x compared to LHE 5x
Maximum output per reactor set at around 600 HU/s, based on the size of the output buffer tank.
this.roundedHeat -= consumedCoolant * coolantCache.specificHeatCapacity;
this.addedHeat += consumedCoolant * coolantCache.specificHeatCapacity;

this.coolantTank.drain(consumedCoolant, true);
this.hotCoolantTank.fill(new FluidStack(coolantCache.hot, consumedCoolant), true);
//for BWRs, convert distilled water to a configured amount of steam instead of the same quantity of hot coolant

Choose a reason for hiding this comment

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

It might be useful to make this fluid multiplying logic generic, but I can't think of anything else that would use it so this is fine for now.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I considered adding generic values but I didn't want to significantly change any templates for an exceptional case

@RecursivePineapple RecursivePineapple merged commit 0401120 into RecursivePineapple:master Oct 15, 2024
1 check passed
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.

2 participants