Pecan: met2model.ED2 melewati tahun kabisat

Solusi
GFDL tidak memiliki tahun kabisat yang dibangun di dalamnya. Solusi pribadi saya adalah untuk memeriksa apakah bertemu adalah GFDL dan jika ya, perlakukan semua tahun kabisat seperti tahun non-kabisat. Ini bisa menggunakan pemeriksaan yang lebih umum untuk produk yang terpenuhi yang tidak memiliki tahun kabisat.

@mccabe , yang bekerja di sekitar tidak akan bekerja -- itu akan membuat Anda melalui alur kerja PEcAn bertemu tetapi model yang membutuhkan tahun kabisat masih akan kekurangan satu hari, dan dengan demikian akan crash. Saya pikir Anda benar-benar perlu menyalin dan menambahkan dalam satu hari ekstra

Saya setuju. Solusi saya benar-benar hackey, dan saya bermaksud untuk mengasingkannya di cabang pribadi untuk saat ini. ED tampaknya baik-baik saja untuk saat ini, jadi saya pikir saya akan kembali ke masalah ini setelah ESA dan menerapkan solusi nyata.

Saya pikir ini bukan bug met2model.ED2, tetapi karena GFDL tidak diproses dengan benar (dengan asumsi GFDL _memiliki_ tahun kabisat), tidak ada pemeriksaan untuk tahun kabisat dalam kode unduhan.GFDL, selalu 2920 nilai per tahun

Juga met2model.ED memproses tahun kabisat untuk AmerifluxLBL

Saya pikir kami telah sepakat sebelumnya bahwa masalahnya adalah dengan GFDL bertemu, pertanyaannya adalah apakah solusinya harus datang dalam unduhan GFDL, dalam met2model, atau menjadi sesuatu yang umum di antaranya. Bagaimanapun, hari kabisat ekstra itu perlu ditambahkan untuk model yang membutuhkannya.

mengerti, saya tidak yakin apakah GFDL memiliki tahun kabisat atau tidak (seharusnya membaca utas lebih hati-hati, bukan hanya judulnya :))

Banyak produk bertemu model (dan bahkan beberapa obs) melewatkan tahun kabisat, beberapa model iklim yang lebih tua bahkan menggunakan tahun 360 hari. Jadi, kita perlu memutuskan bahwa semua produk yang terpenuhi harus mengisi celah hari kabisat (misalnya, dengan mereplikasi 28 Feb dua kali) jika tidak disediakan, atau semua met2model harus melakukan langkah itu jika model memerlukannya. Karena ini tampaknya spesifik model (beberapa model tidak peduli), sepertinya met2model.

Masalah ini sudah basi karena telah dibuka selama 365 hari tanpa aktivitas.

Saya pikir ini sudah ditangani. Fungsi itu membutuhkan argumen leap_year yang mengontrol perilaku ini. Saya tidak yakin bagaimana itu bermain dengan GFDL bertemu.

https://github.com/PecanProject/pecan/blob/9ed21c954f64d24055d91cd8682cb7b2ddf97863/models/ed/R/met2model.ED2.R#L26 -L28