search

ajayk0719 / jai-tools

Automatically exported from code.google.com/p/jai-tools
0 stars 0 forks source link

Add for NoData Ranges support #34

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
Actually, there is support for include/exclusion range representing ranges
of sample values which should be included in/excluded from the statistics
computation. It would be great to give also the capability to specify
noData ranges which represents invalid data which shouldn't be used to
compute statistics. We could also add a getter to know how many nodata
values have been found. Note that in such a context, actually there is only
NaN support.

Original issue reported on code.google.com by dany.Geo...@gmail.com on 12 Apr 2010 at 7:15

GoogleCodeExporter commented 9 years ago 
Thanks Daniele. Yes, I can think of cases where this would be useful.

Original comment by michael.bedward@gmail.com on 16 Apr 2010 at 6:33

GoogleCodeExporter commented 9 years ago 
The attached patch could add this capability.
Can you review it and apply if ok?

Original comment by dany.Geo...@gmail.com on 16 Apr 2010 at 7:08

Attachments:

GoogleCodeExporter commented 9 years ago 
I've made some minor modifications to the patch in the utils module and added 
some unit tests for the treatment 
of NoData values.  One of my tests is failing when I try to set a single value 
(point range) as NoData due to double 
precision errors in the Range class. I want to fix this before committing these 
changes and doing a 1.0.1 release.

Original comment by michael.bedward@gmail.com on 16 Apr 2010 at 9:03

GoogleCodeExporter commented 9 years ago 
I've applied the patch (with some minor changes) to trunk and 1.0.x and 
committed the changes. 

Daniele: can you please add a unit test for NoData ranges to ZonalStatsTest. 
Once that's there I'll do the 1.0.1 
release.

Original comment by michael.bedward@gmail.com on 16 Apr 2010 at 11:28

GoogleCodeExporter commented 9 years ago 
I'm on it. I'll ping you when committed.

Original comment by dany.Geo...@gmail.com on 16 Apr 2010 at 11:39

GoogleCodeExporter commented 9 years ago 
Committed on both 1.0.x and trunk.
Please, let me know

Original comment by dany.Geo...@gmail.com on 16 Apr 2010 at 12:11

GoogleCodeExporter commented 9 years ago 
done and included in version 1.0.1

Original comment by michael.bedward@gmail.com on 17 Apr 2010 at 1:35