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
Jointcal test for decam Tickets/dm 6624 #18
Conversation
@@ -159,7 +159,7 @@ def _build_ccdImage(self, dataRef, associations, jointcalControl): | |||
uRight = afwGeom.Point2I(lLeft.getX() + md.get("NAXIS1")-1, lLeft.getY() + md.get("NAXIS2")-1) |
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 know it's not edited on this PR, but noting that our style guide says there should be whitespace around the subtraction signs.
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.
Same for the modulus operator used for string formatting on lines 169, 220, and 297.
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.
Fixed.
self.all_visits = [176837, 176846] | ||
ccdnums = '^'.join(str(x) for x in range(10,19)) |
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.
Should there be whitespace after the comma in range?
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.
Correct. Fixed.
self.all_visits = [176837, 176846] | ||
ccdnums = '^'.join(str(x) for x in range(10,19)) | ||
self.other_args = ['ccdnum='+ccdnums,] |
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.
Whitespace after the comma (E231) and also around the + operator (according to the style guide).
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.
Fixed.
self.all_visits = [176837, 176846] | ||
ccdnums = '^'.join(str(x) for x in range(10,19)) | ||
self.other_args = ['ccdnum='+ccdnums,] | ||
|
||
@unittest.skipIf(data_dir is None, "validation_data_decam not setup") |
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.
Do you want this to say validation_data_jointcal not setup?
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.
Good catch! Yes.
Update to new decam testdata location. Cleanup pep8 formatting
f654e78
to
525d12a
Compare
Working version of a jointcal test for decam data, post-VisitInfo.