comodin19 / BayesVarSel

BayesVarSel: R package to calculate Bayes factors, model choice and variable selection in linear models
8 stars 5 forks source link

A BAS-type vignette #1

Open gongardo opened 7 years ago

gongardo commented 7 years ago

BAS is one of the most important competitors to BayesVarSel and they have a vignette in the GitHub main page with the main functionalities. Guess this is not so difficult to create!

https://github.com/merliseclyde/BAS

carlosvergara commented 7 years ago

Ok @gongardo: let's work.

carlosvergara commented 7 years ago

Aunque se ha hecho un README.md que da el pego, quizá se podría invertir algo de tiempo en construir una segunda viñeta molona. ¿Qué os parece @gongardo @comodin19?

comodin19 commented 7 years ago

Creo que la idea de @gongardo era pasar el paper a viñeta... pero lo de la viñeta molona me mola!!!

Por cierto, @carlosvergara no importaba que el paquete estuviese en tu cuenta de GitHub... te lo iba a comentar, pero le he dado al link y lo he transferido directamente... fallos de principiante... sorry.

Que os parece si quedamos algún día de la semana que viene y pensamos que y como podemos hacer para mejorar el paquete y avanzar...

carlosvergara commented 7 years ago

Sí que importaba: vosotros sois los creadores y yo solo un pardillo que se ha tropezado con este proyecto, con lo que lo adecuado es que la fuente principal esté en una de vuestras cuentas.

Por mi parte, y dado que no tengo vuestro nivel en lo subyacente​ al contenido y funcionalidad del paquete, creo que podría encargarme de elaborar algunos tests, darle una pequeña limpieza al código haciendo las llamadas con la estructura paquete::funcion() y cambiar las dependencias de DEPENDS a IMPORTS en el archivo DESCRIPTION. Más adelante quizá se podría pasar las llamadas a C++ por Rcpp con lo que añadir nuevas funcionalidades y mantener las existentes sería más sencillo. Es decir:

carlosvergara commented 7 years ago

Y una última cosilla: que aparezca un gráfico al simular los coeficientes no me acaba de convencer. Creo que lo correcto sería separar el comportamiento, generando un gráfico al hacer una llamada tipo plot() sobre el objeto de salida que se guarde.