Re: [Geotools-devel] Approach to handling negative dates

2016-04-12 Thread Jody Garnett
You are module maintainer here so let's focus on approach. I am happy to write the results of this email thread up as a proposal if needed. I kind of like (2) especially we we can use the "bigdate" already recorded. We wrote this api for a situation like this and I would love to see it used. It a

[Geotools-devel] Approach to handling negative dates

2016-04-12 Thread Justin Deoliveira
Hey folks, I wanted to start a new thread on this topic to talk about implementation approach, since I think we can all agree that the current patch (which has been closed) needs to be redone, at least to some extent. In general I can think of two approaches to this: 1. Follow the path laid down

Re: [Geotools-devel] On-the-fly reprojection in ImageMosaic

2016-04-12 Thread Jody Garnett
> We have implemented something similar a few times in the past for > clients specific customizations so we have a pretty clear idea on what > to do. > We have an outline that does static gdal command line transformation on ingest (resulting in a duplication of data - but an opportunity to improve

Re: [Geotools-devel] On-the-fly reprojection in ImageMosaic

2016-04-12 Thread Devon Tucker
Hi Simone, Thanks for the input. I'll take a look at StreamingRenderer although according to Jody it may not be appropriate since we would like to do WCS output as well. Your advice for initially mosaicking by CRS is definitely well-received. Good idea. Cheers, Devon On Tue, Apr 12, 2016 at 11:

Re: [Geotools-devel] beta2 planning

2016-04-12 Thread Ben Caradoc-Davies
Any objections or can I merge PR 1168? Maciej has designed it as a new projection rather than changing the old one so backwards compatibility is preserved, and the only file changes are to hook up the test script: https://github.com/geotools/geotools/pull/1168 Kind regards, Ben. On 12/04/16 18:

Re: [Geotools-devel] proposal: refactor vector mask external footprint generation

2016-04-12 Thread Ben Caradoc-Davies
+1. Looks like a well-considered and beneficial change. On Tue, Apr 12, 2016 at 12:01 PM Jody Garnett wrote: > -- Ben Caradoc-Davies Director Transient Software Limited New Ze

Re: [Geotools-devel] proposal: refactor vector mask external footprint generation

2016-04-12 Thread Jody Garnett
Think we are good to go - thanks for the effort Daniele. On Tue, Apr 12, 2016 at 11:48 AM Justin Deoliveira wrote: > Looks good, added my +1 > > On Tue, Apr 12, 2016 at 12:01 PM Jody Garnett > wrote: > >> Added my +1 - we are standing at: >> >>- Andrea Aime: +1 >>- Ben Caradoc-Davies >>

Re: [Geotools-devel] jdbc negative dates

2016-04-12 Thread Justin Deoliveira
Great, thanks for the input Jonathon and David. I think I have enough to put together a proposal on how to improve the current patch. Going to start that in a new thread. On Tue, Apr 12, 2016 at 1:38 PM Jonathan Marino wrote: > Just to be clear from the MapStory end, to be honest, we don't reall

Re: [Geotools-devel] jdbc negative dates

2016-04-12 Thread Justin Deoliveira
Perfect, that is what I needed :) Ok, so having two types makes sense. One to be able to go back to time zero but also limited in terms of resolution to seconds, and one that doesn’t go all the way back but gives you down the millisecond. Or do you think that two types is overkill and we can get

Re: [Geotools-devel] proposal: refactor vector mask external footprint generation

2016-04-12 Thread Justin Deoliveira
Looks good, added my +1 On Tue, Apr 12, 2016 at 12:01 PM Jody Garnett wrote: > Added my +1 - we are standing at: > >- Andrea Aime: +1 >- Ben Caradoc-Davies >- Christian Mueller >- Ian Turton: +1 >- Justin Deoliveira >- Jody Garnett +1 >- Simone Giannecchini: +1 > > Be

Re: [Geotools-devel] jdbc negative dates

2016-04-12 Thread Justin Deoliveira
Thanks David. I was clear that native postgresql timestamp couldn’t do the job, but my question was whether bigint had a big enough value space for geologic time? If we interpret the value as milliseconds is it big enough? Or do we need to interpret as seconds in order to go back far enough? On Tu

