vector zonal stats: default param request

[mosesckim]

Noticed default is to fill NaNs with zero after aggregation; this might make it difficult to identify original NaNs if there are actual zeros in the aggregation.

https://github.com/thinkingmachines/geowrangler/blob/master/geowrangler/vector_zonal_stats.py#L205

Also, a suggestion would be, in the case fillna option is set to True, make the replacement value (0 in this case at the moment) a variable users can input (e.g. -1, etc.)

Thanks!
Moses

[jtmiclat]

This makes sense. Created an issue #64 to track the task!

[tm-kah-alforja]

Locking this conversation as we have already closed issue #64.