Improving likelihood surface plots in documentation
Nobody has claimed this yet.
Assessment
- Difficulty
- 5/5
- Estimated time
- Over a week
- Newbie friendliness
- 35/100
- Issue type
- Feature
- Clarity
- Needs clarification
- Activity status
- Stale
- Tech stack
- r
- Domain
- data-visualization, documentation
Research direction
Start by locating compute_likelihood_surface and the documentation's likelihood surface plots. Compare the current use of log(R) versus R and inspect how filtering near the mode is handled. Done when the representation and filtering behavior are decided and the documentation plots reflect that decision.
Written by the indexing model from the issue text.
Description
- Revisit decision to use log(R) instead of R
- Contemplate putting the filtering step back in to grey out likelihoods not sufficiently close to the mode. (Rectangles around the mode include some truly horrendous likelihoods, especially for when both R and k are at their joint maximum.)
- Could perhaps add a convenience feature for automatic filtering in
compute_likelihood_surface?
- Could perhaps add a convenience feature for automatic filtering in
Shoutout to @swo for pointing this out.
- Dominant language
- R
- Stars
- 2
- Forks
- 3
- PR merge metrics
- No merged PRs in 30d
Getting set up
This project ships no dev container, Dockerfile or contributing guide, so setting up is up to you: start from its README, and see our first-contribution guide for the general steps.
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.
More from CDCgov/nbbp
-
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 63/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
-
Difficulty 1/5 Under an hour Newbie friendliness 72/100
-
Difficulty 1/5 1-3 hours Newbie friendliness 78/100
Similar issues
-
Difficulty 2/5 1-3 hours Newbie friendliness 65/100
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
tidymodels/censored#414 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 76/100
waldronlab/bugphyzz#256 ·
-
dbRemoveTable() fails on SQLite with 'near "TRUNCATE": syntax error' and leaves the table in placePossibly taken A pull request linked to this issue is open or already merged. Open
Difficulty 2/5 1-3 hours Newbie friendliness 70/100
OHDSI/DatabaseConnector#348 ·
-
bug
Difficulty 2/5 1-3 hours Newbie friendliness 84/100