Re: [Geotools-devel] Fwd: Can/should we activate the github squash commits on merge option?

2016-08-27 Thread Brad Hards
> Oooh... does that mean we also get rid of the blasted merge commit and > retain > a more linear history? :-) Indeed it does. https://github.com/codice/imaging-nitf/commits/master shows a couple of recent samples. I note that it modifies the commit message "headline" to include a ref to the p

[Geotools-devel] Build failed in Jenkins: geotools-master #1349

2016-08-27 Thread monitor
See -- [...truncated 19095 lines...] [INFO] ArcSDE plugin [INFO] ArcSDE dummy api [INFO] ArcSDE support classes [INFO] ArcSDE DataStore plugin [INFO] Dynamic symbolizer module based on JFreeCha

Re: [Geotools-devel] Fwd: Can/should we activate the github squash commits on merge option?

2016-08-27 Thread Andrea Aime
Oooh... does that mean we also get rid of the blasted merge commit and retain a more linear history? :-) Cheers Andrea Il 27 Ago 2016 22:00, "Frank Gasdorf" ha scritto: > At merge-time of a pull request it's possible to choose : > > [image: Inline-Bild 1] > > HTH > -- Forwarded message

[Geotools-devel] Build failed in Jenkins: geotools-master #1348

2016-08-27 Thread monitor
See -- [...truncated 20071 lines...] [INFO] [INFO] --- maven-resources-plugin:2.6:resources (default-resources) @ gt-epsg-hsql --- [INFO] Using 'UTF-8' encoding to copy filtered resources. [I

[Geotools-devel] Build failed in Jenkins: geotools-master #1347

2016-08-27 Thread monitor
See -- [...truncated 19747 lines...] [INFO] [INFO] [INFO] --- git-commit-id-plugin:2.1.2:revision (default) @ gt-sample

[Geotools-devel] Build failed in Jenkins: geotools-master #1346

2016-08-27 Thread monitor
See -- [...truncated 19188 lines...] [INFO] ArcSDE plugin [INFO] ArcSDE dummy api [INFO] ArcSDE support classes [INFO] ArcSDE DataStore plugin [INFO] Dynamic symbolizer module based on JFreeCha

[Geotools-devel] Build failed in Jenkins: geotools-master #1345

2016-08-27 Thread monitor
See -- [...truncated 19257 lines...] [INFO] ArcSDE plugin [INFO] ArcSDE dummy api [INFO] ArcSDE support classes [INFO] ArcSDE DataStore plugin [INFO] Dynamic symbolizer module based on JFreeCha

[Geotools-devel] Build failed in Jenkins: geotools-master #1344

2016-08-27 Thread monitor
See -- [...truncated 19215 lines...] [INFO] ArcSDE plugin [INFO] ArcSDE dummy api [INFO] ArcSDE support classes [INFO] ArcSDE DataStore plugin [INFO] Dynamic symbolizer module based on JFreeCha

[Geotools-devel] Build failed in Jenkins: geotools-master #1343

2016-08-27 Thread monitor
See -- [...truncated 29177 lines...] extracting: docs/_static/img/osgeo-logo.png extracting: docs/_static/img/quickstart.png inflating: docs/_static/img/geotools-logo.png extracting:

[Geotools-devel] Build failed in Jenkins: geotools-master #1342

2016-08-27 Thread monitor
See -- [...truncated 19276 lines...] [INFO] ArcSDE plugin [INFO] ArcSDE dummy api [INFO] ArcSDE support classes [INFO] ArcSDE DataStore plugin [INFO] Dynamic symbolizer module based on JFreeCha

[Geotools-devel] Build failed in Jenkins: geotools-master #1341

2016-08-27 Thread monitor
See -- [...truncated 19241 lines...] [INFO] ArcSDE plugin [INFO] ArcSDE dummy api [INFO] ArcSDE support classes [INFO] ArcSDE DataStore plugin [INFO] Dynamic symbolizer module based on JFreeCha

[Geotools-devel] Build failed in Jenkins: geotools-master #1340

2016-08-27 Thread monitor
See -- [...truncated 19248 lines...] [INFO] ArcSDE plugin [INFO] ArcSDE dummy api [INFO] ArcSDE support classes [INFO] ArcSDE DataStore plugin [INFO] Dynamic symbolizer module based on JFreeCha

[Geotools-devel] Build failed in Jenkins: geotools-master #1339

2016-08-27 Thread monitor
See -- [...truncated 19269 lines...] [INFO] ArcSDE plugin [INFO] ArcSDE dummy api [INFO] ArcSDE support classes [INFO] ArcSDE DataStore plugin [INFO] Dynamic symbolizer module based on JFreeCha

[Geotools-devel] Build failed in Jenkins: geotools-master #1338

2016-08-27 Thread monitor
See -- [...truncated 19239 lines...] [INFO] ArcSDE plugin [INFO] ArcSDE dummy api [INFO] ArcSDE support classes [INFO] ArcSDE DataStore plugin [INFO] Dynamic symbolizer module based on JFreeCha

[Geotools-devel] Build failed in Jenkins: geotools-master #1337

2016-08-27 Thread monitor
See -- [...truncated 19264 lines...] [INFO] ArcSDE plugin [INFO] ArcSDE dummy api [INFO] ArcSDE support classes [INFO] ArcSDE DataStore plugin [INFO] Dynamic symbolizer module based on JFreeCha

[Geotools-devel] Boundless repo problems?

2016-08-27 Thread Ben Caradoc-Davies
Chris, I see a flood of build failures that seem to be related the the Boundless repo: [ERROR] Failed to execute goal org.apache.maven.plugins:maven-deploy-plugin:2.7:deploy (default-deploy) on project geotools: Failed to retrieve remote metadata org.geotools:geotools:16-SNAPSHOT/maven-metada

[Geotools-devel] Build failed in Jenkins: geotools-master #1336

2016-08-27 Thread monitor
See -- [...truncated 19270 lines...] [INFO] ArcSDE plugin [INFO] ArcSDE dummy api [INFO] ArcSDE support classes [INFO] ArcSDE DataStore plugin [INFO] Dynamic symbolizer module based on JFreeCha

[Geotools-devel] Build failed in Jenkins: geotools-master #1335

2016-08-27 Thread monitor
See -- [...truncated 19166 lines...] [INFO] ArcSDE plugin [INFO] ArcSDE dummy api [INFO] ArcSDE support classes [INFO] ArcSDE DataStore plugin [INFO] Dynamic symbolizer module based on JFreeCha

[Geotools-devel] Build failed in Jenkins: geotools-master #1334

2016-08-27 Thread monitor
See -- [...truncated 19271 lines...] [INFO] ArcSDE plugin [INFO] ArcSDE dummy api [INFO] ArcSDE support classes [INFO] ArcSDE DataStore plugin [INFO] Dynamic symbolizer module based on JFreeCha

[Geotools-devel] Build failed in Jenkins: geotools-master #1333

2016-08-27 Thread monitor
See -- [...truncated 24967 lines...] [INFO] --- git-commit-id-plugin:2.1.2:revision (default) @ gt-swing --- [INFO] [INFO] --- maven-resources-plugin:2.6:resources (default-resources) @ gt-swi

Re: [Geotools-devel] How to use a variation of BufferedCoordinateOperationFactory implementation?

2016-08-27 Thread Felipe Castro
I think I know the answer. Just extend BufferedCoordinateOperationFactory and do something like this at the starts of the whole system: CoordinateOperationFactory myFactory = new MyCoordinateOperationFactory(); Hints.putSystemDefault(Hints.COORDINATE_OPERATION_FACTORY, myFactory

[Geotools-devel] Build failed in Jenkins: geotools-master #1332

2016-08-27 Thread monitor
See -- [...truncated 19272 lines...] [INFO] ArcSDE plugin [INFO] ArcSDE dummy api [INFO] ArcSDE support classes [INFO] ArcSDE DataStore plugin [INFO] Dynamic symbolizer module based on JFreeCha

[Geotools-devel] Build failed in Jenkins: geotools-master #1331

2016-08-27 Thread monitor
See -- [...truncated 19226 lines...] [INFO] ArcSDE plugin [INFO] ArcSDE dummy api [INFO] ArcSDE support classes [INFO] ArcSDE DataStore plugin [INFO] Dynamic symbolizer module based on JFreeCha

[Geotools-devel] Build failed in Jenkins: geotools-master #1330

2016-08-27 Thread monitor
See -- [...truncated 25665 lines...] [INFO] [INFO] --- maven-resources-plugin:2.6:resources (default-resources) @ gt-wps --- [INFO] Using 'UTF-8' encoding to copy filtered resources. [INFO] sk

[Geotools-devel] Build failed in Jenkins: geotools-master #1329

2016-08-27 Thread monitor
See -- [...truncated 19221 lines...] [INFO] ArcSDE plugin [INFO] ArcSDE dummy api [INFO] ArcSDE support classes [INFO] ArcSDE DataStore plugin [INFO] Dynamic symbolizer module based on JFreeCha

[Geotools-devel] Build failed in Jenkins: geotools-master #1328

2016-08-27 Thread monitor
See -- [...truncated 20849 lines...] [INFO] --- git-commit-id-plugin:2.1.2:revision (default) @ gt-property --- [INFO] [INFO] --- maven-resources-plugin:2.6:resources (default-resources) @ gt

[Geotools-devel] Build failed in Jenkins: geotools-master #1327

2016-08-27 Thread monitor
See -- [...truncated 19211 lines...] [INFO] ArcSDE plugin [INFO] ArcSDE dummy api [INFO] ArcSDE support classes [INFO] ArcSDE DataStore plugin [INFO] Dynamic symbolizer module based on JFreeCha

[Geotools-devel] Build failed in Jenkins: geotools-master #1326

2016-08-27 Thread monitor
See -- [...truncated 19272 lines...] [INFO] ArcSDE plugin [INFO] ArcSDE dummy api [INFO] ArcSDE support classes [INFO] ArcSDE DataStore plugin [INFO] Dynamic symbolizer module based on JFreeCha

[Geotools-devel] How to use a variation of BufferedCoordinateOperationFactory implementation?

2016-08-27 Thread Felipe Castro
Hello, I need to use a variation over BufferedCoordinateOperationFactory; I just need to extend the class and provide my implementation of a small method. However, I don't know how to register it so it gets used when going through StreamingRenderer.paint process. Any suggestion would be very u

[Geotools-devel] Build failed in Jenkins: geotools-master #1325

2016-08-27 Thread monitor
See -- [...truncated 29326 lines...] extracting: docs/_static/img/osgeo-logo.png extracting: docs/_static/img/quickstart.png inflating: docs/_static/img/geotools-logo.png extracting:

Re: [Geotools-devel] Memory leak when using StreamingRenderer with variations in WKT based CRS's

2016-08-27 Thread Felipe Castro
So here is what I have found so far. To correct for the hundreds of thousands of TheadLocal entries I just needed to do CRS. cleanupThreadLocals() at the end each request to my service. The case of the eternal instances of ObliqueStereographic and ObliqueOrthographic is because the pool of CRSP

[Geotools-devel] Fwd: Can/should we activate the github squash commits on merge option?

2016-08-27 Thread Frank Gasdorf
At merge-time of a pull request it's possible to choose : [image: Inline-Bild 1] HTH -- Forwarded message -- From: Frank Gasdorf Date: 2016-08-27 21:57 GMT+02:00 Subject: Re: [Geotools-devel] Can/should we activate the github squash commits on merge option? To: Ian Turton Ian,

Re: [Geotools-devel] GeoTools usage of public domain

2016-08-27 Thread Eric Engle
Thanks for the quick response all. > Some of those files are in the public domain because they were written by US Government employees. Okay, should be fine if those still public domain at the end, then. Let's focus on the others. > Others are specifically placed in the public domain as part of

Re: [Geotools-devel] Can/should we activate the github squash commits on merge option?

2016-08-27 Thread Frank Gasdorf
Here are my thoughts: Pro for using squashes: * it may be easier to revert commits and the timeline looks much consistant in a way that "feature" / "bugfix" commits are listed as once and are not listed when the steps (single commits) are listed when they happened * easier to merge "complete" feat

Re: [Geotools-devel] Can/should we activate the github squash commits on merge option?

2016-08-27 Thread Ian Turton
I think you choose at commit time if we want. Ian On 27 Aug 2016 21:41, "Andrea Aime" wrote: > I thought it was possible to choose at merge time if room squash or not. > Have I misread? > > Cheers > Andrea > > Il 27 Ago 2016 20:37, "Justin Deoliveira" ha scritto: > > I would be slightly hesita

Re: [Geotools-devel] Can/should we activate the github squash commits on merge option?

2016-08-27 Thread Andrea Aime
I thought it was possible to choose at merge time if room squash or not. Have I misread? Cheers Andrea Il 27 Ago 2016 20:37, "Justin Deoliveira" ha scritto: I would be slightly hesitant because it would mean you couldn’t have a pull request with multiple commits even when it makes sense? Maybe

Re: [Geotools-devel] Can/should we activate the github squash commits on merge option?

2016-08-27 Thread Justin Deoliveira
I would be slightly hesitant because it would mean you couldn’t have a pull request with multiple commits even when it makes sense? Maybe that is an ok thing to always have a single commit for a pull requests but I know I often like to break up commits in a change when it makes logical sense. For i

Re: [Geotools-devel] Memory leak when using StreamingRenderer with variations in WKT based CRS's

2016-08-27 Thread Felipe Castro
This is part of the memory map after rendering ~24K Orthographic and Stereographic projections: #instances #bytes class name - 2281117 72995744 java.lang.ThreadLocal$ThreadLocalMap$Entry 1963617854440 org.geotools.util.WeakHashSe

Re: [Geotools-devel] Can/should we activate the github squash commits on merge option?

2016-08-27 Thread Andrea Aime
I'd certainly be +1 on that, it would have avoided me quite some manual squashing when merging pull requests Cheers Andrea On Sat, Aug 27, 2016 at 6:01 PM, Ian Turton wrote: > Following this post https://github.com/blog/2141-squash-your-commits we > can set GitHub to offer the option of squash

Re: [Geotools-devel] Memory leak when using StreamingRenderer with variations in WKT based CRS's

2016-08-27 Thread Andrea Aime
Follow the instructions to directly subscribe to the list here: http://geotools.org/getinvolved.html Cheers Andrea On Sat, Aug 27, 2016 at 6:42 PM, Felipe Castro wrote: > Hi Andrea, thanks for posting ideas. > > How should I post my comments? > > > > -- > View this message in context: http://os

Re: [Geotools-devel] Memory leak when using StreamingRenderer with variations in WKT based CRS's

2016-08-27 Thread Felipe Castro
Hi Andrea, thanks for posting ideas. How should I post my comments? -- View this message in context: http://osgeo-org.1560.x6.nabble.com/Memory-leak-when-using-StreamingRenderer-with-variations-in-WKT-based-CRS-s-tp5282556p5282573.html Sent from the geotools-devel mailing list archive at Nabbl

[Geotools-devel] Can/should we activate the github squash commits on merge option?

2016-08-27 Thread Ian Turton
Following this post https://github.com/blog/2141-squash-your-commits we can set GitHub to offer the option of squashing merges on acceptance. It would save me fighting git every time I need to squash commits. Ian -- Ian Turton

Re: [Geotools-devel] Memory leak when using StreamingRenderer with variations in WKT based CRS's

2016-08-27 Thread Andrea Aime
Hi Felipe, you should not be posting through nabble, we don't get the images and part of the formatting on the actual mailing list if you do. I checked the image on Nabble for the time being (but mind, I won't do it next time), as far as I can see you have a lot of weak references, the first time t

Re: [Geotools-devel] Memory leak when using StreamingRenderer with variations in WKT based CRS's

2016-08-27 Thread Felipe Castro
This is part of a memory map after rendering ~24K Orthographic and Stereographic projections: -- View this message in context: http://osgeo-org.1560.x6.nabble.com/Memory-leak-when-using-StreamingRenderer-with-variations-in-WKT-based-CRS-s-tp5282556p5282566.html Sent from the geotools-devel ma

Re: [Geotools-devel] Memory leak when using StreamingRenderer with variations in WKT based CRS's

2016-08-27 Thread Felipe Castro
Another thing I noticed is a comment in BufferedCoordinateOperationFactory: The pool of cached transformations. This map can not be static, because the values may be different for the same ({@code sourceCRS}, {@code targetCRS}) pair dependending of hint values like {@link Hints#LENIENT

[Geotools-devel] [JIRA] (GEOT-5502) Implement Valdimir's Polygon label point algorithm

2016-08-27 Thread Ian Turton (JIRA)
Title: Message Title Ian Turton created an

[Geotools-devel] Memory leak when using StreamingRenderer with variations in WKT based CRS's

2016-08-27 Thread Felipe Castro
I would like some opinions about this. I am running a multi-threaded process to render thousands of maps using the Stereographic projection. Basically, I do the following for each new map: - Take a WKT definition for an Stereographic projection. - Modify latitude_of_origin and central_meridian.

Re: [Geotools-devel] GeoTools usage of public domain

2016-08-27 Thread Jody Garnett
I have no problem with making these BSD or wtfpl etc... they key point is we want everyone to be able to use these as cut and paste programming. -- Jody Garnett On 27 August 2016 at 09:40, Ian Turton wrote: > Some of those files are in the public domain because they were

Re: [Geotools-devel] GeoTools usage of public domain

2016-08-27 Thread Ian Turton
Some of those files are in the public domain because they were written by US Government employees as part of their work, so I think they really are in the public domain (as mentioned in your second link) and we can't (? shouldn't?) change that. Others are specifically placed in the public domain as