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

Change ComputeRCSCharacteristics to use the sum of positive thrusts, … #2651

Merged
merged 1 commit into from
Jul 17, 2020

Conversation

RCrockford
Copy link
Contributor

…rather than the average since multi-port thrusters appear to be able to deliver full thrust down any single port.

This fixes #2571.

@eggrobin eggrobin added the LGTM label Jul 17, 2020
@pleroy
Copy link
Member

pleroy commented Jul 17, 2020

[Automated message from GitHub Pull Request Builder] Answer "ok to test" to trigger testing of this PR.

Sorry, something went wrong.

@eggrobin
Copy link
Member

ok to test

Sorry, something went wrong.

@eggrobin eggrobin merged commit cadff58 into mockingbirdnest:master Jul 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

RCS burns in editor are incorrect for multi-port thrusters
3 participants