Skip to content

Deprecate the options argument that is passed on to pROC#296

Merged
DavisVaughan merged 3 commits intomainfrom
feature/deprecate-options
Apr 16, 2022
Merged

Deprecate the options argument that is passed on to pROC#296
DavisVaughan merged 3 commits intomainfrom
feature/deprecate-options

Conversation

@DavisVaughan
Copy link
Copy Markdown
Member

This is in preparation of a switch to a custom roc-curve implementation that supports case weights.

It will no longer support any of the options passed on to pROC, so we have to deprecate that argument. It was present in:

  • roc_curve()
  • roc_auc()
  • roc_aunp()
  • roc_aunu()
  • metrics()

Also contains a little cleanup of the pr-curve internals. Extracted binary_threshold_curve() for use in roc-curve.

@DavisVaughan DavisVaughan merged commit 86b565b into main Apr 16, 2022
@DavisVaughan DavisVaughan deleted the feature/deprecate-options branch April 16, 2022 19:54
@github-actions
Copy link
Copy Markdown

github-actions bot commented May 1, 2022

This pull request has been automatically locked. If you believe you have found a related problem, please file a new issue (with a reprex: https://reprex.tidyverse.org) and link to this issue.

@github-actions github-actions bot locked and limited conversation to collaborators May 1, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant