NSW-OEH-EMS-KST / grid-garage

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

raster - aggregate #76

Closed TomWaterat closed 7 years ago

TomWaterat commented 7 years ago

Dave, if you run the same tool twice.. should the second run add a counter (eg. 0,1 or 2) to the output file name.. and not over-write the old file?

When I ran tool twice.. with the same settings.. it caused the following error.. see MXD

File "C:\Data\gg3\grid-garage-3\base\log.py", line 145, in error_trap, yield, ERROR 000871: C:\Data\gg3\grid-garage-3-tests\raster-aggregate\aggregate\t02_demfill25m.tif: Unable to delete the output .

Note I specified the output workspace to be a folder.. so the output file type was TIF.

byezy commented 7 years ago

did u push that up? cant see that message in the fail table. ran the last gp action, all worked fine...???

byezy commented 7 years ago

I pushed my test up.

TomWaterat commented 7 years ago

Pushed new version up.. Update: When you specify a TIF as the output .. the second time you run the tool it over-rides the old version.. causes an error if you have loaded the old TIF into ArcMap

byezy commented 7 years ago

pull code and test and see what u think. the esri call was previously being painful when adding the extension. now it works but it doesn't give a number, I goes like x.tif -> x_tif.tif

TomWaterat commented 7 years ago

That method works for three runs.. the fourth one over-writes again.. Maybe just note in user guide?

Run 1 - demfill25m.tif Run 2 - demfill25m_tif.tif Run 3 - demfill25m_tif0.tif Run 4 - demfill25m_tif0.tif (Over-writes)

byezy commented 7 years ago

yes I think so. as you said recommended is to write to fgdb's until final format, so should not be much of an issue... strange tho!

TomWaterat commented 7 years ago

changed to 'add to manual'