Skip to content

Commit 5260ef3

Browse files
committed
unify algorithm name and source file name, formatting fixes
1 parent 31aed14 commit 5260ef3

File tree

2 files changed

+35
-39
lines changed

2 files changed

+35
-39
lines changed

python/plugins/processing/algs/gdal/GdalOgrAlgorithmProvider.py

+2-2
Original file line numberDiff line numberDiff line change
@@ -65,7 +65,7 @@
6565
from GridNearest import GridNearest
6666
from GridDataMetrics import GridDataMetrics
6767
from gdaltindex import gdaltindex
68-
from GdalRasterCalc import GdalRasterCalc
68+
from gdalcalc import gdalcalc
6969

7070
from ogr2ogr import Ogr2Ogr
7171
from ogr2ogrclip import Ogr2OgrClip
@@ -123,7 +123,7 @@ def createAlgsList(self):
123123
sieve(), fillnodata(), ExtractProjection(), gdal2xyz(),
124124
hillshade(), slope(), aspect(), tri(), tpi(), roughness(),
125125
ColorRelief(), GridInvDist(), GridAverage(), GridNearest(),
126-
GridDataMetrics(), gdaltindex(), GdalRasterCalc(),
126+
GridDataMetrics(), gdaltindex(), gdalcalc(),
127127
# ----- OGR tools -----
128128
OgrInfo(), Ogr2Ogr(), Ogr2OgrClip(), Ogr2OgrClipExtent(),
129129
Ogr2OgrToPostGis(), Ogr2OgrToPostGisList(), OgrSql(),

python/plugins/processing/algs/gdal/GdalRasterCalc.py python/plugins/processing/algs/gdal/gdalcalc.py

+33-37
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22

33
"""
44
***************************************************************************
5-
GdalRasterCalc.py
5+
gdalcalc.py
66
---------------------
77
Date : Janaury 2015
88
Copyright : (C) 2015 by Giovanni Manghi
@@ -39,14 +39,14 @@
3939

4040
from processing.algs.gdal.GdalUtils import GdalUtils
4141

42-
class GdalRasterCalc(GdalAlgorithm):
42+
class gdalcalc(GdalAlgorithm):
4343

4444
INPUT_A = 'INPUT_A'
4545
INPUT_B = 'INPUT_B'
4646
INPUT_C = 'INPUT_C'
4747
INPUT_D = 'INPUT_D'
4848
INPUT_E = 'INPUT_E'
49-
INPUT_F = 'INPUT_F'
49+
INPUT_F = 'INPUT_F'
5050
BAND_A = 'BAND_A'
5151
BAND_B = 'BAND_B'
5252
BAND_C = 'BAND_C'
@@ -62,42 +62,40 @@ class GdalRasterCalc(GdalAlgorithm):
6262
#DEBUG = 'DEBUG'
6363

6464
def defineCharacteristics(self):
65-
self.name = 'GDAL Raster Calculator'
65+
self.name = 'Raster calculator'
6666
self.group = '[GDAL] Miscellaneous'
67-
self.addParameter(ParameterRaster(self.INPUT_A, self.tr('Input layer A'),
68-
False))
67+
self.addParameter(ParameterRaster(
68+
self.INPUT_A, self.tr('Input layer A'), False))
6969
self.addParameter(ParameterString(self.BAND_A,
7070
self.tr('Number of raster band for raster A'), '1', optional=True))
71-
self.addParameter(ParameterRaster(self.INPUT_B, self.tr('Input layer B'),
72-
True))
71+
self.addParameter(ParameterRaster(
72+
self.INPUT_B, self.tr('Input layer B'), True))
7373
self.addParameter(ParameterString(self.BAND_B,
7474
self.tr('Number of raster band for raster B'), '1', optional=True))
75-
self.addParameter(ParameterRaster(self.INPUT_C, self.tr('Input layer C'),
76-
True))
75+
self.addParameter(ParameterRaster(
76+
self.INPUT_C, self.tr('Input layer C'), True))
7777
self.addParameter(ParameterString(self.BAND_C,
7878
self.tr('Number of raster band for raster C'), '1', optional=True))
79-
self.addParameter(ParameterRaster(self.INPUT_D, self.tr('Input layer D'),
80-
True))
79+
self.addParameter(ParameterRaster(
80+
self.INPUT_D, self.tr('Input layer D'), True))
8181
self.addParameter(ParameterString(self.BAND_D,
8282
self.tr('Number of raster band for raster D'), '1', optional=True))
83-
self.addParameter(ParameterRaster(self.INPUT_E, self.tr('Input layer E'),
84-
True))
83+
self.addParameter(ParameterRaster(
84+
self.INPUT_E, self.tr('Input layer E'), True))
8585
self.addParameter(ParameterString(self.BAND_E,
8686
self.tr('Number of raster band for raster E'), '1', optional=True))
87-
self.addParameter(ParameterRaster(self.INPUT_F, self.tr('Input layer F'),
88-
True))
87+
self.addParameter(ParameterRaster(
88+
self.INPUT_F, self.tr('Input layer F'), True))
8989
self.addParameter(ParameterString(self.BAND_F,
9090
self.tr('Number of raster band for raster F'), '1', optional=True))
9191
self.addParameter(ParameterString(self.FORMULA,
9292
self.tr('Calculation in gdalnumeric syntax using +-/* or any numpy array functions (i.e. logical_and())'), 'A*2', optional=False))
9393
self.addParameter(ParameterString(self.NO_DATA,
94-
self.tr("Set output nodata value"),
95-
'-9999'))
94+
self.tr('Set output nodata value'), '-9999'))
9695
self.addParameter(ParameterSelection(self.RTYPE,
9796
self.tr('Output raster type'), self.TYPE, 5))
98-
#self.addParameter(ParameterBoolean(self.DEBUG,
99-
#self.tr('Print debugging information'),
100-
#False))
97+
#self.addParameter(ParameterBoolean(
98+
# self.DEBUG, self.tr('Print debugging information'), False))
10199
self.addParameter(ParameterString(self.EXTRA,
102100
self.tr('Additional creation parameters'), '', optional=True))
103101
self.addOutput(OutputRaster(self.OUTPUT, self.tr('Output layer')))
@@ -108,56 +106,54 @@ def processAlgorithm(self, progress):
108106
#debug = self.getParameterValue(self.DEBUG)
109107
formula = self.getParameterValue(self.FORMULA)
110108
noData = str(self.getParameterValue(self.NO_DATA))
111-
109+
112110
arguments = []
113111
arguments.append('--calc')
114-
arguments.append('"'+formula+'"')
115-
arguments.append('--format')
112+
arguments.append('"' + formula + '"')
113+
arguments.append('--format')
116114
arguments.append(GdalUtils.getFormatShortNameFromFilename(out))
117115
arguments.append('--type')
118116
arguments.append(self.TYPE[self.getParameterValue(self.RTYPE)])
119117
if len(noData) > 0:
120118
arguments.append('--NoDataValue')
121119
arguments.append(noData)
122120
if len(extra) > 0:
123-
arguments.append(extra)
124-
#if debug is True:
125-
#arguments.append('--debug')
121+
arguments.append(extra)
122+
#if debug:
123+
# arguments.append('--debug')
126124
arguments.append('-A')
127125
arguments.append(self.getParameterValue(self.INPUT_A))
128126
if self.getParameterValue(self.BAND_A):
129-
arguments.append('--A_band '+self.getParameterValue(self.BAND_A))
127+
arguments.append('--A_band ' + self.getParameterValue(self.BAND_A))
130128
if self.getParameterValue(self.INPUT_B):
131129
arguments.append('-B')
132130
arguments.append(self.getParameterValue(self.INPUT_B))
133131
if self.getParameterValue(self.BAND_B):
134-
arguments.append('--B_band '+self.getParameterValue(self.BAND_B))
132+
arguments.append('--B_band ' + self.getParameterValue(self.BAND_B))
135133
if self.getParameterValue(self.INPUT_C):
136134
arguments.append('-C')
137135
arguments.append(self.getParameterValue(self.INPUT_C))
138136
if self.getParameterValue(self.BAND_C):
139-
arguments.append('--C_band '+self.getParameterValue(self.BAND_C))
137+
arguments.append('--C_band ' + self.getParameterValue(self.BAND_C))
140138
if self.getParameterValue(self.INPUT_D):
141139
arguments.append('-D')
142140
arguments.append(self.getParameterValue(self.INPUT_D))
143141
if self.getParameterValue(self.BAND_D):
144-
arguments.append('--D_band '+self.getParameterValue(self.BAND_D))
142+
arguments.append('--D_band ' + self.getParameterValue(self.BAND_D))
145143
if self.getParameterValue(self.INPUT_E):
146144
arguments.append('-E')
147145
arguments.append(self.getParameterValue(self.INPUT_E))
148146
if self.getParameterValue(self.BAND_E):
149-
arguments.append('--E_band '+self.getParameterValue(self.BAND_E))
147+
arguments.append('--E_band ' + self.getParameterValue(self.BAND_E))
150148
if self.getParameterValue(self.INPUT_F):
151149
arguments.append('-F')
152150
arguments.append(self.getParameterValue(self.INPUT_F))
153151
if self.getParameterValue(self.BAND_F):
154-
arguments.append('--F_band '+self.getParameterValue(self.BAND_F))
152+
arguments.append('--F_band ' + self.getParameterValue(self.BAND_F))
155153
arguments.append('--outfile')
156154
arguments.append(out)
157155

158156
if isWindows():
159-
GdalUtils.runGdal(['gdal_calc',
160-
GdalUtils.escapeAndJoin(arguments)], progress)
157+
GdalUtils.runGdal(['gdal_calc', GdalUtils.escapeAndJoin(arguments)], progress)
161158
else:
162-
GdalUtils.runGdal(['gdal_calc.py',
163-
GdalUtils.escapeAndJoin(arguments)], progress)
159+
GdalUtils.runGdal(['gdal_calc.py', GdalUtils.escapeAndJoin(arguments)], progress)

0 commit comments

Comments
 (0)