Skip to content
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

Adding functions that enable plotting the AUC and Calibration of multiple databases in one plot #458

Open
asaelmans opened this issue Jun 5, 2024 · 2 comments
Assignees

Comments

@asaelmans
Copy link

asaelmans commented Jun 5, 2024

Is your feature request related to a problem? Please describe.
When performing external validation of an existing model on multiple databases, I ran into a problem when wanting to plot several databases' calibration in one calibration plot.

Describe the solution you'd like
I would like two new functions: multiPlotCalibration and multiPlotROC. These functions should enable the plotting of the results of several databases into one.

Describe alternatives you've considered
Diving into the code of the PLP package and extracting the datapoints used for the plots by launching the individual database's results files.

Additional context
I added two examples of what these plots could look like in the future.

image

image

@rossdwilliams
Copy link
Contributor

Hi Alex,

this is good. I think we want to be able to pass multiple objects of plpResult into the function and then to produce these plots.

It would also be good to have a facet option per model or per database so that we have the flexibility to provide both options (the plots can get quite busy).

can you make a branch off of develop and work on this there?

@egillax
Copy link
Collaborator

egillax commented Jun 5, 2024

Thanks Alex for the issue.

How did you create the plots for your poster? Were you able to use the current plotting functions in the package or did you have to make one from scratch?

I think ideally you'd just have one flexible multiPlot function and it would use the underlying PLP functions for whatever it's plotting (supporting faceting). At least internally, then you could add multiPlotROC or whatever around that for the user. But if the current plotting functions are not flexible enough that's something we'd have to look at.

I also think for completeness in the future this should support at least:

  • plotSparseRoc
  • plotSparseCalibration
  • plotPrecisionRecall
  • plotSmoothCalibration

I'd also like net benefit but I see we don't have a plotting function for that, it's only in the shiny.

But feel free to just implement whatever suits you for your needs now and we can make follow up issues for the rest. We just need to make sure it would be compatible with extending it in the future.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants