The hist() and survival_plot() (and also the new pie() in #355) duplicate functionality for file naming and saving - this should be pulled out into a separate function and the plot functions refactored to use it.
Also, when suppression is enabled and the data fails the threshold tests, they save a blank image - this seems pointless to me. @jim-smith is that intentional?
The
hist()andsurvival_plot()(and also the newpie()in #355) duplicate functionality for file naming and saving - this should be pulled out into a separate function and the plot functions refactored to use it.Also, when suppression is enabled and the data fails the threshold tests, they save a blank image - this seems pointless to me. @jim-smith is that intentional?