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

Altitude is negative #30

Open
sbonaime opened this issue May 22, 2024 · 4 comments
Open

Altitude is negative #30

sbonaime opened this issue May 22, 2024 · 4 comments

Comments

@sbonaime
Copy link

Hi,
I found this in log for a specific set.
Altitude is negative (-X.XXX) : viewing directions are probably divergent. Using default altitude of 1.0

As Altitude is get from GPS data it maybe it is not a good idea to set it arbitrary to 1.0 Sometimes we can have images taken below GPS Sea level value (https://en.wikipedia.org/wiki/List_of_places_on_land_with_elevations_below_sea_level)

Could it be possible to force (with an option) the found negative altitude ?
Thanks

@smathermather
Copy link

smathermather commented May 22, 2024

Proposed pull request to address, keeping it simple: if Z is below Lake Assal, it's probably invalid, but treat as valid otherwise:
#32

@sbonaime
Copy link
Author

@smathermather I am now using ODM for underwater data ... so maybe we can have an option for real underwater images ?

@smathermather
Copy link

smathermather commented May 22, 2024

This would work for any underwater images above the Afar Depression, but I just realized the Dead Sea is lower. I've changed the proposed value. So, anyone doing underwater images on the Dead Sea will be out of luck, or need to change their version.

This also applies to areas where the geoid is well below the ellipsoid, which is an even longer list than the above. Much of the coast of Tanzania is below sea level on the elipsoid, but the India Ocean hasn't read the map, so folks stay dry enough.

@smathermather
Copy link

@sbonaime -- do you have an example dataset you can share for testing?

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

No branches or pull requests

2 participants