-
Notifications
You must be signed in to change notification settings - Fork 628
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
Barge Solars Adjustment #2695
base: master
Are you sure you want to change the base?
Barge Solars Adjustment #2695
Conversation
Also a price bump to 49000 to pay for the SMES. I could muck with the inventory to keep it under budget but that feels like it should be reserved for a deeper examination of the shuttle. |
#2734 contains a barge update that will cause a merge conflict if merged first, however it may be discarded (this commit can override that incomming one wholesale for an actual update) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Confirmed all information stated in the PR description is correct - power is working and sufficient for the number of thrusters (togglable thruster button should help regardless), air alarm setup seems fine. Bridge is fine, thruster setup is balanced and reasonable for the ship's size.
Looks good to me.
About the PR
Power room changes:
Bridge changes:
Misc changes:
Misc changes:
Why / Balance
horis97 on the Frontier discord raised an issue with the barge's power failing on them fairly consistently. A quick review revealed that the solars were feeling into the substation but not the SMES. This housekeeping change feeds the solars and generator both into the same power pool which should alleviate some of the headaches of maintaining consistent charge from that power system.
Now the solars are more actively providing juice for the entire power system and all systems can be actively monitored from the pilot's seat.
How to test
Buy it. Check that the power is going where it should be.
Double check that all thrusters are actually turning on/off with the switch.
Media
Requirements
Breaking changes
#2734 overwrites these changes. So we'll make adjustments when one or the other merges in.
Changelog
🆑