cedricngakou
commented
4 months ago The yield in the publication is the yearly average given by treatment and country. It's true that some of treatment doesn't have yield but after merge, the yield for those treatments have been set to empty (NA) and were just removed from the data.
It was also difficult for me to decide if this could be in fertilizer group or soil but I think it's more fertilizer group than soil.
Le dim. 7 juil. 2024, 19:13, Robert Hijmans @.***> a écrit :
I move this to pending. I am not sure what to do with this.
There is no treatment level yield data, so we cannot use this for the "fertilizer" group unless we also aggregated the treatments by country/crop and use the published average yields.
Alternatively, this could be added to the "soil" group, but note that most soil data are also averages from the publication. That would be OK for the "fertilizer" group, but not for the "soils" group.
— Reply to this email directly, view it on GitHub https://github.com/reagro/carob/pull/437#issuecomment-2212527973, or unsubscribe https://github.com/notifications/unsubscribe-auth/AVX3X7OL2KBQIZMELDXRC4LZLGALPAVCNFSM6AAAAABKOZE3RSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMJSGUZDOOJXGM . You are receiving this because you authored the thread.Message ID: @.***>
I move this to pending. I am not sure what to do with this.
There is no treatment level yield data, so we cannot use this for the "fertilizer" group unless we also aggregated the treatments by country/crop and use the published average yields.
Alternatively, this could be added to the "soil" group, but note that most soil data are also averages from the publication. That would be OK for the "fertilizer" group, but not for the "soils" group.