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

Tools: build_options.py: add AP_BARO_PROBE_EXTERNAL_COMPASSES #27516

Merged
merged 1 commit into from
Jul 10, 2024
Merged

Tools: build_options.py: add AP_BARO_PROBE_EXTERNAL_COMPASSES #27516

merged 1 commit into from
Jul 10, 2024

Conversation

peterbarker
Copy link
Contributor

@peterbarker peterbarker commented Jul 10, 2024 •
edited
Loading 

###### Enabling BARO_PROBEXT(AP_BARO_PROBE_EXTERNAL_I2C_BUSES) on copter costs -10304 bytes
###### Enabling BARO_PROBEXT(AP_BARO_PROBE_EXTERNAL_I2C_BUSES) on plane costs -10280 bytes
###### Enabling BARO_PROBEXT(AP_BARO_PROBE_EXTERNAL_I2C_BUSES) on rover costs -10280 bytes
###### Enabling BARO_PROBEXT(AP_BARO_PROBE_EXTERNAL_I2C_BUSES) on antennatracker costs -10528 bytes
###### Enabling BARO_PROBEXT(AP_BARO_PROBE_EXTERNAL_I2C_BUSES) on sub costs -10352 bytes
###### Enabling BARO_PROBEXT(AP_BARO_PROBE_EXTERNAL_I2C_BUSES) on blimp costs -10528 bytes

@peterbarker peterbarker mentioned this pull request Jul 10, 2024
Open
@peterbarker peterbarker merged commit 0d12cc7 into ArduPilot:master Jul 10, 2024
41 checks passed
@peterbarker peterbarker deleted the pr/build-options-probe-external branch July 10, 2024 23:46
@rmackay9
Copy link
Contributor

I tried to include this in 4.5.6-beta1 but was unable to due to a merge conflict. If someone is keen on including this in 4.5 then a backport PR would be greatly appreciated!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
MergeOnCIPass
Projects
4.5 Backports
Status: merge conflict
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@peterbarker @rmackay9