Changes

Fix inverse rotation maneuver bugs

GetOrbitalStateVectorsAtUT() is particularly whack in the way that it
applies the inverse rotation constructed at a future time via using
Planetarium.ZupAtT().  That means that for the most part it is only
useful for constructing values which can only be compared to other
vectors constructed at the same time.  This bug only occurs when the
vessel is below the inverse rotation threshold, though, so most of
the time works fine when there's no rotation being applied.

The changes to RightHandedStateVectorsAtUT mean that we apply our
own rotation in the current frames rotation to get RH rotating
vectors.  This is consistent with the old API, but should probably
be retired and everything migrated to RH non-rotating vectors now
that I can see how to get them out of the API correctly.

This may also fix other bugs in consumers of the underlying
maneuvers class (e.g. rendezvous autopilot, etc).

Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
 (commit: 07b9f1c)

Remove old orbit manipulation APIs

These use GetOrbitalStateVectorsAtUT() which is problematic, they don't
call Init() and have other sketchy looking behavior and nothing has
used them in awhile.

Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
 (commit: 92cb9f8)

Transfer planner: GetOrbitalStateVectorsAtUT fix

This fixes a (now) obvious bug in the transfer planner in the
use of GetOrbitalStateVectorsAtUT(), although it seems to only
affect debug log output.

Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
 (commit: d189c96)

Prevent Reflection errors from other mods tanking MJ

Avoids using the KSP.IO.File.Exist<T>() API that walks the loaded
assemblies and throws, looking for the path to the assembly with
the type T.

The MuUtils helper replicates the side effect of this API of
creating the directory.

Since we construct the path afterwards anyway to load the file the
only reason I can see for the reflection-driven-API is for that
side-effect, and to cause weird bugs if someone ever moves the DLL
location around and makes the two APIs start to disagree.
 (commit: d6adb4e)

Fix another NRE

fixes #1611 by just creating a lazy accessor

Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
 (commit: 47b3328)

Fix staging controller for stock fairings

somehow i had convinced myself the stock fairing was a moduledecouple
when it is actually not.
 (commit: d15b46c)

Fix stock fairings correctly

This makes stock fairings work correctly when they also have a payload
 (commit: b8fe8fe)

Fix node executor to align COT to the node direction

Should help fix e.g. shuttles work with the node executor

Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
 (commit: 82bde4c)

Classic ascent circularization

Don't try to set the SMA based on desired altitude, but just
circularize at whatever the apoapsis actually is.

We could try to circularize at the precise altitude if the desired altitude <
apo, but that could get more costly so I'm going to skip thinking about
it (some people will want it, some people probably won't, heuristics
will add more edge conditions and buttons and I don't think it is worth
it).

closes #1971
closes #888 (maybe)
 (commit: a1e0426)

Fix limit throttle/accel editing linkage

closes #1590 finally
 (commit: d2ba1b1)

Fix RCS solver NRE

Closes #2217

Couple other incredibly minor style fixes snuck in.
 (commit: 3f21600)

v2.15.3

This also updates the other versions and establishes a more
sensible versioning policy.
 (commit: a5a2218)