dkavolis / Ferram-Aerospace-Research

Aerodynamics model for Kerbal Space Program
Other
80 stars 32 forks source link

Stability derivates broken in latest release #75

Closed siimav closed 4 years ago

siimav commented 5 years ago

The latest release appears to have broken the stability derivates window on KSP 1.6.1 using RSS/RO/RP-1. This issue happens with every rocket I tried to load up.

screenshot334

dkavolis commented 5 years ago

Can you give a screenshot of what it looked like in the previous release? I've changed the stability derivatives to show NaN if no stable AoA was found whereas the previous release still computed the derivatives at that AoA which was unstable (and possibly random since the root solver would not converge so the derivative values would be garbage anyway). In regards to rockets, there shouldn't be stable AoA since they don't have enough lift. IMO there should be an input for the angle at which to compute the derivatives in addition to computing them at the stable AoA.

siimav commented 5 years ago

Reverted the FARc version and this is what I got with the same rocket. Truth be told, I don't really care about the stability derivatives directly. What I do find important is the Static analysis to figure out at what mach values my rocket or re-entry vehicle is statically stable at.

screenshot334

dkavolis commented 5 years ago

I can display stability derivatives at 0 AoA if there is no stable AoA found if that helps.

siimav commented 5 years ago

Not sure what that means. I have to ask though - why change the stability derivative system at all because it looked to be working fine before. At least previously the Static analysis tool was definitely giving meaningful values.

Gordon-Dry commented 5 years ago

Not sure if this is related, but when enabling one of the Transonic Design curves, a second "curve" in black appears nearby - but it's just a line (see left side of picture): screenshot_2019-07-14--19-09-53

This did not happen in the previous build.