Skip to content

Correcting unit (mb -> hPa) for atmospheric pressure according to WMO #197

Merged
akrherz merged 2 commits into
python-metar:mainfrom
radekdomin:pressure_unit_fix
Mar 16, 2026
Merged

Correcting unit (mb -> hPa) for atmospheric pressure according to WMO #197
akrherz merged 2 commits into
python-metar:mainfrom
radekdomin:pressure_unit_fix

Conversation

@radekdomin

Copy link
Copy Markdown

Correcting unit for atmospheric pressure. According to World Meteorological Organization, the standard unit for atmospheric pressure is hectopascal (hPa).

https://en.wikipedia.org/wiki/World_Meteorological_Organization#:~:text=Hectopascals%20(hPa)%20for%20atmospheric%20pressure.

@akrherz

akrherz commented Mar 6, 2026

Copy link
Copy Markdown
Collaborator

Thanks, I am lukewarm on this, but we at least need tests updated for this to be merged.

@radekdomin

Copy link
Copy Markdown
Author

My bad. Fixed. Thanks for fast response.

@akrherz akrherz left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I don't believe this breaks any users as nobody should be parsing the str repr :) Will await others here to review and merge if nobody else chimes in...

@phobson phobson left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This is fine for me as well.

@akrherz

akrherz commented Mar 16, 2026

Copy link
Copy Markdown
Collaborator

Thanks, I'm about to prepare a release...

@akrherz akrherz merged commit c76f1f4 into python-metar:main Mar 16, 2026
6 checks passed
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.

3 participants