Skip to content

Commit

Permalink
Respect shielding for chute deployment
Browse files Browse the repository at this point in the history
  • Loading branch information
dkavolis committed Aug 30, 2022
1 parent efec236 commit 4121b20
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions FerramAerospaceResearch/RealChuteLite/RealChuteFAR.cs
Expand Up @@ -272,13 +272,13 @@ public bool CanDeploy
{
get
{
if (GroundStop || atmPressure.NearlyEqual(0))
if (GroundStop || atmPressure.NearlyEqual(0) || part.ShieldedFromAirstream)
return false;
if (DeploymentState == DeploymentStates.CUT)
return false;
if (PressureCheck)
return true;
return !PressureCheck && IsDeployed;
return IsDeployed;
}
}

Expand Down

2 comments on commit 4121b20

@thunder175
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Potentially causing an issue with Bluedog Hermes/Mercury main parachute. Chute is inside another part and will not deploy properly. See below post:

https://forum.kerbalspaceprogram.com/index.php?/topic/122020-1123-bluedog-design-bureau-stockalike-saturn-apollo-and-more-v1104-%D0%BB%D1%83%D0%BD%D0%B0-19july2022/&do=findComment&comment=4179007

@dkavolis
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This now respects shieldedCanDeploy field from the stock config, it should match stock behaviour in such cases.

Please sign in to comment.