search

locationtech-labs / geopyspark

GeoTrellis for PySpark
Other
179 stars 59 forks source link

get_point_values Will Now Return the Correct Number of Results for Temporal Layers #620

Closed jbouffard closed 6 years ago

jbouffard commented 6 years ago

This PR fixes a bug where get_point_values would only return the values for one instant in a temporal layer regardless of how many times were in the layer itself. Now, all times and their values are returned. Because of this, the return type of get_point_values for temporal layers is now different.

Before:


# return for lists
[(Point, (datetime.datetime, [float]))]

# return for dicts

{k: (Point, (datetime.datetime, [float]))}

After:


# return for lists
[(Point, [(datetime.datetime, [float])])]

# return for dicts
{k: (Point, [(datetime.datetime, [float])])}

This PR resolves #619