search

AgRoMeteorologiaINTA / agroclimatico

Conjunto de funciones para calcular índices y estadísticos climáticos hidrológicos a partir de datos tidy. Incluye una función para graficar resultados georeferenciados y e información cartográfica.
https://agrometeorologiainta.github.io/agroclimatico/
Other
17 stars 0 forks source link

scPDSI ya no está en CRAN #19

Closed paocorrales closed 3 years ago

paocorrales commented 3 years ago

Bajaron el paquete porque no resolvieron algunos problemas. Ahora este paquete está como dependencia de agromet. Si queremos mandar esto a CRAN vamos a tener que resolver como hacemos para esquivar este paquete.

yabellini commented 3 years ago

Ups...lo usabamos para calcular algún índice? Podríamos generar el cálculo nosotros?

eliocamp commented 3 years ago

Se usa para el índice de Palmer. Se basa en un modelo de balance de humedad que en el paquete scPDSI está en C++ y me da la impresión de que correrlo en R sería muy lento.

eliocamp commented 3 years ago

Pregunté en el github del paquete a ver qué onda. https://github.com/Sibada/scPDSI/issues/4 Sino, vemos si podemos forkear y arreglarlo nosotres.

eliocamp commented 3 years ago

Agregué las rutinas de C y todo eso a nuestro paquete y anda. @yabellini , ¿sabés cómo hay que hacer para dar crédito a les autores del otro paquete? ¿Te parece que esto está bien?

https://github.com/AgRoMeteorologiaINTA/agromet/blob/8ffafb9ed29a4cba4ae599c059daf955d0cce6f4/R/scpdsi-internal.R#L1-L11

yabellini commented 3 years ago

Buena pregunta. Esto como mínimo creo. Voy a consultar en algunos foros a ver que recomiendan.

yabellini commented 3 years ago

Comentarios que me dejaron luego de la consulta:

Some resources: https://r-pkgs.org/license.html#code-you-bundle is quite fitting for your use case. Licensing stuff as well as "how to include" (which covers giving credit) is discussed.

I also looked through the archive on R-package-devel to see relevant convos but this is less useful I think (apart from seeing this is a recurring challenge https://www.mail-archive.com/r-package-devel@r-project.org/msg03609.html you are doing solution 1 i.e. "vendoring". https://stat.ethz.ch/pipermail/r-package-devel/2021q2/007026.html which tells you should "check" but sadly not know how/where.

Lastly, have you checked whether the author watched their own GitHub repo? It might be worth emailing them (or simply tagging them in an issue, if you see they don't watch the repo but are active in other repos)?

eliocamp commented 3 years ago

Leyendo r-packages lo que tendríamos que hacer es agregar la información de la licencia en el archivo donde está el código que copiamos y agregar al autor del paquete como "copyright holder" en el DESCRIPTION.

Igual antes de eso podríamos tratar de contactar a Sibaba por otros medios; no parece que tenga mucha actividad en github.

yabellini commented 3 years ago

Me parece bien lo que propones de intentar contactarlo antes, no se me ocurre como.

eliocamp commented 3 years ago

Le voy a mandar un mail a la dirección que está en el DESCRIPTION: https://github.com/Sibada/scPDSI/blob/master/DESCRIPTION

eliocamp commented 3 years ago

No recibí respuesta, así que ya fue. Agregué al autor como copyright holder (de paso también al del spi).

yabellini commented 3 years ago

Perfecto Elio!!

eliocamp commented 3 years ago

Ya estaría en condiciones de mandar a CRAN.

yabellini commented 3 years ago

Hay que genial. Ahora nos ponemos de acuerdo en un día de esta semana asi lo hacemos :-)