-
-
Notifications
You must be signed in to change notification settings - Fork 30
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
Develop additional metadata elements for OAM Catalog #55
Comments
Below is a recap of the future additions tracked in previous threads (https://github.com/hotosm/OpenAerialMap/tree/master/metadata, #31, #17) plus input received from the SPC and the PacDID project. Future additions from our last review
Input from SPC/PacDID
@cgiovando Additional metadata items for input before we select and finalize? Input on how we want to finalize? License and tag are candidates that we'll definitely want to include. |
Just wanted to update that we're planning to start work on this enhancement. Some aspects can't be developed until the list is finalized (form validation, for example), but generically scaffolding support for more metadata properties is something we can begin now. |
From our thinking, we're leaning towards not having UAV-specific metadata items in the spec. @mojodna Based on the mosaic work, is there anything we want to be thinking about for helping group items or connect between the Uploader and Catalog better? |
How would contributors input bands in the form? Would it be a set of checkboxes with some common ones? RE R G B NIR SWIR PAN? Or maybe Color Multispectral Panchromatic? |
@smit1678 - I'm going to need staging credentials and URLs for the catalog API in order to work on this feature next week. If anyone has access to these, could you get ahold of me on the Slack channel? As per the docs, the needed variables are:
|
To run this locally, you just need to set these yourself. Use your own AWS keys and then set the secret token. For our staging environment, you should have access through heroku.
…_____________________________
From: Nick Bumbarger <[email protected]>
Sent: Saturday, December 17, 2016 05:22
Subject: Re: [hotosm/OpenAerialMap] Develop additional metadata elements for OAM Catalog (#55)
To: hotosm/OpenAerialMap <[email protected]>
Cc: Nate Smith <[email protected]>, Mention <[email protected]>
@smit1678 - I'm going to need staging credentials and URLs for the catalog API in order to work on this feature next week. If anyone has access to these, could you get ahold of me on the Slack channel? As per the docs, the needed variables are:OAM_DEBUG - Debug mode true or false (default)AWS_SECRET_KEY_ID - AWS secret key id for reading OIN bucketsAWS_SECRET_ACCESS_KEY - AWS secret access key for reading OIN bucketsDBURI - MongoDB connection urlSECRET_TOKEN - The token used for post requests to /tms endpoint
cc @mojodna @danielfdsilva
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.
|
Thanks, @smit1678. I have it running. |
For the small-scale mosaicking (multiple images that are part of the same scene), we may want to add an optional "scene id" UUID (to group them together). That would also help people who've subdivided imagery and want to be able to associate it back together. The the large-scale mosaicking (everything), gsd and acquisition dates are probably the main inputs. Having "quality" or "priority" might be helpful, but that's incredibly subjective. |
@mojodna I think the scene concept is implicit in the upload form, because it allows the user to add a list of images within a dataset but also allows for adding multiple datasets. It is true, however, that images submitted in a group within a dataset are not distinguished in the database. Are you proposing that the images in each submitted dataset be automatically assigned a unique group ID? |
@smit1678 @mojodna I added support for |
In addition to adding a description of the new metadata properties to the API documentation, we also need to discuss where they should be exposed in the browser. For example, the license should probably be included in the image preview panel, but maybe we don't want to allow tags, considering that the user is currently able to attach an arbitrary number of them. |
Yes. The uploader is aware of relationships between images, but the rest of the toolchain isn't. I'm tempted to advocate for the inclusion of bands, but we can table that until/if we decide that RGB(A) imagery is overly limiting. We can also follow a philosophy of allowing common properties to emerge using well-known tags. This would require that metadata is mutable (in order to change them in response to consensus), which I'm not sure is the case right now. @cgiovando recommended looking through OGC Earth Observation Metadata profile of Observations & Measurements to see if there's anything we're missing and to keep an eye out for future harmonization. |
Further elaborating on this, the uploader should assign unique ids to each scene/dataset. Right now, the entire upload gets an id, as do the individual images, but there's no id that ties images to scenes within an upload. E.g.: https://upload-api.openaerialmap.org/uploads/58655b07f91c99bd00e9c7ab |
I started looking at the OGC EO profile, http://docs.opengeospatial.org/is/10-157r4/10-157r4.html, to get a sense of what/if we're missing. The spec covers much more than just optical and can be very mission/equipment specific. I've tried to capture the Optical earth observation product information needed here: https://docs.google.com/spreadsheets/d/1yhX1cTfpa75wSKCDtRJTJ3rViw6exenTMXWsb1gpnnc/edit?usp=sharing. Not mapped to OIN/OAM's current metadata spec. I think there will be ways for future harmonization. We seem to capture a subset of the information. Couple items that we are missing that are specific to Optical products:
|
Related to hotosm/oam-uploader-api#52 and looking at this further, it looks like one useful upgrade to the core OIN spec is
This differs from for upload IDsPerhaps we can look at how OGC does it by using something like:
|
While working on re-indexing the HOT OIN bucket, I realized that the OIN metadata JSON should include something that identifies it as such (along with a version number), similar to how TileJSON does it: {
"...": "...",
"tilejson": "2.1.0"
} (The indexer currently attempts to treat all JSON files present in a bucket as OIN metadata, which is no longer true for the HOT bucket (there's footprint GeoJSON + TileJSON for the tiler); I have a temporary workaround that'll show up in a PR shortly, but it checks for |
I'm using |
This skips JSON files that do not contain a `uuid` property. It would be better if there were an OIN-specific key/value to check for. Refs hotosm/OpenAerialMap#55
Ok, let's bring this home to a close. I started this in OIN: openimagerynetwork/oin-metadata-spec#14. @mojodna @nbumbarger @cgiovando These changes look right and agreed upon? Current OAM through Uploader:{
"uuid": "",
"title": "",
"projection": "",
"bbox": [],
"footprint": "",
"gsd": ,
"file_size": ,
"acquisition_start": "",
"acquisition_end": "",
"platform": "",
"provider": "",
"contact": "",
"properties": {
"sensor": "",
"thumbnail": "",
"tms": ""
}
} New additions proposed to OAM through Uploader:{
"oin": "", # OIN Version number. An update to OIN spec
"uploaded_at" : "", # date metdata uploaded into OIN. An update to OIN spec
"uuid": "",
"title": "",
"projection": "",
"bbox": [],
"footprint": "",
"gsd": ,
"file_size": ,
"acquisition_start": "",
"acquisition_end": "",
"platform": "",
"provider": "",
"contact": "",
"creation_date": "",
"properties": {
"sensor": "",
"thumbnail": "",
"license": "", # new addition, doesn't affect OIN spec
"tags": "", # new addition, doesn't affect OIN spec
"tms": "",
"wmts": "" # new addition, doesn't affect OIN spec
}
} |
👍 1.1 for the OIN version? |
From other OAM improvements, we're going to look to implement some additional metadata elements for PacDID.
Next action: itemize the elements we need to add.
Recap of current:
Current metadata tracked
The text was updated successfully, but these errors were encountered: