Yes, my use case would require positive integer results only, so adding
that change is good.
I'll have a look at the patch.
On Wed, Jun 6, 2012 at 12:15 AM, Jody Garnett <[email protected]>wrote:
> Martin the Modulo function is here:
> http://jira.codehaus.org/browse/GEOT-4169
>
> For some reason I could not assign this issue to you to review :(
>
> I did a couple quick checks resulting in one change. I assume you want
> this to quickly categorize features into groups (example: 16%5=1) allowing
> you to split numbers into five even groups.
> However the Java % operator is willing to return negative numbers
> (-16%5=-1) and would not work for that use-case.
>
> The classical "modulo" definition implemented here is always positive
> (example: modulo(-16,5)=1 ).
> --
> Jody Garnett
>
>
--
Martin Davis
OpenGeo - http://opengeo.org
Expert service straight from the developers.
------------------------------------------------------------------------------
Live Security Virtual Conference
Exclusive live event will cover all the ways today's security and
threat landscape has changed and how IT managers can respond. Discussions
will include endpoint security, mobile security and the latest in malware
threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/
_______________________________________________
GeoTools-Devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/geotools-devel
