search

post-kerbin-mining-corporation / NearFutureSolar

Adds more solar panel parts to Kerbal Space Program
32 stars 25 forks source link

Replace hardcoded Kerbin altitude above sun #42

Closed jsolson closed 6 years ago

jsolson commented 6 years ago

It will be different in a rescaled system.

ChrisAdderley commented 6 years ago

Ironically, this will keep it hardcoded. You need a [KSPField(isPersistant = false)] to read data from a config.

Also, please target to dev

jsolson commented 6 years ago

It's not a field. The value is retrieved in the Start() method. The initialization value is completely unnecessary - just habit.

Apologies it was late. I'll resubmit to dev.

ChrisAdderley commented 6 years ago

Ah I probably shouldn't read code in 5 minutes at work then :P.