visibility += int(num) / int(denom): ZeroDivisionError: division by zero

[wxbinge]

What went wrong?

22Z.TXT

Caught a bug in the 22Z.TXT metar file from the tgftp server. I have attached the file that is causing the error. I'm unable to open the metar file at all. It will last the entire hour until the next metar hour is produced.

Operating System

Linux

Version

1.2.0

Python Version

3.8

Code to Reproduce

import metpy
from metpy.io import metar
df_metar = metar.parse_metar_file('22Z.TXT')

Errors, Traceback, and Logs

Traceback (most recent call last):
  File "/home/wxbinge/fewx/scripts/radar/pyscripts/national_composite_archival.py", line 524, in <module>
    df_metar = metar.parse_metar_file('%s/%s' % (metar_dir,nt))
  File "/home/wxbinge/.local/lib/python3.8/site-packages/metpy/io/metar.py", line 381, in parse_metar_file
    return _metars_to_dataframe(filter(lambda m: len(m) > 25, full_metars(myfile)),
  File "/home/wxbinge/.local/lib/python3.8/site-packages/metpy/io/metar.py", line 442, in _metars_to_dataframe
    metars.append(parse_metar(metar, year=year, month=month))
  File "/home/wxbinge/.local/lib/python3.8/site-packages/metpy/io/metar.py", line 241, in parse_metar
    visibility += int(num) / int(denom)
ZeroDivisionError: division by zero

[akrherz]

The troublesome METAR in this file is:

KGYR 072147Z 12006KT 1/0SM FEW100 SCT250 41/14 A2992

I am unsure how best to account for this with the Metpy parser.

[wxbinge]

Oh, wow. Thanks for pointing that out.

Right now, I have a try/exception condition to bypass this issue, but that’s not what I want to do. Lol.

[dopplershift]

Thanks for the clear, reproducible report. Thanks to @akrherz, this will be fixed in the next release (1.4.0) that should be out in the next few weeks.