Thanks for the feedback everyone. I’ve updated the proposal as per all of
the feedback here, and submitted the pull request.
https://github.com/geoserver/geoserver/pull/2443
I’ll give all of that a bit of time to settle and then I’ll poke the list
for an official vote.
Thanks!
On Thu, Jun 29, 2017 at 8:16 AM Justin Deoliveira <[email protected]>
wrote:
> Thanks Jody, sounds like definitely consensus for the second option so
> I’ll update the proposal and code.
>
> Regarding making the format options more general… I imagine there is
> potential for this to apply to other formats but until we have something
> more concrete my gut tells me to keep this KML specific for now, and expand
> its scope when we have another format option that wants to do the same
> thing.
>
>
> On Wed, Jun 28, 2017 at 6:24 PM Jody Garnett <[email protected]>
> wrote:
>
>> I like the top-level format option.
>>
>> Is there any other output format that has to handle centroid? If so we
>> could replace
>> "kmcentroid_contain:true;kmcentroid_samples:10;kmcentroid_clip:true" with
>> "centroid_contain:true;centroid_samples:10;centroid_clip:true" (and then
>> documenting the kml format as supporting these options).
>>
>> --
>> Jody Garnett
>>
>> On 28 June 2017 at 07:34, Justin Deoliveira <[email protected]> wrote:
>>
>>> Hi folks,
>>>
>>> I did an initial sanity check on this change a few months back, and am
>>> now ready to push forward so I whipped up a proposal.
>>>
>>> https://github.com/geoserver/geoserver/wiki/GSIP-160
>>>
>>> Feedback appreciated.
>>>
>>> Once there is a concensus on the proposal I’ll update the current branch
>>> with documentation updates for KML, and then submit the pull request.
>>>
>>> Thanks!
>>>
>>> -Justin
>>>
>>>
>>> ------------------------------------------------------------------------------
>>> Check out the vibrant tech community on one of the world's most
>>> engaging tech sites, Slashdot.org! http://sdm.link/slashdot
>>> _______________________________________________
>>> Geoserver-devel mailing list
>>> [email protected]
>>> https://lists.sourceforge.net/lists/listinfo/geoserver-devel
>>>
>>>
------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
Geoserver-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/geoserver-devel
