Skip to content

Commit

Permalink
fix km_h typo bug (#48)
Browse files Browse the repository at this point in the history
  • Loading branch information
MatthewFlamm committed Jul 1, 2020
1 parent dce838c commit 4b9636b
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 4 deletions.
2 changes: 1 addition & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@ NWS API does not expose all possible units for observations. Known units are co
|------------|----------------|------------|
|temperature | degF, degC | Celsius |
|pressure | Pa | Pascal |
|speed | m_s-1, km_hr-1 | km_hr-1 |
|speed | m_s-1, km_h-1 | km_h-1 |
|percent | percent | percent |
|angle | degree_(angle) | degrees |
|distance | m | meter |
2 changes: 1 addition & 1 deletion pynws/simple_nws.py
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,7 @@ def m_p_s_to_km_p_hr(m_p_s):
unit_conversion = {
"unit:degC": unchanged,
"unit:degF": f_to_c,
"unit:km_hr-1": unchanged,
"unit:km_h-1": unchanged,
"unit:m_s-1": m_p_s_to_km_p_hr,
"unit:m": unchanged,
"unit:Pa": unchanged,
Expand Down
4 changes: 2 additions & 2 deletions tests/fixtures/stations_observations_alternate_units.json
Original file line number Diff line number Diff line change
Expand Up @@ -83,12 +83,12 @@
},
"windSpeed": {
"value": 10,
"unitCode": "unit:km_hr-1",
"unitCode": "unit:km_h-1",
"qualityControl": "qc:V"
},
"windGust": {
"value": 10,
"unitCode": "unit:km_hr-1",
"unitCode": "unit:km_h-1",
"qualityControl": "qc:Z"
},
"barometricPressure": {
Expand Down

0 comments on commit 4b9636b

Please sign in to comment.