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
Enable plot replaying #1112
Enable plot replaying #1112
Conversation
This should be good to go now. |
} | ||
|
||
|
||
render <- reactive({ |
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.
Nit: I don't like verbs as names of reactive expressions. render()
looks like it's a directive to render the plot, when the whole point of it is to not be that. A noun would be better.
I've made the changes to address the feedback: |
When a plot is made in
renderPlot
, it records the display list withrecordPlot
. Later, if only the height or width has changed (and no other inputs), the plot can be replayed withreplayPlot
, which can give a speed boost, especially with ggplot2.