NSW-OEH-EMS-KST / grid-garage

ArcGIS python toolbox supporting table-based processing
4 stars 2 forks source link

raster - tweak values #260

Closed TomWaterat closed 5 years ago

TomWaterat commented 5 years ago

See error below. Also replicated in the the test branch "raster-tweak_values" in project raster-tweak_values_AGIS10-4.mxd. GG bug

byezy commented 5 years ago

@TomWaterat Hi Tom, I renamed the test repo to i260, but if you look at the repo after your last commit... it is empty. can you check your local repo and see if you forgot to commit before a push or something... push anything into test i260, i'm over the long names image

byezy commented 5 years ago

Hi Tom, I recreated a test. You might want to expand upon it sensibly. Problem was an errant line of debug code. i260/i260 cheers

TomWaterat commented 5 years ago

Hi Dave, Running ok now but found one issue (replicated in i260 in 'i260-10.4.mxd' this time using the same path names as you). When you specify a minimum value of 700 and to replace values under this value with 'NoData'... bug_TweakVal2

..the pixels with value=700 are also replaced with NoData (it should only replace values under this min value). The top grid is correct and was the result of selecting 'Minimum' in the 'Values under Minimum' input box instead of 'NoData'. bug_TweakVal

byezy commented 5 years ago

Yes for some reason the bounds were included, seems clearly wrong. not sure 1/why or 2/how it passed all previous tests...

Anyway changed... see how it suits

i260/i260

TomWaterat commented 5 years ago

Yep, @byezy ya fixed it.. merge back and close. Thanks Dave