Closed NPJuncal closed 1 year ago
Hola @Julenasti ya cambie el codigo para lapply. Podrias echarle un ojo cuando puedas?
para probarlo puedes usar el dataset ejemplo bluecarbon_data, pero tendrias que pasarlo primero por la funcion transform_om_oc para tener datos de carbono organico
:)
Qué bien Nerea!! Veo que lo has pillado enseguida 😃 El miércoles lo miro seguro y si antes tengo algún huequito antes
Hola Nerea,
Gran trabajo!! Te he hecho varios cambios y comentarios en la rama principal. Empieza con un pull request para trabajar sobre ello. Algunas sugerencias/dudas que me han surgido:
estimate_oc_stock
por ser consistente con lo anteriordplyr
por rbaseestimate_core
fuera como nos sugirió Paco (Paco puedes ver si esto (R/estimate_oc_stock.R) es lo que decÃas? o sino basandonos en tu ejemplo lo podemos modificar). He modificado los nombres de los parámetros de la función estimate_h
pero también estarÃa bien sacar las funciones de aquà fueraLa mayorÃa de los comentarios que figuran aquà ya los he incluido. Si tienes alguna duda lo puedes comparar con la versión anterior fácilmente en GitHub 😉
Gracias @Julenasti entre mañana y lunes me lo miro :)
Hola Nerea! Era para saber cómo vas con los progresos y si necesitas que te eche una mano con algo. Cuando puedas, no worries!
checkear que las muestras estan por orden
It checks that the samples are in order.
modificar los nombres de las columnas por los usuarios
modelos para predecir stock en cores cortos: log(massa accumulada)-profundidad (exp() despues!!)
usar predict para sacar el stock y el se de la prediccion