Skip to content
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

New input and output format defined in GeoJson as Feature sets. #34

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jodymarca
Copy link
Contributor

You can use both input and output or just one of those. You can also select dynamically output using tag "format":"geojsonfeatures" in the request.
You can select at startup --geojsonfeature and --geojsonfeaturesinput (if you use server mode) or use them in a spark program in Hadoop ecosystem (I tried and used them in both versions)

@smattheis
Copy link

Please keep this pull request open for a while. I will merge (and revise) it as soon as possible. I just don't have the capacity to do it now. Thanks a lot for this pull request.

@jodymarca
Copy link
Contributor Author

Ok no problem
Regards
Jody

On Sat, Oct 29, 2016 at 12:41 PM, Sebastian Mattheis <
[email protected]> wrote:

Please keep this pull request open for a while. I will merge (and revise)
it as soon as possible. I just don't have the capacity to do it now. Thanks
a lot for this pull request.


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#34 (comment), or mute
the thread
https://github.com/notifications/unsubscribe-auth/AS1GJsgiDriobyOrOZuUyLUieYOGh5ypks5q4yLKgaJpZM4KVKNz
.

@smattheis
Copy link

I will check this pull request togehter with #39 once I have a strategy for #39. - ... to give a ping that it's not forgotten or ignored. ;)
I'm trying to keep a good test coverage in the project and I know that it's a little bit tricky with the output formats. To be able to do a "by-hand and look-with-eyes" test, can you briefly tell me what the output format must look like? - I also thought about a simple test that reads the output as GeoJSON with some library to at least test if it's valid GeoJSON. I would also adapt this kind of test strategy to the other output format(s) based on GeoJSON. Do you think that this could work and check the basic of your suggested output formatter?

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

Successfully merging this pull request may close these issues.

2 participants