-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Include building metadata in OBJ and COLLADA files #17
Comments
As a start, I've implemented the existing method that I used with the original Polygon City – this adds simple comments to the top of the obj file for each of the origin longitude/latitude and the elevation. For the collada files I've added longitude, latitude and elevation properties to the document structure. For example… obj:
collada:
Happy to change the structure and naming of these as I just went with something simple to start with. |
Sounds good. Maybe we could also include a stringified JSON blob to make it easier to parse all properties out at one time, but could also keep the more human- (and machine-) readable versions you have? |
(I mean for OBJ, COLLADA already supports reasonably structured data) |
That's a good idea – I've pushed an update that places a JSON string as the first line of an obj, and also includes the same string in a 'metadata' tag in the collada header.
|
This is a small nitpick but perhaps we could prefix the JSON with something like
To make it a bit more reliable parsing wise? Basically someone is going to have to look for a regex or make other format expectations to parse these, so the more precise we can be about it the better. |
It's easy enough to change – the reason it isn't prefixed at the moment is in my head imagined it was easier to know that the first line is always the JSON and to get to it you need to only remove the I'll change it for now and we can always change it back if we want. |
In addition to the GeoJSON files, let's include building properties in the OBJ and COLLADA files, to simplify usage (no need to request multiple files).
The text was updated successfully, but these errors were encountered: