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-42741: Make focalPlaneGeometryPlot look a bit better #203
Conversation
ee5c772
to
72a6f39
Compare
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.
LGTM! Just a few questions for clarification.
@@ -343,6 +355,10 @@ def makePlot( | |||
fig : `matplotlib.figure.Figure` | |||
The resulting figure. | |||
""" | |||
|
|||
cmap = mkColormap(["midnightBlue", "lightcyan", "darkgreen"]) |
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.
Why are there 3 options here, and could we add "tableau-colorblind10"?
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.
The three work together to make the colormap, I do want to move all the plots over to a coloublind friendly set of palates, if these ones aren't already, but I would prefer to change all the plots at once.
valuesPlot = np.clip(values, vmin, vmax) | ||
|
||
patchCollection = PatchCollection( | ||
patches, edgecolor="white", cmap=cmap, linewidth=0.5, linestyle=(0, (0.5, 3)) |
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.
would it be too much to make these options (e.g., edgecolor, linewidth...) configurable too, or is it not worth it?
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 would prefer not to make all of these things configurable. I would like the plots produced to be uniform and not clutter up the code with too many config options.
72a6f39
to
cb410d2
Compare
cb410d2
to
b11cc3a
Compare
No description provided.