beckerbenj / beckerbenj/eatAnalysis

Use of 'sheet' and 'varColumn' in `copyScanSelection()`

Open
#1 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
R
Stars
0
Forks
0
PR merge metrics
No merged PRs in 30d

Description

The optional arguments 'sheet' and 'varColumn' in function `copyScanSelection()` aren't used consistently, but sometimes plainly ignored, e. g. in

- line 71 `readxl::read_excel(codebook, sheet = "Codebook", skip = startRow-1, col_types = "text")` or
- line 82 `length(setdiff(unique(codebook[rw,"Variable"]), ""))`

but perhaps also on other occasions.

Contributor guide

No contributing guide indexed for this repository

Research direction

Locate copyScanSelection() and inspect every use of sheet and varColumn, starting with the read_excel call around line 71 and the codebook access around line 82. Confirm the intended arguments from the function's callers, then verify that all relevant uses consistently honor them.

Written by the indexing model from the issue text.

Assessment

Tech stack
r
Domain
data
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
40/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.