Skip to content

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

Open
jodymarca wants to merge 1 commit into
bmwcarit:masterfrom
jodymarca:master
Open

New input and output format defined in GeoJson as Feature sets.#34
jodymarca wants to merge 1 commit into
bmwcarit:masterfrom
jodymarca:master

Conversation

@jodymarca

Copy link
Copy Markdown
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
Copy Markdown

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
Copy Markdown
Contributor Author

Ok no problem
Regards
Jody

On Sat, Oct 29, 2016 at 12:41 PM, Sebastian Mattheis <
notifications@github.com> 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
Copy Markdown

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