Re: [Geotools-devel] ImageMosaic API refactor proposal

2016-06-09 Thread Devon Tucker 
Thanks for all the feedback. Like Jody said, we're probably going to break
the proposal apart a bit. I've started pulling apart the index/catalog
management into its own proposal:

https://github.com/geotools/geotools/wiki/Refactor-ImageMosaic-Index-Catalog-management-for-improved-extensibility

Still a work in progress. Will work on adding code examples for the catalog
manager part. In the meantime you can take a look at the WIP here:

https://github.com/dvntucker/geotools/tree/im_api_refactor

The biggest thing to note is that:

- A number of methods in CatalogManager have been changed from static to
non-static
- CatalogManager has been added as a field to the ImageMosaicConfigHandler
- Logic that was in ImageMosaicConfigHandler for checking ColorModel and
CRS has been moved to CatalogManager (see the second part of the proposal)

Cheers

On Wed, Jun 8, 2016 at 1:08 PM, Jody Garnett  wrote:

> Thanks for email feedback Andrea, Daniele. Had a good chat with Simone 
>
> Going to try and break this proposal into three small proposals on 1)
> index generation 2) dynamic processing 3) harvest flexibility.
>
> Some of the functionality (like data prep) is indeed already covered. I
> was not aware, for example, that geoserver importer could be used add
> granules to an *existing* mosaic (that is not functionality that can be
> done for vector import).
>
> --
> Jody Garnett
>
> On 5 April 2016 at 16:44, Devon Tucker  wrote:
>
>> Hi all,
>>
>> Just noticed that there wasn't a dedicated email thread for this proposal
>> and figured I'd kick one off since I've also been involved in it.
>>
>>
>> https://github.com/geotools/geotools/wiki/Refactor-ImageMosaic-for-extensibility
>>
>> I think the description gives a pretty good rationale for what we want to
>> accomplish with this proposal and I'd like to solicit any discussion,
>> advice, feedback, etc. from everyone.
>>
>> Cheers,
>> Devon
>>
>>
>> --
>>
>> ___
>> GeoTools-Devel mailing list
>> GeoTools-Devel@lists.sourceforge.net
>> https://lists.sourceforge.net/lists/listinfo/geotools-devel
>>
>>
>
--
What NetFlow Analyzer can do for you? Monitors network bandwidth and traffic
patterns at an interface-level. Reveals which users, apps, and protocols are 
consuming the most bandwidth. Provides multi-vendor support for NetFlow, 
J-Flow, sFlow and other flows. Make informed decisions using capacity 
planning reports. https://ad.doubleclick.net/ddm/clk/305295220;132659582;e___
GeoTools-Devel mailing list
GeoTools-Devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geotools-devel

Re: [Geotools-devel] Build failures in GeoTools-Master-OpenJDK8 (and GeoServer)

2016-06-09 Thread Jody Garnett 
I think there is a difference between openjdk and oracle jdk on font
metrics. You may consider using explicit bold fonts (rather than trust font
metric calculations).



--
Jody Garnett

On 9 June 2016 at 14:52, Ben Caradoc-Davies  wrote:

> Alessandro,
>
> you might have noticed some gt-render failures in
> GeoTools-Master-OpenJDK8 (and the downstream failures of
> GeoServer-Master-OpenJDK8 that might be caused by this).
>
> http://build.geo-solutions.it/jenkins/view/GeoTools/job/GeoTools-Master-OpenJDK8/
>
> junit.framework.AssertionFailedError: More labels in image2 than image1
> at junit.framework.Assert.fail(Assert.java:57)
> at junit.framework.Assert.assertTrue(Assert.java:22)
> at junit.framework.TestCase.assertTrue(TestCase.java:192)
> at
>
> org.geotools.renderer.lite.LetterConflictTest.testLetterConflictEnabledCurvedLine(LetterConflictTest.java:196)
>
> The build passes for me with Maven 3.3.9 and openjdk-8-jdk 8u91-b14-2
> amd64 on debian/sid:
>
> $ mvn -version
> Apache Maven 3.3.9 (bb52d8502b132ec0a5a3f4c09453c07478323dc5;
> 2015-11-11T05:41:47+13:00)
> Maven home: /home/ben/java/maven
> Java version: 1.8.0_91, vendor: Oracle Corporation
> Java home: /usr/lib/jvm/java-8-openjdk-amd64/jre
> Default locale: en_GB, platform encoding: UTF-8
> OS name: "linux", version: "4.6.0-1-amd64", arch: "amd64", family: "unix"
>
> One of the differences between Oracle JDK and OpenJDK is that OpenJDK
> ships with no fonts and makes use of system fonts. This has in the past
> broken some gt-render unit tests on systems with unusual font
> configuration. Have you tried running LetterConflictTest in interactive
> mode on build.geo-solutions.it?
>
> Kind regards,
>
> --
> Ben Caradoc-Davies 
> Director
> Transient Software Limited 
> New Zealand
>
>
> --
> What NetFlow Analyzer can do for you? Monitors network bandwidth and
> traffic
> patterns at an interface-level. Reveals which users, apps, and protocols
> are
> consuming the most bandwidth. Provides multi-vendor support for NetFlow,
> J-Flow, sFlow and other flows. Make informed decisions using capacity
> planning reports. https://ad.doubleclick.net/ddm/clk/305295220;132659582;e
> ___
> GeoTools-Devel mailing list
> GeoTools-Devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/geotools-devel
>
--
What NetFlow Analyzer can do for you? Monitors network bandwidth and traffic
patterns at an interface-level. Reveals which users, apps, and protocols are 
consuming the most bandwidth. Provides multi-vendor support for NetFlow, 
J-Flow, sFlow and other flows. Make informed decisions using capacity 
planning reports. https://ad.doubleclick.net/ddm/clk/305295220;132659582;e___
GeoTools-Devel mailing list
GeoTools-Devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geotools-devel

