search

developmentseed / label-maker

Data Preparation for Satellite Machine Learning
http://devseed.com/label-maker/
MIT License
461 stars 110 forks source link

changes to generalize get_bounds #170

Closed martham93 closed 4 years ago

martham93 commented 4 years ago

to address issue #169

cc @drewbo

martham93 commented 4 years ago

thanks for the style suggestions, those should all be addressed now and I added a test.
I think the way I implemented the new test in a separate file that is pretty repetitive to what's in integration/test_classification_labels_geojson.py. I don't know if it would be better to add the test for the linestring geojson into the already existing polygon geojson test?

cc @drewbo