Closed vincentvanhees closed 6 months ago
Thanks, I have been reflecting on this and think it is better to cancel this effort and instead expect GGIR users who want this to take care of sleep diary imputation outside GGIR.
Motivation: Accounting for weekday requires access to day of the week which the basic sleep log does not offer, which then forces us to retrieve this by matching with accelerometer data. This significantly increases code complexity. Further, I designed GGIR to intentionally exclude nights when sleep diary data is not complete, because there is no evidence that combining guiders (sleeplog and HDCZA) or imputing guiders truly provides meaningful estimates of sleep.
Adds option to impute sleeplog entries for onset and wakeup time with the median of the values on the remaining days. Fixes #1088
Checklist before merging:
inst/NEWS.Rd
with a user-readable summary. Please, include references to relevant issues or PR discussions.DESCRIPTION
file, if you think you made a significant contribution.If NEW GGIR parameter(s) were added then these NEW parameter(s) are :
man/GGIR.Rd
R/load_params.R
R/check_params.R
vignettes/GGIRParameters.Rmd
with references to the GGIR parts the parameter is used in.If GGIR parameter(s) were deprecated these parameter(s) are:
man/GGIR.Rd
R/load_params.R
R/check_params.R
vignettes/GGIRParameters.Rmd
R/extract_params.R
with deprecated parameters such that these do not produce warnings when found in old config.csv files.