search

conveyal / analysis-backend

Server component of Conveyal Analysis
http://conveyal.com/analysis
MIT License
23 stars 12 forks source link

fix AggregationAreaController (convert to WGS84) #239

Closed abyrd closed 5 years ago

abyrd commented 5 years ago

Previously it was streaming the raw, possibly projected geometries without even checking whether they were in WGS84 or not. It did not complain that they were not in WGS84, just said they were too big (since the units were meters or feet instead of degrees as expected).

This is one small part of #213 which we are applying as a bugfix.

codecov-io commented 5 years ago

Codecov Report

:exclamation: No coverage uploaded for pull request base (dev@dca754b). Click here to learn what that means. The diff coverage is 0%.

Impacted file tree graph

@@          Coverage Diff           @@
##             dev     #239   +/-   ##
======================================
  Coverage       ?   22.74%           
  Complexity     ?      105           
======================================
  Files          ?       63           
  Lines          ?     2467           
  Branches       ?      231           
======================================
  Hits           ?      561           
  Misses         ?     1870           
  Partials       ?       36
Impacted Files Coverage Δ Complexity Δ
...al/taui/controllers/AggregationAreaController.java 12.32% <0%> (ø) 2 <0> (?)

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update dca754b...ac3e1a8. Read the comment docs.