search

CityScope / CS_CityIO

Serverside script for the cityscope platform
https://cityio.media.mit.edu
14 stars 8 forks source link

json from Aalto is still using the old version #34

Closed popabczhang closed 6 years ago

popabczhang commented 6 years ago

@agrignard mentioned: "Speaking about Aalto how come they can produce a json that is still the old school version. I think that raise an issue on the GH side (see https://github.com/CityScope/CS_Rhino_Grasshopper_Template/issues/1). We have to be careful that any new comer is at least using our new template otherwise we loose the track"

popabczhang commented 6 years ago

@agrignard, the json composer and sender was updated to be compliant with the new json format at commit 493614d. I emailed the Aalto researchers for the update. image I will send a reminder email again for them to update to the latest version.

popabczhang commented 6 years ago

Reminder email sent. Will close this issue when the Aalto table compliant with the new JSON format.

yasushisakai commented 6 years ago

it seems it generally complies with the new format. If we were to be picky the header/block order is different from what we decided at #31. Yet, as long as the each cell element follows this order, I think it is fine.