Esri/terraformer

bbox included unnecessarily

timwis opened this issue · 4 comments

Hey guys, I'm using terraformer-arcgis-parser for my project, but I believe this issue to stem from this repo. I'm converting geoservice json geometry to geojson and it seems that it includes a bbox property every time, even just for points. For my use case this is unexpected and unnecessary. I would expect it to be an argument I could enable if I wanted it.

I believe the culprit is is on terraformer.js#L747 - when I disable that it seems to work as expected, without bbox.

Any ideas? Am I misunderstanding something perhaps?

EDIT: Oddly enough, when I use util.inspect() on the object, it doesn't include bbox even when it's there...$10 for whomever can explain that one!

Am I misunderstanding something perhaps?

it sure doesn't sound like it 😄. i tend to agree with you that it'd be helpful to make the inclusion of a bbox configurable.

we are planning on teasing out much of the functionality in Terraformer and repackaging in micro libraries so i'll take a look soon.

Sounds exciting! Thanks

still a work in progress, but please check out the new project:
https://github.com/Esri/arcgis-to-geojson-utils

i don't believe bboxs are created at all in the code there...

Awesome! Thanks @jgravois