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

Enables some formerly debug suffixes for steering manager #2972

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

RCrockford
Copy link
Contributor

They are handy!

@nuggreat
Copy link

Not all of these suffixes should be exposed though the steering manager and some of them are set to be exposed on SHIP through PR #2881 specifically MOI and torque.

I will also comment here as I did there that any exposed vectors should be in ship-raw frame or ship-ship frame and should be structured such that they are convertible between by multiplying by SHIP:FACING or SHIP:FACING:INVERSE depending on the initial frame.

Also the usual note about "new" suffixes needing documentation.

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.

2 participants