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-39745: Merge matched difference plot/metric classes #148
Conversation
da41bad
to
129520a
Compare
# Doesn't work; interpreted as float | ||
# atools.matchedRefCModelMagDiff.produce.plot.xLims: [parameters.mag_x_min, parameters.mag_x_max] |
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.
Does this need to be here for any particular historical reason?
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.
Sort of; it's a note that this doesn't work even though it should. In this case I ended up preferring to define lims_mag_x
in the python block. Anyway, I'm going to address this on a subsequent ticket (DM-41049).
optional=False, | ||
) | ||
|
||
def finalize(self): | ||
# Check if it has already been finalized | ||
if not hasattr(self.process.buildActions, "diff"): | ||
# Set before MagnitudeScatterPlot.finalize to undo PSF default |
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.
Missing period at the end of this comment, unless you intend for these two lines to be read as one sentence (I think not).
129520a
to
c72ff9a
Compare
c72ff9a
to
e113a1b
Compare
e113a1b
to
0e2c02b
Compare
No description provided.