CityScope / CS_CityIO

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

Multiple unknown servers [which is a GOOD thing] #33

Closed RELNO closed 6 years ago

RELNO commented 6 years ago
screen shot 2018-02-25 at 17 16 55
yasushisakai commented 6 years ago

on adding fields to the meta or the header, I think it is fine and good to have, but the incentive to add such info will be lower then fields like the grid. The question is whether we want it to be a required field or a optional or something like a convention. Should we have a enforcing mechanism something like the server rejecting posts without them? or the sever can verify and add a isVerified(bool)

other facts

agrignard commented 6 years ago

Several thing on this issue that I didn't take time to read carefully

RELNO commented 6 years ago

@yasushisakai @agrignard -- In my view, forcing some 'handshake' is counterproductive to the openness we wish to have. Instead, let's just add a header field that defaults with the following:

"header":{ "mapping":{ "type":{ "0":"RL", ... } }, "spatial":{ "nrows":20, "latitude":42.360357, "rotation":0, "physical_latitude":42.360357, "ncols":20, "physical_longitude":-71.087264, "longitude":-71.087264, "cellsize":10 }, "name":"virtual_table",

"owner":"NAME, TITLE, INSTITUTE"

"block":[ "type", "height", "rotation" ] },

So if users don't fill this in, they see this with every GET

yasushisakai commented 6 years ago

@agrignard your 2nd bullet should be a different issue.

@RELNO so is what you're are saying we have it in the wiki, but we don't validate?

popabczhang commented 6 years ago

Agree with @yasushisakai that 2nd bullet should be a different issue. Opening one now.

yasushisakai commented 6 years ago

added in both virtual_table and wiki

closing

agrignard commented 6 years ago

just a small detail

"institude" => institute or institution?

yasushisakai commented 6 years ago

'institute' ref

closing as for a5900e51d2468fcdaaf01ad5ffaacdba1dcc5930