Menu
▾
▴
Re: [Geotools-devel] GEOT-5555, a new property to be added
|
From: Ben Caradoc-D. <be...@tr...> - 2016-10-29 19:30:34
|
Cosmin, please refer to the developer guide: http://docs.geotools.org/latest/developer/index.html It includes a guide for using Git: http://docs.geotools.org/latest/developer/procedures/git.html The usual workflow is to make a local branch with your changes (squashed into a single commit for clarity), push this branch into your GitHub fork, and issue a GitHub pull request. GitHub will alert developers. See this guide for details: http://docs.geotools.org/latest/developer/procedures/pull_requests.html Please also see the project conventions for details on source code formatting. There is an Eclipse formatter configuration in the project source: http://docs.geotools.org/latest/developer/conventions/index.html http://docs.geotools.org/latest/developer/conventions/code/code.html It is normal to first target master (17-SNAPSHOT). Changes might later be backported to the stable (16.x) branch. Kind regards, Ben. On 30/10/16 00:35, Cosmin CIORANU wrote: > Salve Guys, > > First, > I'm new to geotools project, so I apologize, if by mistake, I might break > some protocols. > > Second, I whish to get down to the matter at hand. > I did issue a new functionality GEOT-5555. It is related to > gt-process-feature project, to gs:PointStacker in which points stacked in > cluster go outside of the region of interest. I do recognize in some > use-cases this behavior is needed but in my particular one is not. > > So I propose to be added a new property called *weightClusterPosition *which > will be calculated as an average between points staked under this point. > The default values is false in order not to break the current behavior. > > Any other thoughts on the matter ? > > More information here > https://osgeo-org.atlassian.net/browse/GEOT-5555 > > *Other info:* > The above functionality(as proposed) was in implemented on demo geoserver > project (on 14.5 geotools). The screens attached to issue were created > using that version > > Also, I have made the necessary codding updates on the 17 SNAPSHOT. I'm not > quite sure how to proceed to make them available on github (Quite new to > git also). > > > Cosmin > > > > ------------------------------------------------------------------------------ > The Command Line: Reinvented for Modern Developers > Did the resurgence of CLI tooling catch you by surprise? > Reconnect with the command line and become more productive. > Learn the new .NET and ASP.NET CLI. Get your free copy! > http://sdm.link/telerik > > > > _______________________________________________ > GeoTools-Devel mailing list > Geo...@li... > https://lists.sourceforge.net/lists/listinfo/geotools-devel > -- Ben Caradoc-Davies <be...@tr...> Director Transient Software Limited <http://transient.nz/> New Zealand |
