Skip to content

Commit

Permalink
Ignore N807 on lines defining functions for classes
Browse files Browse the repository at this point in the history
  • Loading branch information
brianv0 committed Jul 9, 2018
1 parent f08c310 commit 72b2632
Show file tree
Hide file tree
Showing 8 changed files with 25 additions and 24 deletions.
2 changes: 1 addition & 1 deletion python/lsst/afw/cameraGeom/detector.py
Expand Up @@ -33,7 +33,7 @@
WAVEFRONT = DetectorType.WAVEFRONT


def __iter__(self):
def __iter__(self): # noqa: N807
return (self[i] for i in range(len(self)))


Expand Down
6 changes: 3 additions & 3 deletions python/lsst/afw/geom/ellipses/axes.py
Expand Up @@ -3,21 +3,21 @@
__all__ = [] # import this module only for its side effects


def __repr__(self):
def __repr__(self): # noqa: N807
return "Axes(a=%r, b=%r, theta=%r)" % (self.getA(), self.getB(), self.getTheta())


Axes.__repr__ = __repr__


def __str__(self):
def __str__(self): # noqa: N807
return "(a=%s, b=%s, theta=%s)" % (self.getA(), self.getB(), self.getTheta())


Axes.__str__ = __str__


def __reduce__(self):
def __reduce__(self): # noqa: N807
return (Axes, (self.getA(), self.getB(), self.getTheta()))


Expand Down
6 changes: 3 additions & 3 deletions python/lsst/afw/geom/ellipses/ellipse.py
Expand Up @@ -3,21 +3,21 @@
__all__ = [] # import this module only for its side effects


def __repr__(self):
def __repr__(self): # noqa: N807
return "Ellipse(%r, %r)" % (self.getCore(), self.getCenter())


Ellipse.__repr__ = __repr__


def __reduce__(self):
def __reduce__(self): # noqa: N807
return (Ellipse, (self.getCore(), self.getCenter()))


Ellipse.__reduce__ = __reduce__


def __str__(self):
def __str__(self): # noqa: N807
return "(%s, %s)" % (self.getCore(), self.getCenter())


Expand Down
6 changes: 3 additions & 3 deletions python/lsst/afw/geom/ellipses/quadrupole.py
Expand Up @@ -3,15 +3,15 @@
__all__ = [] # import this module only for its side effects


def __repr__(self):
def __repr__(self): # noqa: N807
return "Quadrupole(ixx=%r, iyy=%r, ixy=%r)" % (self.getIxx(), self.getIyy(), self.getIxy())


def __reduce__(self):
def __reduce__(self): # noqa: N807
return (Quadrupole, (self.getIxx(), self.getIyy(), self.getIxy()))


def __str__(self):
def __str__(self): # noqa: N807
return "(ixx=%s, iyy=%s, ixy=%s)" % (self.getIxx(), self.getIyy(), self.getIxy())


Expand Down
12 changes: 6 additions & 6 deletions python/lsst/afw/geom/polygon/polygon.py
Expand Up @@ -3,29 +3,29 @@
__all__ = [] # import this module only for its side effects


def __repr__(self):
def __repr__(self): # noqa: N807
return "%s(%s)" % (self.__class__.__name__, [p for p in self.getVertices()])


def __reduce__(self):
def __reduce__(self): # noqa: N807
return self.__class__, (self.getVertices(),)


def __iter__(self):
def __iter__(self): # noqa: N807
"""Iterator over vertices"""
vertices = self.getVertices()
return iter(vertices)


def __getitem__(self, i):
def __getitem__(self, i): # noqa: N807
return [pt for pt in self][i]


def __len__(self):
def __len__(self): # noqa: N807
return self.getNumEdges()


def __contains__(self, point):
def __contains__(self, point): # noqa: N807
"""point in polygon?"""
return self.contains(point)

Expand Down
5 changes: 3 additions & 2 deletions python/lsst/afw/image/slicing.py
Expand Up @@ -213,14 +213,14 @@ def clone(self):
return cls(self, True)
cls.clone = clone

def __getitem__(self, imageSlice):
def __getitem__(self, imageSlice): # noqa: N807
box, index, origin = interpretSliceArgs(imageSlice, self.getBBox)
if box is not None:
return self.subset(box, origin=origin)
return self._get(index, origin=origin)
cls.__getitem__ = __getitem__

def __setitem__(self, imageSlice, rhs):
def __setitem__(self, imageSlice, rhs): # noqa: N807
box, index, origin = interpretSliceArgs(imageSlice, self.getBBox)
if box is not None:
if self.assign(rhs, box, origin) is NotImplemented:
Expand All @@ -229,3 +229,4 @@ def __setitem__(self, imageSlice, rhs):
else:
self._set(index, origin=origin, value=rhs)
cls.__setitem__ = __setitem__

2 changes: 1 addition & 1 deletion python/lsst/afw/math/background.py
Expand Up @@ -3,7 +3,7 @@
__all__ = [] # import this module only for its side effects


def __reduce__(self):
def __reduce__(self): # noqa: N807
"""Pickling"""
return self.__class__, (self.getImageBBox(), self.getStatsImage())

Expand Down
10 changes: 5 additions & 5 deletions python/lsst/afw/table/match/matchContinued.py
Expand Up @@ -27,12 +27,12 @@
from .match import SimpleMatch, ReferenceMatch, SourceMatch


def __repr__(self):
def __repr__(self): # noqa: N807
return "Match(%s,\n %s,\n %g)" % \
(repr(self.first), repr(self.second), self.distance)


def __str__(self):
def __str__(self): # noqa: N807
def sourceRaDec(s):
if hasattr(s, "getRa") and hasattr(s, "getDec"):
return " RA,Dec=(%g,%g) deg" % (s.getRa().asDegrees(), s.getDec().asDegrees())
Expand All @@ -49,7 +49,7 @@ def sourceStr(s):
return "Match(%s, %s, dist %g)" % (sourceStr(self.first), sourceStr(self.second), self.distance,)


def __getitem__(self, i):
def __getitem__(self, i): # noqa: N807
"""Treat a Match as a tuple of length 3: (first, second, distance)"""
if i > 2 or i < -3:
raise IndexError(i)
Expand All @@ -63,7 +63,7 @@ def __getitem__(self, i):
return self.distance


def __setitem__(self, i, val):
def __setitem__(self, i, val): # noqa: N807
"""Treat a Match as a tuple of length 3: (first, second, distance)"""
if i > 2 or i < -3:
raise IndexError(i)
Expand All @@ -77,7 +77,7 @@ def __setitem__(self, i, val):
self.distance = val


def __len__(self):
def __len__(self): # noqa: N807
return 3


Expand Down

0 comments on commit 72b2632

Please sign in to comment.