qgroundcontrol icon indicating copy to clipboard operation
qgroundcontrol copied to clipboard

Follow failed: Home position not set

Open twistedwings opened this issue 1 year ago • 6 comments

If "Stream GCS Position" is enabled in Application settings, the message "Follow failed: Home position not set" pops up.

System Information

When posting bug reports, include the following information

  • Operating System: Windows 10
  • QGC Version: image
  • QGC build: daily
  • Flight Controller: SITL
  • Autopilot (with version): Ardupilot Copter 4.6.0

Detailed Description

Provide further details about your issue/bug.

Log Files and Screenshots

image

twistedwings avatar Oct 03 '24 17:10 twistedwings

This will probably happen until takeoff when you get your new home position. But actually I don't see why the vehicle position is needed for the gcs, seems kinda like a trick that assumes GCS and Vehicle are at same altitude initially and then the GCS is always at that altitude. I would think this is not a valid assumption to make and altMetersAMSL from the motion report should be used instead. @DonLakeFlyer thoughts on this? relevant code is here

HTRamsey avatar Oct 06 '24 17:10 HTRamsey

The code looks like it's related to follow mode. In the stable version is an option Stream GCS Position-only in follow mode. In the daily the option is on/off only. So there shouldn't be any dependency to follow.

twistedwings avatar Oct 06 '24 21:10 twistedwings

I remember a years lasting discussion about follow mode with the Ardupilot folks. I think the code uses the home altitude of the vehicle instead of gps altitude. Assuming gcs and vehicle are both at the same altitude, which is true in most cases. The reason for this trick is that the altitude information is garbage on most smartphones. So the code makes sense, only the error message is confusing and should be changed in "waiting for home" or the like.

twistedwings avatar Oct 07 '24 04:10 twistedwings

first thing to do is to restore the settings #11975

HTRamsey avatar Oct 07 '24 04:10 HTRamsey

I think the code uses the home altitude of the vehicle instead of gps altitude. Assuming gcs and vehicle are both at the same altitude, which is true in most cases. The reason for this trick is that the altitude information is garbage on most smartphones.

Correct.

The original pull is here: https://github.com/mavlink/qgroundcontrol/pull/8350.

DonLakeFlyer avatar Oct 09 '24 17:10 DonLakeFlyer

So the code makes sense, only the error message is confusing and should be changed in "waiting for home" or the like.

I'll take a look...

DonLakeFlyer avatar Oct 09 '24 17:10 DonLakeFlyer