Re: [Geotools-devel] proposal: refactor vector mask external footprint generation

2016-04-12 Thread Jody Garnett
Added my +1 - we are standing at: - Andrea Aime: +1 - Ben Caradoc-Davies - Christian Mueller - Ian Turton: +1 - Justin Deoliveira - Jody Garnett +1 - Simone Giannecchini: +1 Ben, Justin, Christian are you in position to respond (the proposal is here

Re: [Geotools-devel] On-the-fly reprojection in ImageMosaic

2016-04-12 Thread Simone Giannecchini
Ciao Devon, quick answer, andrea will provide more. We have implemented something similar a few times in the past for clients specific customizations so we have a pretty clear idea on what to do. First thing, GeoServer is already able to mosaic raster in different CRS but at an higher level, insid

Re: [Geotools-devel] beta2 planning

2016-04-12 Thread Torben Barsballe
Mark - I have reviewed and merged those two PR's. Torben On Tue, Apr 12, 2016 at 3:24 AM, Mark Prins wrote: > On 12-04-16 08:45, Jody Garnett wrote: > > I am looking to make a beta2 release this week, I notice Ben and Maciej > > working away on a pull request. > > > > Are there any other pull r

[Geotools-devel] On-the-fly reprojection in ImageMosaic

2016-04-12 Thread Devon Tucker
Hi all, We've been looking into doing on-the-fly reprojection in image mosaic to handle multiple projections dynamically (kinda similar to what QGIS can do). Some of the issues with this we've worked out already, but one thing I've been wondering is where might be an appropriate place in the code

Re: [Geotools-devel] jdbc negative dates

2016-04-12 Thread Justin Deoliveira
Thanks for the info guys, You’ll have to bear with me as you walk me through the nuances of representing time in this way. So in order to represent full geologic time do you need more than just a numeric value? Or is that you just need a numeric type with a bigger value space? On Tue, Apr 12, 201

Re: [Geotools-devel] jdbc negative dates

2016-04-12 Thread Justin Deoliveira
Thanks David, that helps a lot. So to summarize the ask is to be able to store dates as milliseconds from the epoch, and have the geotools datastore map that to a java date type . With the current patch that logic is triggered is by using a domain type of “bigdate” in postgres. And because of the

Re: [Geotools-devel] jdbc negative dates

2016-04-12 Thread Justin Deoliveira
This was my finding as well… couldn’t make up what was going on here… which makes me think there is perhaps more to this patch on the GeoNode side? Rather than flying blind has anyone tried to reach out to Ian Schneider, who afaik is the original author of the patch? If not I can give that a shot

Re: [Geotools-devel] beta2 planning

2016-04-12 Thread Mark Prins
On 12-04-16 08:45, Jody Garnett wrote: > I am looking to make a beta2 release this week, I notice Ben and Maciej > working away on a pull request. > > Are there any other pull requests that are a priority to get into the beta? Any chance for [GEOT-5388] (https://github.com/geotools/geotools/pull/

Re: [Geotools-devel] jdbc negative dates

2016-04-12 Thread Andrea Aime
On Tue, Apr 12, 2016 at 8:59 AM, Jody Garnett wrote: > Near as I can tell the point of this exercise is to provide a mapping for > bigdate: > > mappings.put("bigdate", XDate.class); > Err yeah but I'm still lost, I can't see any such type in postgresql, nor I can find one searching the inter

Re: [Geotools-devel] proposal: refactor vector mask external footprint generation

2016-04-12 Thread Daniele Romagnoli
Hi, We have 3 positive and NO negative. Could I consider the proposal accepted and proceed with the merge? Please, let me know. Cheers, Daniele On Fri, Apr 8, 2016 at 1:32 PM, Ian Turton wrote: > Sorry I've been busy but it looks good to me +1 > > Ian > > On 7 April 2016 at 17:04, Daniele Roma

Re: [Geotools-devel] jdbc negative dates

2016-04-12 Thread Jody Garnett
> Right now the project is running a fork of geotools to have the following >> fix (from #997 ): >> >> Date converted = null; >> if (target == PostGISDialect.XDate.class) { >> converted = Converters.convert(expression.eva