CityScope / CS_Urban_Indicators

Scripts for getting urban data and building urban indicators. A module for computing urban indicators in response to real-time CityScope inputs.
GNU General Public License v3.0
7 stars 4 forks source link

Redundant repeating attr at every point #9

Closed RELNO closed 4 years ago

RELNO commented 4 years ago

in the context of https://github.com/CityScope/CS_cityscopeJS/issues/65,

Screen Shot 2020-03-27 at 20 55 08

^ these attr appear on every access cell. Instead, array of values [0.4, 0.1....] with a dictionary at the upper level (geometry:{}, attributes:{}...) will reduce json size quite dramatically.

doorleyr commented 4 years ago

Transferred the issue to this repo because accessibility is now integrated in the Urban_indicators repo as the proximity indicator. I've updated the proximity indicator as suggested by @RELNO above. The structure of the resulting geojson is now: { "type": "FeatureCollection", "properties": {"attributes": ["restaurants', "shopping", ...], "features": [ .... ] }

However, the properties seem to be stripped out of the geojson by the handler before being sent to cityIO. @crisjf is it possible to stop this from happening?

RELNO commented 4 years ago

@doorleyr cool thanks -- didn't know moving issues is a feature. Would make sense to therefore remove the old repo, I know HCU and others were looking into it and it might create some confusion

crisjf commented 4 years ago

See issue #8 that was reopened

crisjf commented 4 years ago

Will close this as we are fixing this in #8