Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix much Torquestuff #1542

Open
wants to merge 2 commits into
base: dev
Choose a base branch
from
Open

Fix much Torquestuff #1542

wants to merge 2 commits into from

Conversation

lamont-granquist
Copy link
Collaborator

  • convert Vector6 to a struct
  • convert availableTorque to a Vector6
  • convert Vector6 ToVector3d in a few places to maintain backcompat
    (places that may continue to be buggy I suspect, but out of scope for now)
  • in the BetterController select which torque 'sense' to use based on
    the sign of the component of the error vector (asymmetric torque
    handling)
  • Abs() the values coming off of GetPotentialTorque() and fix the sense
    of the vector manually to work around KSP bugginess with pos/neg
    values being all over the place.

- convert Vector6 to a struct
- convert availableTorque to a Vector6
- convert Vector6 ToVector3d in a few places to maintain backcompat
  (places that may continue to be buggy I suspect, but out of scope for now)
- in the BetterController select which torque 'sense' to use based on
  the sign of the component of the error vector (asymmetric torque
  handling)
- Abs() the values coming off of GetPotentialTorque() and fix the sense
  of the vector manually to work around KSP bugginess with pos/neg
  values being all over the place.

Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant