search

ua-snap / mapventure

Frontend for maps built with GeoServer
3 stars 0 forks source link

Fixes #335, ensure all polygon/multipolygon fires have a sane marker … #408

Closed brucecrevensten closed 7 years ago

brucecrevensten commented 7 years ago

…position

The problem was that the code never attempted to identify a marker position if the fire geometry type was MultiPolygon. This change accounts for those geometries by "flattening" the array of polygons into a single polygon (not perfect, but works OK in practice!).

This change is Reviewable

cstephen commented 7 years ago

I looked at the current fires near the Canadian border and confirmed that this is placing an active fire icon where there wasn't one before:

Current v16 branch: image

fix_centroid branch: image