|
From: | Christophe De Wagter |
Subject: | Re: [Paparazzi-devel] Exif latitude error |
Date: | Wed, 18 Nov 2015 17:27:44 +0100 |
I solved the altitude_ref error msg in master.
The GPS does not give altitude abbve STRM (ground). It always computes altitude above ellipsoid and most often also geoid (MSL) with a good earth model. The latter is now added to the pictures.
Hi ChristopheA break in the weather gave me the chance to undertake a stationary exif code (latest master version) test at a different location. Here is the image properties output for a single shot photo. The altitude compares well with the bebop GCS alt read out. Obviously there is a discrepancy between the SRTM elevation and the bebop gps recorded alt.From memory this is not the case with the ardrone2 ublox GPS.RegardsStephenFrom: Stephen L HulmeSent: Wednesday, November 11, 2015 10:37 AMSubject: Re: [Paparazzi-devel] Exif latitude errorHi Christophe,Thank you. I will test, weather permitting. I too do not have a compile error after instituting the latest exif code changes. I certainly must have caused the previous issue without noticing – sorry about that.Dear Stephen,Glad to see the exif at least has E/W and N/S now.
I looked at the documentation of EXIF and the altitude should be above mean sea level. At this time I was filling altitude above ellipsoid. I also added the code for negative altitudes while I had the docs open.I do not understand your compile problem. I can not reproduce, and I have a different version as well. The code in the pull request is version v5.7-devel-471 now and compiles fine for me.@flixr does state.h have a way to get AMSL?-ChristopheOn Thu, Nov 5, 2015 at 7:55 AM, Stephen L Hulme <address@hidden> wrote:
Hi Christophe,It may be relevant that I retrieved the altitude and location exif jpeg data from a stationary (motors off) srtm 50 metres elevation.It was too windy, but nevertheless not raining at the time.RegardsStephenFrom: Stephen L HulmeSent: Thursday, November 05, 2015 7:33 PMSubject: Re: [Paparazzi-devel] Exif latitude errorHi Christophe,First, thank you very much, it worked, apart from a small compile issue with the new exif_alt code.Reverting to the original IMAV2015 alt_code and the latest location revisions,Ubuntu produced this: http://www.omo.co.nz/Exif_ubuntu_2015-11-05.pngand Windows this: http://www.omo.co.nz/Exif_properties_5-11-15.pngRegardsStephenOn Thursday, November 05, 2015 2:20 AM, Christophe De Wagter <address@hidden> wrote:Apparently they are not allowed to be negative, but a separate REF tag is needed for north/south east/west. There is very little documentation so here is my attempt:Christophe
From: Christophe De WagterSent: Thursday, November 05, 2015 2:20 AMSubject: Re: [Paparazzi-devel] Exif latitude errorApparently they are not allowed to be negative, but a separate REF tag is needed for north/south east/west. There is very little documentation so here is my attempt:-ChristopheOn Mon, Nov 2, 2015 at 9:16 PM, Stephen L Hulme <address@hidden> wrote:
Hi ChristopheI must make a correction . I have not tried ExifSRational and int32_t simultaneously.I will certainly try this combination along with your changes when possible.RegardsStephenFrom: Stephen L HulmeSent: Tuesday, November 03, 2015 8:33 AMSubject: Re: [Paparazzi-devel] Exif latitude errorHi Christophe,I have tried int versus uinit and SRational for lat. with no change.Nonetheless, I appreciate your effort, but I think Felix maybe right about exif not being able to resolve negative values.I am prepared to try any changes, but it is difficult at this time of year with wind - http://www.metservice.com/marine-surf/recreational-marine/wellingtonI will update my repo and test as soon as possible.RegardsStephenOn November 03, 2015 6:37 AM, Christophe De Wagter <address@hidden> wrote:Trying to solve the issue for everyone here:https://github.com/paparazzi/paparazzi/pull/1399
From: Christophe De WagterSent: Tuesday, November 03, 2015 6:37 AMTo: paparazzi-develSubject: Re: [Paparazzi-devel] Exif latitude errorTrying to solve the issue for everyone here:
https://github.com/paparazzi/paparazzi/pull/1399
On Oct 26, 2015 8:47 PM, "Stephen L Hulme" <address@hidden> wrote:
Hi Christophe,Thank you for the prompt and possibly definitive reply. I will trial the change a soon as the weather permits.In respect of the EXIF module and the IMAV 2015 flight plan procedure, the pleasure has been all mine. Enac’s IMAV 2014 flight plan and now TU Delft’s offer great insight into the extensive possibilities of Paparazzi NAV mode flight. I thank all concerned, even Parrot for the platform.RegardsStephenFrom: Christophe De WagterSent: Monday, October 26, 2015 11:54 PMSubject: Re: [Paparazzi-devel] Exif latitude errorOn Monday, October 26.2015 11:54 PM, Christophe De Wagter <address@hidden> wrote:Hi Stephen,Thank you for using the module. Sorry about the issue of neglecting the southern hemisphere.I'm pretty sure the problem is the uint32_t type of attitude which obviously should get a sign: https://github.com/paparazzi/paparazzi/blob/master/sw/airborne/modules/computer_vision/lib/exif/exif_module.c#L30-ChristopheOn Sat, Oct 24, 2015 at 6:35 PM, Stephen L Hulme <address@hidden> wrote:
Hi Christophe De WagterI posted the following question on paparazzi/discuss – Gitter (20/10/15) and failed to get a reply.“Any ideas why the video_exif.xml module maybe outputting unintelligible GPS latitude reports in a southern hemisphere roaring forties location? View jpeg properties example”Could you please offer some hint how to rectify this error.RegardsStephen Hulme
_______________________________________________
Paparazzi-devel mailing list
address@hidden
https://lists.nongnu.org/mailman/listinfo/paparazzi-devel
_______________________________________________
Paparazzi-devel mailing list
address@hidden
https://lists.nongnu.org/mailman/listinfo/paparazzi-devel
_______________________________________________
Paparazzi-devel mailing list
address@hidden
https://lists.nongnu.org/mailman/listinfo/paparazzi-devel
_______________________________________________
Paparazzi-devel mailing list
address@hidden
https://lists.nongnu.org/mailman/listinfo/paparazzi-devel
_______________________________________________
Paparazzi-devel mailing list
address@hidden
https://lists.nongnu.org/mailman/listinfo/paparazzi-devel
_______________________________________________
Paparazzi-devel mailing list
address@hidden
https://lists.nongnu.org/mailman/listinfo/paparazzi-devel
_______________________________________________
Paparazzi-devel mailing list
address@hidden
https://lists.nongnu.org/mailman/listinfo/paparazzi-devel
_______________________________________________
Paparazzi-devel mailing list
address@hidden
https://lists.nongnu.org/mailman/listinfo/paparazzi-devel
_______________________________________________
Paparazzi-devel mailing list
address@hidden
https://lists.nongnu.org/mailman/listinfo/paparazzi-devel
_______________________________________________
Paparazzi-devel mailing list
address@hidden
https://lists.nongnu.org/mailman/listinfo/paparazzi-devel
_______________________________________________
Paparazzi-devel mailing list
address@hidden
https://lists.nongnu.org/mailman/listinfo/paparazzi-devel
_______________________________________________
Paparazzi-devel mailing list
address@hidden
https://lists.nongnu.org/mailman/listinfo/paparazzi-devel
[Prev in Thread] | Current Thread | [Next in Thread] |