-
Notifications
You must be signed in to change notification settings - Fork 54
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
[BUG] YAFC thinks Wind Turbines require power #205
Comments
I reproduce the Sea Block Pack analysis error, but I can't reproduce this particular claim. With only KS Power enabled, YAFC determines that electricity is accessible. Even with SeaBlock (but not SeaBlockMetaPack), YAFC decides that electricity is accessible, so it looks to me like something added by SeaBlockMetaPack is confusing YAFC. Will you find the minimum mod set required to get YAFC to come to the wrong conclusion, please? |
IssueWith just KS Power mod, YAFC thinks that Wind Turbines require Electricity. This is incorrect. In this configuration, this doesn't cause the user any problem. But it's still incorrect. Instead, it should say Void (Power). Minimum Mod ListIf you want to see the Analysis warning from mods on the mod portal, mod list:
Test ModAlternatively, I've made a tiny test mod that demonstrates the same issue. Mod list:
|
Ah, OK. I see now.
YAFC also knows that it consumes power: I agree that changing
is inelegant, but I don't really see a better fix. Technically, YAFC's current behavior is correct, in that (as far as I can tell) all EEIs both consume and produce power. But the changed behavior matches the actual usage of EEIs in mods, which is to generate free power. Nullius gets the wrong answer for its wind turbines and Stirling engines, but the answer after the change isn't particularly wronger than the answer before the change, and I think that's something that will require special hooks for Nullius. |
I ran in the same issue with Pyanodon mod-suite for the Multiblade "fish" Turbine. I have no idea how EEIs work, but I guess the mod keeps inserting energy so it produces/delivers power to the network? YAFC knows (somehow) the amount/maximum power the EEI can provide (I guess it is part of the proptype?). Maybe it makes sense to add a UI component to let the user confirm that the EEI indeed delivers this amount of power manually? It prevents that YAFC is incorrectly assuming that the EEI is indeed meant as a power source, but still allows the modded EEI act as power sources... 🤔 |
This fixes #226 and ShadowTheAge#205. There are still improvements possible; see #228 for more, but I think that's a separate issue.
Reproduction
YAFC v0.5.8
Mod List:
Issue can be seen in YAFC with just KS Power mod.
Enable Sea Block Pack mod (and prerequisites) to see Analysis Warning.
https://mods.factorio.com/mod/KS_Power
https://mods.factorio.com/mod/SeaBlockMetaPack
Issue Description
When using the full Sea Block mod pack, YAFC gives an Analysis Warning.
YAFC doesn't think that
Electricity generating (Mechanics)
is unlocked.Wind Turbine (entity) should not depend on Electricity (power). It generates power! YAFC knows that it generates power:
Wind Turbine Details
Type:
electric-energy-interface
Electric Energy Source Prototype:
The text was updated successfully, but these errors were encountered: