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

AP_Notify: flash green lights based off location not GPS #27385

Merged
merged 1 commit into from
Jun 26, 2024

Conversation

peterbarker
Copy link
Contributor

@peterbarker peterbarker commented Jun 25, 2024

Your Copter in stabilize mode can flash rapid-green indicating a good DGPS lock or better, and yet your vehicle still doesn't have a good idea of where it is.

That means that your vehicle may end up RTL'ing onto a point somewhere along your flight path, when you were given an all-green indication by teh vehicle before you armed in stabilize.

Past this PR, we require the same GPS as before, but in addition we must have been happy enough with our location to set the navigation origin, and currently be able to get a location.

A user will receive slow-flashing blue lights if they can't currently get a location, or the navigation origin isn't set, even if they've got a "good" fix.

We also require a good location to get a solid green light - you will get a solid blue light if you can't get a location and have a navigation origin, even if you have a good GPS lock

Closes #27320

Tested in SITL using --rgbled

Your Copter in stabilize mode can flash rapid-green indicating a good DGPS lock or better, and yet your vehicle still doesn't have a good idea of where it is.

That means that your vehicle may end up RTL'ing onto a point somewhere along your flight path, when you were given an all-green indication by teh vehicle before you armed in stabilize.

Past this PR, we require the same GPS as before, but in addition we must have been happy enough with our location to set the navigation origin, *and* currently be able to get a location.

A user will receive slow-flashing blue lights if they can't currently get a location, or the navigation origin isn't set, even if they've got a "good" fix.

We also require a good location to get a solid green light - you will get a solid blue light if you can't get a location or don't have a navigation origin, even if you have a good GPS lock
@peterbarker peterbarker force-pushed the pr/ahrs-not-gps-for-lights branch from 5549fce to 48706f4 Compare June 25, 2024 07:37
@andyp1per
Copy link
Collaborator

I really want this as a pre-arm option as well. I am forever waiting for status in loiter before switching to acro and flying because I want to make sure that RTL will work well.

@okok125
Copy link

okok125 commented Jun 25, 2024

Thank You Peter Barker.

Copy link
Contributor

@rmackay9 rmackay9 left a comment

Choose a reason for hiding this comment

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

LGTM.

Now we can look forward to the support calls, "I upgraded to 4.6 and I can't get a GPS lock any more". Despite this I still think it's a good change, thanks!

@peterbarker
Copy link
Contributor Author

I really want this as a pre-arm option as well. I am forever waiting for status in loiter before switching to acro and flying because I want to make sure that RTL will work well.

#27367

You're welcome ;-)

@peterbarker
Copy link
Contributor Author

Now we can look forward to the support calls, "I upgraded to 4.6 and I can't get a GPS lock any more". Despite this I still think it's a good change, thanks!

Is there somewhere we should post about this to head those questions off?

@rmackay9
Copy link
Contributor

Hi @peterbarker,

This wiki page should probably be updated https://ardupilot.org/copter/docs/common-leds-pixhawk.html

I've added an item to my 4.6.0 release warnings text document so when it comes time to do the beta testing and final release I'll add a warning to the blog post and release announcements

@okok125
Copy link

okok125 commented Jun 26, 2024 via email

@tridge tridge merged commit 2425023 into ArduPilot:master Jun 26, 2024
92 checks passed
@peterbarker peterbarker deleted the pr/ahrs-not-gps-for-lights branch June 26, 2024 08:59
@peterbarker peterbarker added the WikiNeeded needs wiki update label Jun 26, 2024
@Hwurzburg
Copy link
Collaborator

@peterbarker this seems incomplete...what about systems using LEDA and LEDB??this only affects RGB notify leds

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
WikiNeeded needs wiki update
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ArduPilot should flash green lights when AHRS-is-happy, *not* GPS-is-happy
7 participants