Skip to content

Altitude hold#161

Merged
rtlopez merged 6 commits intomasterfrom
althold
Jan 9, 2026
Merged

Altitude hold#161
rtlopez merged 6 commits intomasterfrom
althold

Conversation

@rtlopez
Copy link
Owner

@rtlopez rtlopez commented Mar 2, 2025

No description provided.

@rtlopez rtlopez marked this pull request as draft March 2, 2025 22:32
@rtlopez rtlopez mentioned this pull request Dec 4, 2025
@DanielKusyDev
Copy link

Hi @rtlopez ,

Excellent work on altitude hold! I've analyzed your barometer implementation and would like to propose extending it with VL53L0X rangefinder support for precision low-altitude control (0-2m, +/-3mm accuracy). There is no particular reason for that exact sensor (VL53L0X), I just happen to have one.

  • Barometer: Extended range (2m+) -> Already in your PR
  • Rangefinder: Precision at low altitude (0-2m, +/- 3mm vs +/- 1m)
  • Automatic sensor fusion with smooth transitions

Approach:

  • Build on your existing altitude hold architecture
  • Add rangefinder as alternative altitude source
  • Keep your PID controller and mode logic unchanged
  • HAL design for easy addition of more sensors (VL53L family).

Does that sound okay? If so, would you prefer I extend your PR or create a separate one that builds on yours?

@rtlopez rtlopez marked this pull request as ready for review January 9, 2026 21:24
@rtlopez rtlopez merged commit d44487e into master Jan 9, 2026
16 checks passed
@rtlopez rtlopez deleted the althold branch January 9, 2026 21:31
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