gotmachine / PhysicsHold

KSP Mod : Control physics for landed vessels
MIT License
9 stars 0 forks source link

Physics Hold + Kerbal Health incompatibility #2

Open yalov opened 2 years ago

yalov commented 2 years ago

way to reproduce:

@GarwelGarwel

gotmachine commented 2 years ago

KerbalHealth seems to be have different code paths based on if the vessel is packed or not, see here : https://github.com/GarwelGarwel/KerbalHealth/blob/25adc8c7089c7b33da7362f1bccaae2e88b2df24/Core.cs#L186-L195

Can't say why the issue you describe is happening exactely but in any case there is nothing that can be done from the Physics Hold side, as it put vessels in the packed state by design, which is a standard stock state.

I guess what happens is that KH only update stuff on loaded and unpacked vessels. If so, this is a debatable assumption but I guess there is some technical explanation behind that choice since this was explicitly introduced in this commit : https://github.com/GarwelGarwel/KerbalHealth/commit/24cbc1a14494c7db4fa24529c766deab497c7952