Skip to content

Conversation

@kelly-sovacool
Copy link
Member

@kelly-sovacool kelly-sovacool commented May 29, 2023

Design discussion w/ @courtneyarmour:

  • this only matters when outcome is binary.
  • force users to supply a factor for outcome column when outcome is binary -- throw an error otherwise.
  • don't use pos_outcome argument - feature bloat.
  • make sure original factor levels preserved in calc perf metrics.

Issues

Change(s) made

Checklist

(Strikethrough any points that are not applicable.)

  • Write unit tests for any new functionality or bug fixes.
  • Update docs if there are any API changes:
    • roxygen comments
    • vignettes
  • Update NEWS.md if this includes any user-facing changes.
  • The check workflow succeeds on your most recent commit. This is always required before the PR can be merged.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants