Skip to content
This repository was archived by the owner on May 2, 2025. It is now read-only.

Commit ace7ecd

Browse files
committed
Convert getTargetList to a classmethod
1 parent 17aeff3 commit ace7ecd

File tree

5 files changed

+9
-10
lines changed

5 files changed

+9
-10
lines changed

python/lsst/pipe/drivers/coaddDriver.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -64,8 +64,8 @@ def __init__(self, TaskClass, parsedCmd, doReturnResults=False):
6464
def makeTask(self, parsedCmd=None, args=None):
6565
return self.TaskClass(config=self.config, log=self.log, reuse=self.reuse)
6666

67-
@staticmethod
68-
def getTargetList(parsedCmd, **kwargs):
67+
@classmethod
68+
def getTargetList(cls, parsedCmd, **kwargs):
6969
"""!Get bare butler into Task
7070
7171
@param parsedCmd results of parsing command input

python/lsst/pipe/drivers/constructCalibs.py

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -350,8 +350,8 @@ def setDefaults(self):
350350

351351
class CalibTaskRunner(TaskRunner):
352352
"""Get parsed values into the CalibTask.run"""
353-
@staticmethod
354-
def getTargetList(parsedCmd, **kwargs):
353+
@classmethod
354+
def getTargetList(cls, parsedCmd, **kwargs):
355355
return [dict(expRefList=parsedCmd.id.refList, butler=parsedCmd.butler, calibId=parsedCmd.calibId)]
356356

357357
def __call__(self, args):
@@ -406,8 +406,7 @@ def __init__(self, *args, **kwargs):
406406
@classmethod
407407
def batchWallTime(cls, time, parsedCmd, numCores):
408408
numCcds = len(parsedCmd.butler.get("camera"))
409-
numExps = len(cls.RunnerClass.getTargetList(
410-
parsedCmd)[0]['expRefList'])
409+
numExps = len(cls.getTargetList(parsedCmd)[0]['expRefList'])
411410
numCycles = int(numCcds/float(numCores) + 0.5)
412411
return time*numExps*numCycles
413412

python/lsst/pipe/drivers/skyCorrection.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -92,7 +92,7 @@ def batchWallTime(cls, time, parsedCmd, numCores):
9292
numCores : `int`
9393
Number of cores.
9494
"""
95-
numTargets = len(cls.RunnerClass.getTargetList(parsedCmd))
95+
numTargets = len(cls.getTargetList(parsedCmd))
9696
return time*numTargets
9797

9898
def runDataRef(self, expRef):

python/lsst/pipe/drivers/utils.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -8,8 +8,8 @@
88

99
class ButlerTaskRunner(TaskRunner):
1010
"""Get a butler into the Task scripts"""
11-
@staticmethod
12-
def getTargetList(parsedCmd, **kwargs):
11+
@classmethod
12+
def getTargetList(cls, parsedCmd, **kwargs):
1313
"""Task.runDataRef should receive a butler in the kwargs"""
1414
return TaskRunner.getTargetList(parsedCmd, butler=parsedCmd.butler, **kwargs)
1515

python/lsst/pipe/drivers/visualizeVisit.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -103,7 +103,7 @@ def batchWallTime(cls, time, parsedCmd, numCores):
103103
numCores : `int`
104104
Number of cores.
105105
"""
106-
numTargets = len(cls.RunnerClass.getTargetList(parsedCmd))
106+
numTargets = len(cls.getTargetList(parsedCmd))
107107
return time*numTargets
108108

109109
def runDataRef(self, expRef):

0 commit comments

Comments
 (0)