FFTrees icon indicating copy to clipboard operation
FFTrees copied to clipboard

Add additional sub-plotting options

Open ndphillips opened this issue 8 years ago • 4 comments

in plot.FFTrees(), add options to only show ROC curve, and main tree with icons

ndphillips avatar Jul 14 '17 06:07 ndphillips

Need to restructure the layout of plot.FFTrees pretty heavily. It's a bit of a mess right now :)

ndphillips avatar Nov 05 '17 11:11 ndphillips

Started to re-structure plot.FFTrees(), but splitting into multiple parts (e.g., ROC curve as separate function) may facilitate maintenance.

When trying to call showcues() with data = "test" (e.g., plot(heart.fft, data = "test", what = "cues") for heart.fft with test data) I only get the Error: "There are no test statistics for this object."
Is this a bug or a feature?

hneth avatar Jul 26 '22 10:07 hneth

The release of FFTrees 1.7.0 has fixed the most obvious plotting issues, but the corresponding code is still pretty bulky and should be simplified further. Beyond additional cleanups, I'd advocate using more intuitive plotting options.

Issue: The interplay between what and stats seems redundant, as the only function of stats is to hide some options. Also, the default of what = "tree" is counter-intuitive, as the real default appears to be to plot 'all' or 'everything', whereas the "tree" intuitively seems to denote the tree diagram in the center.

Suggestion:

  • Change what options to a new default setting what = 'all', which enables all options.

  • By contrast, what = 'tree' only plots the tree diagram (i.e., replacing stats = FALSE).

This would leave the current defaults (i.e., showing everything, when nothing has been specified) intact, but allow for more nuanced selections, e.g., using what = 'tree' with additional options.

  • Deprecate stats by correcting user inputs of stats = FALSE to what = 'tree' instead.

hneth avatar Sep 01 '22 07:09 hneth

I'd consider the most immediate needs to be addressed in the recent release of FFTrees version 1.7.5 (on 2022-09-15).

However, as new ideas/needs may arise and plot.FFTrees() is still pretty clunky, I suggest keeping this issue open for now.

hneth avatar Sep 16 '22 07:09 hneth