Move uwot to Suggests?
Open
Nobody has claimed this yet.
upkeep
- Dominant language
- R
- Stars
- 146
- Forks
- 23
- PR merge metrics
- No merged PRs in 30d
Description
Follow-up to https://github.com/tidymodels/embed/pull/225/files#r1703885441
uwot fails to install in the GHA for current oldrel-4. We should look into if we can enable that again by
- moving uwot to Suggests
- skipping installation of that package conditional on R version in the GHA yaml
- safeguarding the corresponding tests with
skip_if_installed()
This is based on https://github.com/tidymodels/censored/pull/322
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start with the linked pull request discussion, then inspect the GHA YAML and the tests that use uwot. Determine which dependency and R-version paths are involved; done means oldrel-4 can run the workflow and the affected tests are safely handled when uwot is unavailable.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- github-actions, r
- Domain
- ci-cd, testing
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100