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-43561: Update to fgcm 3.10.6 with distutils removal and plotting bug fixes #42
Conversation
Remove distutils and fix image bugs
fgcm/fgcmZeropoints.py
Outdated
text = r'(%s)\n' % (filterName) | ||
text += r'%s\n' % (plotType) | ||
text += r"Blue: %s\n" % (self.blueString) | ||
text += r"Red: %s\n" % (self.redString) |
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.
Usual mix-n-match of single vs double quotes (and below).
@@ -450,7 +450,7 @@ def computeEpsilonPerCcd(self): | |||
# ignore bands that have a range > 10 | |||
deltaRange = scaleRange[:, 1] - scaleRange[:, 0] | |||
filtersToMatchRange, = np.where(deltaRange < 20.0) | |||
if filtersToMatchRange.size == 0: | |||
if filtersToMatchRange.size > 0: |
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.
Squash this into d8c6a71 (or is it too late?)
text += r"Red: %s\n" % (self.redString) | ||
text = "(%s)\n" % (filterName) | ||
text += "Blue: %s\n" % (self.blueString) | ||
text += "Red: %s\n" % (self.redString) |
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.
Oh…but again, squash?
Since these are all unchecked formatting bits in upstream code that has already been merged to fgcm master, I think the ship has sailed... |
No description provided.