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

higher TEG ciurculator flow #36435

Merged
merged 1 commit into from
May 18, 2024

Conversation

CrazyAmphibian
Copy link
Contributor

[tweak]

alternative to #36431

What this does

increases the max flow rate of the TEG so that it can go to 100%. this affects power output and gas use rates.
Capture

Why it's good

fixes the flow rate being stuck at 33%.

Changelog

🆑

  • tweak: the TEG can now use 100% of its flow capacity instead of being limited to 33%

@D3athrow-Issues D3athrow-Issues added the Tweak I changed the thermostat from 70 degrees to 69 degrees. label Apr 30, 2024
@CrazyAmphibian
Copy link
Contributor Author

it doesn't change anything dealing with lube or waste+ethanol

@west3436
Copy link
Collaborator

What does the power generation look like for a basic roundstart Box TEG before and after this change is implemented?

@CrazyAmphibian
Copy link
Contributor Author

without: 440-100 kw

with: 400-90 kW

yes, the roundstart produces around 10% less from this one test (could be due to slight variations)
the numbers aren't that much different, which makes sense actually, because the same amount of energy is being given to the TEG, just at different speeds

@jwhitak jwhitak added Balance Potentially going to upset people as it changes balance of the game. ⛔ Requires Server Vote ⛔ This needs a server vote created ingame in order to be kept. labels May 4, 2024
@DamianX
Copy link
Contributor

DamianX commented May 4, 2024

Why it's good

fixes the flow rate being stuck at 33%.

That is a what, not a why!

without: 440-100 kw

with: 400-90 kW

yes, the roundstart produces around 10% less from this one test (could be due to slight variations) the numbers aren't that much different, which makes sense actually, because the same amount of energy is being given to the TEG, just at different speeds

You probably messed up testing. Did you use lube at all? You should have seen 0 difference if you didn't use lube, and 3x difference if you did, if the code comments are to be believed.

@DamianX
Copy link
Contributor

DamianX commented May 4, 2024

And why in the gosh darn is this a server vote? Are you going to have a server vote for every PR that conflicts with another? No one knows how this works, apparently not even the author.

@CrazyAmphibian
Copy link
Contributor Author

You probably messed up testing.

i mean yeah, the setups were not exactly the same due to the time it takes for gas to flow into the hot and cooling loops, but in terms of fuel supply they were the same

Did you use lube at all?

the roundstart setup does not include lube.

You should have seen 0 difference if you didn't use lube, and 3x difference if you did, if the code comments are to be believed.

there'd be a difference since more gas is available for use in the TEG than before. This affects it without lube. The result shows that the impact on low-end setups is negligible.

And why in the gosh darn is this a server vote

i don't really know but i don't think it's a bad thing

@DamianX
Copy link
Contributor

DamianX commented May 5, 2024

You should have seen 0 difference if you didn't use lube, and 3x difference if you did, if the code comments are to be believed.

there'd be a difference since more gas is available for use in the TEG than before. This affects it without lube. The result shows that the impact on low-end setups is negligible.

Wrong.
Your change only affects this line:
volume_capacity_used = min((last_pressure_delta * air1.volume / 3) / (input_starting_pressure * air1.volume), 1) //How much of the gas in the input air volume is consumed.
Which only affects the variable volume_capacity_used, which is only used here:

//If our circulators are lubed get extra power
if(circ1.reagents.get_reagent_amount(LUBE)>=1)
	last_gen *= 1 + ((circ1.volume_capacity_used * 100) / 16.5) //Up to x3 if flow capacity is 33%
if(circ2.reagents.get_reagent_amount(LUBE)>=1)
	last_gen *= 1 + ((circ2.volume_capacity_used * 100) / 16.5)

which means it will only have ANY impact when lube is used, therefore any testing you have done is irrelevant if you didn't use lube.

@DamianX
Copy link
Contributor

DamianX commented May 5, 2024

I should clarify that variable is also used by the "Flow capacity" element in the UI, which is why the progress bar goes up to 99 instead of 33 now.

@CrazyAmphibian
Copy link
Contributor Author

which means it will only have ANY impact when lube is used, therefore any testing you have done is irrelevant if you didn't use lube.

no? test the code for yourself if you don't believe me.
lube has no impact on the flow rate at all. all it does is multiply the power output.

@DamianX
Copy link
Contributor

DamianX commented May 5, 2024

That's correct, lube is supposed to multiply the power output. Your change only affects power generation when lube is used.

@CrazyAmphibian
Copy link
Contributor Author

That's correct, lube is supposed to multiply the power output. Your change only affects power generation when lube is used.

it doesn't. it always affects the maximum power generation.

@DamianX
Copy link
Contributor

DamianX commented May 5, 2024

Could you point me to the line or lines responsible for that?

@CrazyAmphibian
Copy link
Contributor Author

Nevermind, you are correct about the lube thing
i was confused because it uses 2 varibles for flow (thank you oldcoders)

@jwhitak
Copy link
Collaborator

jwhitak commented May 18, 2024

Lubing your TEG is now more powerful. Beware.

@jwhitak jwhitak merged commit ed7abbe into vgstation-coders:Bleeding-Edge May 18, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Balance Potentially going to upset people as it changes balance of the game. ⛔ Requires Server Vote ⛔ This needs a server vote created ingame in order to be kept. Tweak I changed the thermostat from 70 degrees to 69 degrees.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants