Skip to content

Commit

Permalink
remove getEupsProductName method from CameraMapper subclasses
Browse files Browse the repository at this point in the history
Replaced by the packageName class variable and getPackageName() method.
  • Loading branch information
jhoblitt committed May 26, 2015
1 parent fc6a288 commit 55e5a03
Showing 1 changed file with 2 additions and 4 deletions.
6 changes: 2 additions & 4 deletions python/lsst/obs/test/testMapper.py
Original file line number Diff line number Diff line change
Expand Up @@ -31,6 +31,8 @@
__all__ = ["TestMapper"]

class TestMapper(CameraMapper):
packageName = 'obs_test'

def __init__(self, inputPolicy=None, **kwargs):
policyFile = pexPolicy.DefaultPolicyFile("obs_test", "testMapper.paf", "policy")

This comment has been minimized.

Copy link
@timj

timj May 26, 2015

Member

Consider using self.packageName here instead of duplicating "obs_test".

This comment has been minimized.

Copy link
@jhoblitt

jhoblitt May 26, 2015

Author Contributor

DRYed.

policy = pexPolicy.Policy(policyFile)
Expand Down Expand Up @@ -90,10 +92,6 @@ def bypass_ccdExposureId(self, datasetType, pythonType, location, dataId):
def bypass_ccdExposureId_bits(self, datasetType, pythonType, location, dataId):
return 41

@classmethod
def getEupsProductName(cls):
return "obs_test"

def validate(self, dataId):
visit = dataId.get("visit")
if visit is not None and not isinstance(visit, int):
Expand Down

0 comments on commit 55e5a03

Please sign in to comment.