openplantpathology / Mungbean_PM

A meta-analysis of mungbean powdery mildew control fungicide efficacy trials
https://openplantpathology.github.io/Mungbean_PM/
Creative Commons Zero v1.0 Universal
1 stars 0 forks source link

Create simplified data file #9

Closed adamhsparks closed 4 years ago

adamhsparks commented 4 years ago

@PaulMelloy, could you please create a simplified data file in .csv format for us to use that has the following columns and put it in the /data folder of the repository?

year (int) location (char) host_genotype (char) row_spacing.m (double) first_sign_disease (date) fungicide_ai (char) fungicide_application_1 (date) fungicide_application_2 (date) fungicide_application_3 (date) total_fungicide (int) grain_yield.t.ha (double) grain_yield_error (double) PM_final_severity (double) PM_final_severity_error (double)

In cases where there are more than one control due to a fungicide application not being made, please average those rows into one value.

@emdelponte, are there other columns you would like to have or other suggestions?

emdelponte commented 4 years ago

This looks fine, just need to create the fungicide timing variable later after loading the data. I have two suggestions to clean up the data even more.

1) create a "trial" variable: dat %>% unite(trial, year, location, host_genotype, rowspacing.m, sep = "", remove = F)

2) clean up the fungicide levels to dat %>% filter(fungicide_ai %in% c("control", "tebuconazole", "propiconazole"))

adamhsparks commented 4 years ago

Yes, I was thinking to add those via the analysis script. This file would just provide the basic data we need to work from. But we can certainly add them to this file.

adamhsparks commented 4 years ago

Colnames don't have to be exactly that if you're using others already. But I think those should indicate the data we'd like.

PaulMelloy commented 4 years ago

Hello, as requested the slim data has been coded in the script file R/slimming_PM_dat.R and the output file saved to data/slim_PM_dat.csv I did not add the fungicide timing variable as advised, we will do this in the Rmd file

adamhsparks commented 4 years ago

thanks, @PaulMelloy