The-Strategy-Unit / The-Strategy-Unit/nhp_inputs
Replace NEE chart with text presentation
Nobody has claimed this yet.
- Dominant language
- R
- Stars
- 2
- Forks
- 1
- Avg merge
- 15h 39m
- Merged PRs (30d)
- 6
Description
Follows #698. @DCEW confirmed that dev board preferred option 2:
De-emphasise [NEE] by removing the chart, maybe show the values somewhere in the box and link to the paper (reduces influence of the chart being above the slider).
We do something akin to this in tpma-explorer.
Contributor guide
No contributing guide indexed for this repository
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 by locating the NEE chart in the Shiny app, then read the analogous implementation in R/mod_plot_nee.R from tpma-explorer. Replace the chart presentation with the NEE values in the relevant box and a link to the paper, and verify that the values and link appear correctly without the chart.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- r
- Domain
- data-visualization, frontend
- Issue type
- Feature
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Quiet
- Clarity
- Mostly clear
- Newbie friendliness
- 58/100