New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
DM-27547: stringify spatialCells #680
Conversation
python/lsst/afw/math/_spatialCell.py
Outdated
f"status={self.getStatus().name}, rating={self.getCandidateRating()}") | ||
|
||
|
||
SpatialCellCandidate.__repr__ = spatialCellCandidateRepr |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Might want to use lsst.utils.continueClass
for the content of this module.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I was just continuing the pattern that had been set up, but you're right. I'll rework it.
|
||
class SpatialCellImageCandidateTestCase(unittest.TestCase): | ||
|
||
def setUp(self): | ||
# To ensure consistency across tests: width/height are static members | ||
# of SpatialCellImageCandidate, and tests can run in any order. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
width/height are static members
😱 🙈
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I agree, but I'm not even thinking of attempting to fix that on this ticket.
Modify tests to behave properly with the class static height/weight members.
1280743
to
a568e97
Compare
No description provided.