-
Notifications
You must be signed in to change notification settings - Fork 541
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
higher TEG ciurculator flow #36435
Conversation
it doesn't change anything dealing with lube or waste+ethanol |
What does the power generation look like for a basic roundstart Box TEG before and after this change is implemented? |
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) |
That is a what, not a why!
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. |
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. |
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
the roundstart setup does not include lube.
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.
i don't really know but i don't think it's a bad thing |
Wrong.
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. |
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. |
no? test the code for yourself if you don't believe me. |
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. |
Could you point me to the line or lines responsible for that? |
Nevermind, you are correct about the lube thing |
Lubing your TEG is now more powerful. Beware. |
[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.
Why it's good
fixes the flow rate being stuck at 33%.
Changelog
🆑