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
