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

add altimeter pressure correction #308

Merged
merged 2 commits into from
May 4, 2024

Conversation

justin4046
Copy link
Contributor

No description provided.

@wiedehopf
Copy link
Owner

I'm getting the formula a bit more precise, not that it matters.
Just wanted to also provide a reference as to which formula was used.

Your formula was off by 2ft at 10k ft and 6 ft at 40k ft.
Which is obviously irrelevant but wanted to check the code vs this reference: https://www.weather.gov/media/epz/wxcalc/pressureAltitude.pdf

@wiedehopf wiedehopf merged commit 1952368 into wiedehopf:master May 4, 2024
@wiedehopf
Copy link
Owner

See recent commits if you're curious about the changes i made to this PR.

@justin4046
Copy link
Contributor Author

Thanks. This was my source for station pressure. https://www.weather.gov/media/epz/wxcalc/stationPressure.pdf
Probably an unnecessary inhg conversion

@wiedehopf
Copy link
Owner

Think you took the inverse of the exponent from wikipedia.
And that exponent is a bit off on wikipedia for some reason.
And getting rid of conversions helps of course :)

@justin4046 justin4046 deleted the altimeter-correction branch May 5, 2024 22:44
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