Closed aspina7 closed 1 year ago
All about lower case these days 😊
From: Alex Spina notifications@github.com Sent: 01 November 2020 07:38 To: R4EPI/sitrep sitrep@noreply.github.com Cc: Patrick Keating Patrick.Keating@london.msf.org; Mention mention@noreply.github.com Subject: Re: [R4EPI/sitrep] AFS_survey_template_update (#275)
CAUTION: This email originated from outside MSF. Do not click links, open attachments or provide information unless you recognise the sender and know that the content is safe..
@aspina7 commented on this pull request.
In inst/rmarkdown/templates/nutrition/skeleton/skeleton.Rmdhttps://github.com/R4EPI/sitrep/pull/275#discussion_r515587791:
@@ -956,7 +1020,7 @@ current date.
```{r save_cleaned_data}
-# rio::export(study_data_cleaned, str_glue("study_data_cleaned_{Sys.Date()}.xlsx"))
+# rio::export(study_data_cleaned, here::here("Data", str_glue("study_data_cleaned_{Sys.Date()}.xlsx")))
@pbkeating<https://github.com/pbkeating> to confirm folder naming structure - capitalised?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub<https://github.com/R4EPI/sitrep/pull/275#discussion_r515587791>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AE2NEQ52YYAEOITXWACPUSLSNUF4PANCNFSM4SDZNPHA>.
This PR is dependent on this epidict PR in order to function
pr no longer dependent on a branch - should build from main to finalize template
Let's jump on a call and I'll talk you through it
On Thu, 20 Jan 2022, 15:27 pbkeating, @.***> wrote:
@.**** commented on this pull request.
In inst/rmarkdown/templates/mortality/skeleton/skeleton.Rmd https://github.com/R4EPI/sitrep/pull/275#discussion_r788876281:
@@ -924,24 +1088,26 @@ for lack of consent. The reasons for no consent are shown below.
This chunk will create a table of dropped individuals by reason for no consent.
-Note that these reasons are spread over multiple columns, thus the proportion -needs to be of the total and missings need to be dropped.
```{r consent_reasons} I can't see what has been done here, but looks like you already have a solution? — Reply to this email directly, view it on GitHub <https://github.com/R4EPI/sitrep/pull/275#discussion_r788876281>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AEL7IJYSNIWTJRGGVUWLL33UXASWPANCNFSM4SDZNPHA> . Triage notifications on the go with GitHub Mobile for iOS <https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675> or Android <https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>. You are receiving this because you were mentioned.Message ID: ***@***.***>
merging for now
Updating the survey templates to be based of Kobo datasets - starting with mortality and vaccination. (Currently we dont have nutrition dictionary yet)
This PR is dependent on this epidict PR in order to function