-
Notifications
You must be signed in to change notification settings - Fork 6
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
plot utility to annotate figure with platform ID #143
Comments
I a have a draft of this function (we have to fix the nomenclature in the attributes though). It is somewhat hard to choose a universal right location if we have it as a True or False (ex. add_annotation=True as a parameter in all plotting functions) |
Yeah that's on me! A fault in the demo file. Will fix next week after the vocab/format meeting. I don't know how you've designed you draft, but for me this would make sense as a secondary function. Like: fig, ax = plots.plot_ts(ds)
fig = add_meta_to_fig(fig, ds, location='bottom right') |
Yeah that or within the function and it is True as default and can be turned off but then it is annoying to position if you do want it but just not in that exact corner |
Yeah, I think this works better as a secondary/wrapper function. That's why we return fig and ax after all! In my experience, once a function has more than ~ 4 arguments/kwargs it becomes real messy to maintain. So best to separate out the functionality |
I am less worried about whether it shows up in the right corner. I kind of see it as a useful badge for non-production figures, and assume that if anyone wants a figure for an actual publication, that they'll pull out the |
For each plotting function which generates its own
fig
, add some annotation outside the axes (top left? bottom right?) to print the platform ID from theds
input.For example, GMT has a feature to add a little time stamp to figures:
Could instead be the platform ID (logo optional).
The text was updated successfully, but these errors were encountered: