Skip to content
This repository has been archived by the owner on Oct 28, 2022. It is now read-only.

geodata object and implementations #864

Merged
merged 3 commits into from
Mar 29, 2017
Merged

Conversation

mbaudis
Copy link
Member

@mbaudis mbaudis commented Mar 14, 2017

This is a go at the previously discussed geodata object. See e.g.

@mbaudis mbaudis mentioned this pull request Mar 14, 2017
@helenp
Copy link

helenp commented Mar 21, 2017

Summarising here from the notes document:
"As a pragmatic approach, the combination of lat,long + a (geonames.org mapped) location name seems the best compromise, with the addition of a “precision level” providing additional features (e.g. possibility to randomize point locations in a given boundary)."

@helenp
Copy link

helenp commented Mar 24, 2017

+1

@ejacox ejacox merged commit 146308f into metadata-integration Mar 29, 2017
@kozbo kozbo removed the in progress label Mar 29, 2017
@mbaudis
Copy link
Member Author

mbaudis commented Mar 29, 2017

@ejacox Thanks - great! Now, (also @kozbo, @david4096), how/when can we get this as a pick -> push_to_master? IMHO any time ...

@ejacox
Copy link
Contributor

ejacox commented Mar 31, 2017

I didn't realize you were working on a branch of master. I thought we were only forking from master (is that right @kozbo, @david4096?). Anyways, I will let @kozbo merge the branch since that will signify that we are no longer keeping the reference server in sync with the schemas.

@mbaudis
Copy link
Member Author

mbaudis commented Mar 31, 2017

The metadata-integration is thought as a staging branch for master. We had discussed this in Vancouver as a practical way to collect major "metadata" code updates, which however all are intended to be merged to master at some point. It is up to the schema integration team to select what & when is merged ...

So for me, changes in the metadata-integration are "forward looking" and can be used as basis for to add features to our test implementations.

@ejacox
Copy link
Contributor

ejacox commented Mar 31, 2017

Thank you for the clarification @mbauds.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants