tidymodels / tidymodels/extratests

Case weights: test prediction

Open
#303 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

feature
Dominant language
R
Stars
23
Forks
0
Avg merge
3h 57m
Merged PRs (30d)
1

Description

Prompted by https://github.com/tidymodels/censored/issues/363, I've extended testing on case weights in censored to now also check that prediction works, not just fitting a model. This could be applied to other model/engines here as well.

Additional question: do we want to move those tests to the respective packages (i.e. parsnip and extensions)?

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by reviewing the existing case-weight prediction tests added for censored, then compare how related model and engine tests are organized in parsnip and its extensions. Determine which additional combinations need prediction coverage and whether the tests belong here or in the respective packages; done means the placement is decided and the relevant prediction checks are covered.

Written by the indexing model from the issue text.

Assessment

Tech stack
r
Domain
testing-qa
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Quiet
Clarity
Needs clarification
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.