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

Override pre-arm checks #76

Merged
merged 2 commits into from
Mar 24, 2023

Conversation

BluemarkInnovations
Copy link
Contributor

Added a define in option.h to disable all pre-arm checks (OVERRIDE_ARM_STATUS). Default disabled, it is useful for development purposes.

Copy link
Contributor

@tridge tridge left a comment

Choose a reason for hiding this comment

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

I think we could add a uint32 DEV_OPTIONS parameter instead, default to zero, and make one bit of that be force arming checks passed
its safe as end user can't set parameters

@tridge
Copy link
Contributor

tridge commented Jan 10, 2023

@BluemarkInnovations I've implemented OPTIONS param as discussed and force pushed

@BluemarkInnovations
Copy link
Contributor Author

@tridge I have tested your reworked PR; it works great, thanks!

It seems to me also a good idea to set the OPEN_DRONE_ID_ARM_STATUS accordingly. So if OPTIONS_FORCE_ARM_OK is set to 1, also set the OPEN_DRONE_ID_ARM_STATUS to good to arm. I have included that feature in the latest commit and verified with the MP RID plugin.

@tridge tridge merged commit 3e70b26 into ArduPilot:master Mar 24, 2023
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