Fixed some compatibility issues#286
Conversation
Fix compatibility with other planet mods. The previous version removes all the other celestial bodies. Newer version only removes stock celestial bodies.
altitude, maxAltitude, and atmosphereDepth can all set max atmosphere heights. The previous version can only apply to configs using maxAltitude, but if there are configs using altitude or atmosphereDepth, Module Manager errors will pop in. This new version fixes the issue and improves stability and compatibility.
|
The, uh, point of RSS is that it's the real solar system. It should by definition not be compatible with other planet packs, unless they explicitly are designed to add other real bodies (and in which case will know RSS is installed and can run AFTER[] the RSS pass). Other change seems fine. 👍 |
|
I understand your point. However, there are many people telling me they want to play my planet mod together with RSS, currently the only thing I can do is provide them with this version of config. I think it doesn't hurt to let people choose their mod selection, and these changes won't have any impact on either RSS or KSP. |
Also add compatibility for the "altitude" and "atmosphereDepth" parameters (pull #286)
Also add compatibility for the "altitude" and "atmosphereDepth" parameters (pull #286)
|
The author is MIA and the PR cannot be merged due to the Kopernicus settings changes. Relevant navball switcher patches will be integrated manually with #320. |
altitude,maxAltitude, andatmosphereDepthcan all set max atmosphere heights. The previous version can only apply to configs usingmaxAltitude, but if there are configs usingaltitudeoratmosphereDepth, Module Manager errors will pop in. This new version fixes the issue and improves stability and compatibility.