Re: [Geotools-devel] GeoTools doc build failing to deploy

2019-12-11 Thread Jody Garnett
Thanks, an integer value of 2 would work. And yes I should of done a pull request to force travis to run. -- Jody Garnett On Wed, 11 Dec 2019 at 16:51, Daniele Romagnoli < daniele.romagn...@geo-solutions.it> wrote: > Hi Jody, > my latest PR is still failing on travis. I had similar issue this su

Re: [Geotools-devel] GeoTools doc build failing to deploy

2019-12-11 Thread Andrea Aime
Hi, I would caution against using -j auto too, on my machine with 16 cores and 32GB I had a couple of cases where Sphinx used all the memory and caused a swap storm while building the GeoServer docs... guess sphinx uses a ton of memory, and I already had a large chunk used by other apps. Pretty mu

Re: [Geotools-devel] GeoTools doc build failing to deploy

2019-12-11 Thread Daniele Romagnoli
Hi Jody, my latest PR is still failing on travis. I had similar issue this summer when updating the GeoServer doc build instructions. I see your doc fix contains add for "-j auto" parameter The travis build is failing due to this: [exec] Error: -j option argument must be an integer. Looking on

[Geotools-devel] On the opportunity to allow MBStyles translation setting "partials=true"

2019-12-11 Thread Andrea Aime
Hi, I'm looking at a tiled map generated via MBTiles + MBSTyles and getting this output: [image: image.png] Notice the two cut labels, should have been "Piano di Mommio" and "SR439". This is happening because by default Mapbox Styles allow labels to cross tile borders: https://docs.mapbox.com/map

Re: [Geotools-devel] GeoTools doc build failing to deploy

2019-12-11 Thread Jody Garnett
It is fixed -- Jody Garnett On Wed, 11 Dec 2019 at 10:32, Jody Garnett wrote: > I am fixing now > -- > Jody Garnett > > > On Wed, 11 Dec 2019 at 10:30, Ian Turton wrote: > >> Passed all the tests on Travis! Which was why I thought the docs used a >> different set of code. >> >> Ian >> >> On We

Re: [Geotools-devel] GeoTools doc build failing to deploy

2019-12-11 Thread Jody Garnett
I am fixing now -- Jody Garnett On Wed, 11 Dec 2019 at 10:30, Ian Turton wrote: > Passed all the tests on Travis! Which was why I thought the docs used a > different set of code. > > Ian > > On Wed, 11 Dec 2019, 09:04 Andrea Aime, > wrote: > >> Hi, >> did the change go through a PR? The travis

Re: [Geotools-devel] GeoTools doc build failing to deploy

2019-12-11 Thread Ian Turton
Passed all the tests on Travis! Which was why I thought the docs used a different set of code. Ian On Wed, 11 Dec 2019, 09:04 Andrea Aime, wrote: > Hi, > did the change go through a PR? The travis build should fail when docs > fail to build > > Cheers > Andrea > > > On Wed, Dec 11, 2019 at 10:0

Re: [Geotools-devel] GeoTools doc build failing to deploy

2019-12-11 Thread Andrea Aime
Hi, did the change go through a PR? The travis build should fail when docs fail to build Cheers Andrea On Wed, Dec 11, 2019 at 10:01 AM Ian Turton wrote: > Sorry, I thought the docs were running off a separate copy of the code. > > Ian > > On Wed, 11 Dec 2019 at 08:23, Jody Garnett wrote: > >

Re: [Geotools-devel] GeoTools doc build failing to deploy

2019-12-11 Thread Ian Turton
Sorry, I thought the docs were running off a separate copy of the code. Ian On Wed, 11 Dec 2019 at 08:23, Jody Garnett wrote: > Looks to be: > > [exec] Warning, treated as error: > [exec] > /tmp/jenkins-e86d80c8/workspace/geotools-master-docs/docs/user/tutorial/datastore/strategy.rst

Re: [Geotools-devel] GeoTools doc build failing to deploy

2019-12-11 Thread Jody Garnett
Looks to be: [exec] Warning, treated as error: [exec] /tmp/jenkins-e86d80c8/workspace/geotools-master-docs/docs/user/tutorial/datastore/strategy.rst:142:start-after pattern not found: import com.csvreader.CsvWriter; I think Ian made some changes to CSV datastore (to use a supported li

Re: [Geotools-devel] GeoTools doc build failing to deploy

2019-12-11 Thread Jody Garnett
Having a look, we should version control this script. -- Jody Garnett On Tue, 10 Dec 2019 at 15:27, Andrea Aime wrote: > See: > https://build.geoserver.org/job/geotools-master-docs/2622/consoleFull > > Looks like the build has been failing for a while, and it's likely related > to the build scr