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

Fixes combustion turbine power grid #19973

Conversation

FabianK3
Copy link
Contributor

@FabianK3 FabianK3 commented Oct 1, 2024

Introduction

The power grid of the new combustion turbine is behaving very weird. For example, the TEG outputs 300kW of power, but inspecting the wire with a multitool shows very random things like 12mW. The SMES shows a different value as well.

Fix

I noticed that a lot of cabling has been used with the d1,d2 properties set, which the wiki states to not use. I replaced all cabling starting from the TEG up to the main grid with cabling without the d1,d2 properties. This fixed all power values in the power network. The TEG, cabling and SMES now show the exact same value of power available in the network.

Additionally i added a powernet sensor to the turbine grid, as it was missing.

Edit: Grammar

@FabianK3 FabianK3 requested a review from DreamySkrell as a code owner October 1, 2024 23:38
@FabianK3
Copy link
Contributor Author

FabianK3 commented Oct 1, 2024

!bugfix
!review

@github-actions github-actions bot added the 🗺️ Mapping - Horizon The PR touches the Horizon map files. label Oct 1, 2024
@FabianK3
Copy link
Contributor Author

FabianK3 commented Oct 1, 2024

!bugfix

@BotBOREALIS BotBOREALIS added the Bugfix The PR is fixing an issue and requires less reviews and time to be ready for merging. label Oct 1, 2024
@furrycactus
Copy link
Contributor

Oops. Thanks king.

@NonQueueingMatt NonQueueingMatt added this pull request to the merge queue Oct 2, 2024
Merged via the queue into Aurorastation:master with commit 197d44b Oct 2, 2024
24 checks passed
@FabianK3 FabianK3 deleted the bugfix-combustion-turbine-wiring branch October 2, 2024 16:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Awaiting Merge Bugfix The PR is fixing an issue and requires less reviews and time to be ready for merging. 🗺️ Mapping - Horizon The PR touches the Horizon map files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants