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.

Sign up for GitHub

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

Jump to bottom

Pr control allocation mc only #15359

Merged
merged 3 commits into from
Jul 17, 2020
Merged

Pr control allocation mc only #15359

merged 3 commits into from
Jul 17, 2020

Conversation

sfuhrer
Copy link
Contributor

@sfuhrer sfuhrer commented Jul 17, 2020

  • disable prearm for actuator_controls_4 and actuator_controls_5 (control allocation output) --> treat them all as rotors
  • revert the params to simulate actuator failures without re-computation of the effectiveness matrix (needed to show different performance of recovery with effectiveness matrix updated after failure and without)
  • clean up CA_AIRFRAME param a bit

@sfuhrer
mixer module: handle all actuator_controls of 4 and 5 as rotors, e.g.…
f1e5d3d 
… don't output something while pre-armed

Signed-off-by: Silvan Fuhrer <silvan@auterion.com>
@sfuhrer
Revert "Add params to simulate actuator failure"
fd0e253 
This reverts commit b247368.
@sfuhrer
CA_AIRFRAME: clean up param description
5f608c5 
Signed-off-by: Silvan Fuhrer <silvan@auterion.com>
@sfuhrer sfuhrer requested a review from jlecoeur July 17, 2020 08:40
jlecoeur
jlecoeur approved these changes Jul 17, 2020
View reviewed changes
Copy link
Contributor

@jlecoeur jlecoeur left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good

@sfuhrer sfuhrer merged commit 66b066d into pr-control_allocation Jul 17, 2020
@sfuhrer sfuhrer deleted the pr-control_allocation_MC_only branch July 17, 2020 09:18
@weekly-digest weekly-digest bot mentioned this pull request Jul 19, 2020
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jlecoeur jlecoeur jlecoeur approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@sfuhrer @jlecoeur