minor modification in function spatPomp to allow input data.frame in tibble format

[Koohoko]

Hi developpers,

I have made 3 minor modifications in the code of spatPomp constructor, to allow input in tidyverse tibble format. Without these changes, the spread function here (

spatPomp/R/spatPomp.R

Line 221 in 6500790

pomp_data <- pomp_data %>% tidyr::spread(key = .data$obsname, value = .data$val)

) will not work (and results in error) because the ui column were all NAs from the previous step (

spatPomp/R/spatPomp.R

Line 218 in 6500790

pomp_data <- data %>% dplyr::mutate(ui = tmp[match(data[,unitname], names(tmp))])

).

This fix is simple and seems unlikely to affect other functionalities. I hope you could considering including this to the package as I guess many users may use the tidyverse toolkit while preprocessing.

Best,
Haogao

[ionides]

Thanks for the edits!
Ed

[ionides]

Hi Haogao, Thanks for this. I've pulled it into the spatPomp git repo and it will appear in due course on the CRAN version. Best, Ed. Edward L. Ionides Associate Chair for Undergraduate Studies and Professor, Department of Statistics, University of Michigan 1085 South University, Ann Arbor, MI 48109-1107 email: ***@***.*** phone: 734 615 3332 office: 453 West Hall

…

On Sat, Nov 4, 2023 at 10:18 AM Haogao Gu ***@***.***> wrote: Hi developpers, I have made 3 minor modifications in the code of spatPomp constructor, to allow input in tidyverse tibble format. Without these changes, the spread function here ( https://github.com/kidusasfaw/spatPomp/blob/650079052cb5dc6236d023cfe6179b162266d33f/R/spatPomp.R#L221C6-L221C6) will not work (and results in error) because the ui column were all NAs from the previous step ( https://github.com/kidusasfaw/spatPomp/blob/650079052cb5dc6236d023cfe6179b162266d33f/R/spatPomp.R#L218 ). This fix is simple and seems unlikely to affect other functionalities. I hope you could considering including this to the package as I guess many users may use the tidyverse toolkit while preprocessing. Best, Haogao ------------------------------ You can view, comment on, or merge this pull request online at: #19 Commit Summary - 0128378 <0128378> minor modification in function spatPomp to allow input data.frame in tibble format File Changes (1 file <https://github.com/kidusasfaw/spatPomp/pull/19/files>) - *M* R/spatPomp.R <https://github.com/kidusasfaw/spatPomp/pull/19/files#diff-a5f0cacf7a626e484dc98f0848bb5b8b6de30d488b437099cac4de0a25938a39> (6) Patch Links: - https://github.com/kidusasfaw/spatPomp/pull/19.patch - https://github.com/kidusasfaw/spatPomp/pull/19.diff — Reply to this email directly, view it on GitHub <#19>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ABKAFT6SAROYKOQGIMGFFTLYCZFERAVCNFSM6AAAAAA65TQ4H6VHI2DSMVQWIX3LMV43ASLTON2WKOZRHE3TOMZWGMYDKOA> . You are receiving this because you are subscribed to this thread.Message ID: ***@***.***>