KSP-RO/RP-1

Unable to finish Orbital Test Flight (Uncrewed) contract due to landed capsule not registering as "Return Home Safely".

Karhgath opened this issue · 6 comments

Seems to be 100% reproducible for this craft - I used a very minimalist Mercury without all the parts except heatshield. When it lands, it does not register that it is landed safely, so cannot end the contract other than F12.

Craft file and save included, here are different tests I did - even tried crewed with 8h resources and same thing.

Some images of 3 different attempts (the last 2 are sims, last one is crewed)
screenshot535
screenshot536
screenshot538

Save:
_afterLaunchReturnCapsuleBug.zip

Craft:
Mercury Uncrew [Mercury]R Thanos.zip

PS: Also tested by adding the Mercury Landing and Control Module, in case for some reason it had an impact, but no dice. Even worse, it triggered: #2282

screenshot539

Other contracts did not seem to exhibit the same behavior, even if using the same craft file. Not sure what happened with this contract or it was something situational.

siimav commented

The real problem is that "Stay in specified orbit for the duration" parameter is already ticked green while the vessel is sitting on the launch pad. Looks like somehow a previous vessel completed it and for whatever reason it doesn't get recalculated for the new vessel.

Did some more digging and there's 2 separate issues here.
The "return home" parameter doesn't complete because "crew capacity" flipped to unmet status. In some cases the contract widget doesn't update it immediately even though internally the status has already changed.
The other issue is All parameter getting stuck in completed status from other vessels/launches. This is due to it having disableOnStateChange = true coupled with the fact that it doesn't derive from VesselParameter and thus doesn't know how to track multiple vessels at the same time. Unfortunately there doesn't appear to be a VesselParameter that would serve as an alternative for such wrappers.

Please test with this: KSP-RO/ContractConfigurator#30

"Stay in specified orbit for the duration" group needs to be rewritten in the old-style Duration wrapped inside Orbit since KSP-RO/ContractConfigurator#29 should properly support it now.

Tested the above fix from my posted save, I am also nearing orbit in another one and will try there later

  • Normal launch and recovery worked. The requirements were not fluctuating during the whole mission, like when staging.
  • Normal launch, switching to another vessel, and coming back worked. The requirements were invalidated on the other vessel, but came back green when switched back. Recovery worked.
  • Normal launch, changing back to space center and fiddling around, and coming back. Working perfectly.
  • Launched with the Mercury control module which caused #2282 before. No issues with fluctuating requirements.

image

So both issues seem fixed now.

I will keep using the new CC in my current game and see if it's stable as well.