[Geotools-devel] Build failures in GeoTools-Master-OpenJDK8 (and GeoServer)

2016-06-09 Thread Ben Caradoc-Davies 
Alessandro,

you might have noticed some gt-render failures in 
GeoTools-Master-OpenJDK8 (and the downstream failures of 
GeoServer-Master-OpenJDK8 that might be caused by this).
http://build.geo-solutions.it/jenkins/view/GeoTools/job/GeoTools-Master-OpenJDK8/

junit.framework.AssertionFailedError: More labels in image2 than image1
at junit.framework.Assert.fail(Assert.java:57)
at junit.framework.Assert.assertTrue(Assert.java:22)
at junit.framework.TestCase.assertTrue(TestCase.java:192)
at 
org.geotools.renderer.lite.LetterConflictTest.testLetterConflictEnabledCurvedLine(LetterConflictTest.java:196)

The build passes for me with Maven 3.3.9 and openjdk-8-jdk 8u91-b14-2 
amd64 on debian/sid:

$ mvn -version
Apache Maven 3.3.9 (bb52d8502b132ec0a5a3f4c09453c07478323dc5; 
2015-11-11T05:41:47+13:00)
Maven home: /home/ben/java/maven
Java version: 1.8.0_91, vendor: Oracle Corporation
Java home: /usr/lib/jvm/java-8-openjdk-amd64/jre
Default locale: en_GB, platform encoding: UTF-8
OS name: "linux", version: "4.6.0-1-amd64", arch: "amd64", family: "unix"

One of the differences between Oracle JDK and OpenJDK is that OpenJDK 
ships with no fonts and makes use of system fonts. This has in the past 
broken some gt-render unit tests on systems with unusual font 
configuration. Have you tried running LetterConflictTest in interactive 
mode on build.geo-solutions.it?

Kind regards,

-- 
Ben Caradoc-Davies 
Director
Transient Software Limited 
New Zealand

--
What NetFlow Analyzer can do for you? Monitors network bandwidth and traffic
patterns at an interface-level. Reveals which users, apps, and protocols are 
consuming the most bandwidth. Provides multi-vendor support for NetFlow, 
J-Flow, sFlow and other flows. Make informed decisions using capacity 
planning reports. https://ad.doubleclick.net/ddm/clk/305295220;132659582;e
___
GeoTools-Devel mailing list
GeoTools-Devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geotools-devel

Re: [Geotools-devel] Proposal: ask OSGeo for legal advice on copyright headers

2016-06-09 Thread Jody Garnett 
Thanks Ben, the board has agreed to help (and have an open source law firm
they have worked with previously).

--
Jody Garnett

On 6 June 2016 at 12:54, Ben Caradoc-Davies  wrote:

> Jody,
>
> the vote of the committee is unanimous:
>
> Andrea Aime: +1
> Ben Caradoc-Davies: +1
> Christian Mueller: +1
> Ian Turton: +1
> Jody Garnett: +1
> Justin Deoliveira: +1
> Simone Giannecchini: +1
>
> Kind regards,
> Ben.
>
>
> On 01/06/16 07:56, Ben Caradoc-Davies wrote:
>
>> I propose that we ask OSGeo to: "Obtain legal advice on the need for and
>> form of source code copyright headers."
>>
>> So far we have three +1 votes but we need more. PMC?
>>
>> Kind regards,
>>
>>
> --
> Ben Caradoc-Davies 
> Director
> Transient Software Limited 
> New Zealand
>
--
What NetFlow Analyzer can do for you? Monitors network bandwidth and traffic
patterns at an interface-level. Reveals which users, apps, and protocols are 
consuming the most bandwidth. Provides multi-vendor support for NetFlow, 
J-Flow, sFlow and other flows. Make informed decisions using capacity 
planning reports. https://ad.doubleclick.net/ddm/clk/305295220;132659582;e___
GeoTools-Devel mailing list
GeoTools-Devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geotools-devel