Closed choldgraf closed 4 years ago
Merging #66 into master will not change coverage. The diff coverage is
n/a
.
@@ Coverage Diff @@
## master #66 +/- ##
======================================
Coverage 96.5% 96.5%
======================================
Files 10 10
Lines 486 486
======================================
Hits 469 469
Misses 17 17
Thanks for posting the PR -- I'll let you read the comments I left on the issue and see how you'd want to update the doc additions.
Looking at your comments, I think that these doc additions could be useful. This PR gives an example that shows off the pattern that you suggested (using 'display'
to store only the display information), as well as another example to show how to create a custom encoder that stores information about the plot itself. I think both examples would have helped me come to this conclusion more quickly. What do you think?
Hello 👋 per some of my experiences in #65 , I thought it would be helpful to add two examples to the documentation. This PR does the following:
Let me know if this is helpful, or if folks would like to see changes!
closes #65