search

CoVPN / correlates_reporting_usgcove_archive

Reproducible reporting workflows for the immune correlates statistical analyses of the Moderna and Janssen COVID-19 vaccine efficacy trials by the USG/COVE Response Biostatistics Team [Archived 16 October 2021]
GNU General Public License v3.0
6 stars 15 forks source link

decouple code for data processing and correlates analyses? #593

Closed youyifong closed 2 years ago

youyifong commented 2 years ago

@benkeser @nhejazi Two repos?

youyifong commented 2 years ago

moderna_real_params.Rdata need to be reconsidered. It is produced as part of the outcome from the data processing step.

youyifong commented 2 years ago

and data file with risk score needs to include placebo and baseline pos

nhejazi commented 2 years ago

@youyifong where did we leave off on this? There was quite a bit of discussion via email but I’ll have time coming to move on this, and would like to make sure that we have the right organization in place — even if we don’t make it in time for impending deadlines like 705

youyifong commented 2 years ago

@Nima Hejazi @.***> Hi Nima, we are at step h in the list "Decouple data process/risk score from correlates analyses" on https://fredhutch.sharepoint.com/:w:/r/sites/CoVPNCorrelatesTeam/_layouts/15/doc2.aspx?sourcedoc=%7B7CF9AF30-92E6-4ED6-931D-7AF814C293CA%7D&file=correlates%202.0%20design.docx&action=default&mobileredirect=true&wdhostclicktime=1631722909038&cid=7b50e89d-ded1-4b9f-8a7e-0554194da5ef

Let me know if it helps to talk.

On Sat, Oct 9, 2021 at 8:16 AM nima hejazi @.***> wrote:

@youyifong https://github.com/youyifong where did we leave off on this? There was quite a bit of discussion via email but I’ll have time coming to move on this, and would like to make sure that we have the right organization in place — even if we don’t make it in time for impending deadlines like 705

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/CoVPN/correlates_reporting/issues/593#issuecomment-939312889, or unsubscribe https://github.com/notifications/unsubscribe-auth/AHBH5LY24OFV7W2P5PFTEDTUGBMDBANCNFSM5FC6VJKQ . 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.

nhejazi commented 2 years ago

thanks @youyifong, I've started two new repos for the data processing and analysis, respectively. I'll work on their architecture this week; it may help to discuss later in the week, will ping you if so

youyifong commented 2 years ago

Thanks, Nima! Look forward to learning more about the changes. Would it be feasible/good idea to keep using the current repo as the correlates analysis repo and move the data processing components out to a new repo? Youyi

On Mon, Oct 11, 2021 at 2:41 PM nima hejazi @.***> wrote:

thanks @youyifong https://github.com/youyifong, I've started two new repos for the data processing and analysis, respectively. I'll work on their architecture this week; it may help to discuss later in the week, will ping you if so

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/CoVPN/correlates_reporting/issues/593#issuecomment-940462936, or unsubscribe https://github.com/notifications/unsubscribe-auth/AHBH5L5WSHQ5P7DTKE2SYTDUGNK2HANCNFSM5FC6VJKQ . 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.

nhejazi commented 2 years ago

Resolved in discussion/meetings.