So zip on Python 3 returns an iterator (zip object) on Python 3. So this broke reading NIDS products that rely on that behavior. I observed this when trying to plot storm track information from NIDS products.
Just need to look over all the uses of zip in the NIDS reader to make sure that when a list is needed (i.e. not when iterating over the results) we explicitly call list. Also would be a good time to start adding some tests that actually check the results of parsing. Not suggesting exhaustive tests, just making sure there's a test that catches each bad use of zip before we change the code.
So
zipon Python 3 returns an iterator (zip object) on Python 3. So this broke reading NIDS products that rely on that behavior. I observed this when trying to plot storm track information from NIDS products.Just need to look over all the uses of
zipin the NIDS reader to make sure that when a list is needed (i.e. not when iterating over the results) we explicitly calllist. Also would be a good time to start adding some tests that actually check the results of parsing. Not suggesting exhaustive tests, just making sure there's a test that catches each bad use of zip before we change the code.