reagro / carob

Aggregating agronomic data
GNU General Public License v3.0
18 stars 28 forks source link

Update for#500 #501

Closed cedricngakou closed 2 weeks ago

rhijmans commented 2 weeks ago

Thanks. I do not understand this

   d$crop_rotation <- ifelse(grepl("R-M", d$crop_sys), "rice",
                      ifelse(grepl("R-W-MB", d$crop_sys), "rice;mung bean",
                      ifelse(grepl("R-W", d$crop_sys), "rice", "rice"))) 

I would guess that M is maize and W is wheat?

cedricngakou commented 2 weeks ago

Yes, M- maize R- rice Mb- mung bean

Le mar. 27 août 2024, 12:14, Robert Hijmans @.***> a écrit :

Thanks. I do not understand this

d$crop_rotation <- ifelse(grepl("R-M", d$crop_sys), "rice", ifelse(grepl("R-W-MB", d$crop_sys), "rice;mung bean", ifelse(grepl("R-W", d$crop_sys), "rice", "rice")))

I would guess that M is maize and W is wheat?

— Reply to this email directly, view it on GitHub https://github.com/reagro/carob/pull/501#issuecomment-2312270297, or unsubscribe https://github.com/notifications/unsubscribe-auth/AVX3X7KANIJBEWUMDMS4MMLZTRNRJAVCNFSM6AAAAABNF2ZA5OVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMJSGI3TAMRZG4 . You are receiving this because you authored the thread.Message ID: @.***>

cedricngakou commented 2 weeks ago

And W- wheat

Le mar. 27 août 2024, 12:16, Cedric Ngakou Sotakoutsing < @.***> a écrit :

Yes, M- maize R- rice Mb- mung bean

Le mar. 27 août 2024, 12:14, Robert Hijmans @.***> a écrit :

Thanks. I do not understand this

d$crop_rotation <- ifelse(grepl("R-M", d$crop_sys), "rice", ifelse(grepl("R-W-MB", d$crop_sys), "rice;mung bean", ifelse(grepl("R-W", d$crop_sys), "rice", "rice")))

I would guess that M is maize and W is wheat?

— Reply to this email directly, view it on GitHub https://github.com/reagro/carob/pull/501#issuecomment-2312270297, or unsubscribe https://github.com/notifications/unsubscribe-auth/AVX3X7KANIJBEWUMDMS4MMLZTRNRJAVCNFSM6AAAAABNF2ZA5OVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMJSGI3TAMRZG4 . You are receiving this because you authored the thread.Message ID: @.***>

rhijmans commented 2 weeks ago

Also, the fertilizer price is supposed to be by product. It may be possible, or not, to figure that out from the data.

rhijmans commented 2 weeks ago

And, based on the above, can you fix the crop rotation to include all crops.