reagro / carob

Aggregating agronomic data
GNU General Public License v3.0
20 stars 29 forks source link

New dataset (doi:10.21421/D2/FPFACD) #479

Closed cedricngakou closed 3 months ago

rhijmans commented 3 months ago

Thanks. It is good that you do these as it helps define the rules for the long-format (timerecs) data

cedricngakou commented 3 months ago

It's clearer now . Thanks for the clarification

On Tue, Aug 6, 2024 at 2:41 PM Robert Hijmans @.***> wrote:

Thanks. It is good that you do these as it helps define the rules for the long-format (timerecs) data

  • You should not (and there is no need to) combine variables in the long-format (for the two diseases)
  • Variables in the long-format should not also exist in the wide-format (d), except for record_id
  • Variables in the long-format should be in the vocabulary (this is currently not checked, but I will add that as soon as I can.)
  • Note that "75" is not a date. I assumed it is "days-after-planting" and I added variable "DAP" for that. But it is worrisome that you assigned these values to "date" as that is clearly wrong.

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