FlukeAndFeather / jese4sci-FP

Final Project track of the jese4sci short course
0 stars 3 forks source link

FP202 #21

Closed ggsun closed 2 years ago

ggsun commented 2 years ago
modular_workflow
FlukeAndFeather commented 2 years ago

Sounds like a plan! Make sure you put each function in R/ so they're easy to re-use. Question: how do you expect plot_histogram() and refine_plot() to fit together? Is the output of plot_histogram() the input to refine_plot()?

tomjhandley commented 2 years ago

We're working on developing each of the functions with an API from the documentation track to go with it to explain how each one combines with the others

sjeknic commented 2 years ago

I guess we were thinking that plot_histogram() would make just the default histogram plot, while refine_plot() would be used to add formatting (e.g. axis labels, colors, scale axes, etc.). So the output of plot_histogram() as well as any additional user options get passed to refine_plot(). This will hopefully allow us to easily change formatting for different plots.

FlukeAndFeather commented 2 years ago

Gotcha. Just a warning, adding elements/styles to base R plots can be really tricky, so I would suggest going with ggplot. You should also familiarize yourself with the differences between + and %+% (https://ggplot2.tidyverse.org/reference/gg-add.html)