Pecan: met2model.ED2 salta los años bisiestos

Solución alterna
GFDL no tiene años bisiestos incorporados. Mi solución personal es verificar si el cumplimiento es GFDL y, si lo es, tratar todos los años bisiestos como años no bisiestos. Esto podría usar una verificación más general para los productos met que no tienen años bisiestos.

@mccabe , ese trabajo

Estoy de acuerdo. Mi solución es completamente hackey, y tengo la intención de secuestrarla en una sucursal personal por el momento. ED parece estar bien por ahora, así que creo que volveré a este problema después de la ESA e implementaré una solución real.

Creo que esto no es un error met2model.ED2, pero se debe a que GFDL no se procesa correctamente (asumiendo que GFDL _ tiene_ años bisiestos), no hay verificación de años bisiestos en la descarga. Código GFDL, siempre son 2920 valores por año

También met2model.ED procesa años bisiestos para AmerifluxLBL

Creo que acordamos previamente que el problema era con el GFDL cumplido, la pregunta es si la solución debería venir dentro de la descarga de GFDL, dentro de met2model, o ser algo genérico en el medio. De cualquier manera, ese día bisiesto adicional debe agregarse para los modelos que lo necesiten.

Lo tengo, no estaba seguro de si GFDL tiene años bisiestos o no (debería haber leído el hilo con más atención, no solo el título :))

Muchos productos metálicos modelados (e incluso algunas observaciones) se saltan los años bisiestos, algunos modelos climáticos más antiguos incluso usaban un año de 360 ​​días. Por lo tanto, debemos decidir que todos los productos cumplidos deben completar los días bisiestos (por ejemplo, replicando el 28 de febrero dos veces) si no se proporciona, o todos los modelos met2 deben realizar ese paso si el modelo lo requiere. Dado que esto parece específico del modelo (a algunos modelos no les importa), parece met2model.

Este problema está obsoleto porque ha estado abierto los 365 días sin actividad.

Creo que esto se ha abordado. Esa función toma un argumento leap_year que controla este comportamiento. Sin embargo, no estoy seguro de cómo se encontró eso con GFDL.

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