aldro61 / mmit

Regression trees for interval censored output data
https://aldro61.github.io/mmit/
GNU General Public License v3.0
7 stars 7 forks source link

R package #19

Closed aldro61 closed 6 years ago

aldro61 commented 6 years ago

Should we move the R package to another repo for the GSoC project, or should we leave this inside the current repo? I'm not familiar with how R packages are made public, so your insight could be useful @tdhock.

If we plan on submitting the R package to a server (e.g., CRAN), then I think its fine that it stays inside the main repo.

aldro61 commented 6 years ago

Update: from what I see in the README, we could keep the R package here and install it using:

if(!require(devtools))install.package("devtools")
devtools::install_github("aldro61/mmit/Rpackage")

That's probably the way to go, since it keep everything in one place and we don't need to manage multiple copies of the solver code.

tdhock commented 6 years ago

I think either way is fine as long as it is possible to easily install the pkg from github via devtools. I would slightly prefer keeping it in one repo because it is maybe less confusing and easier to maintain

On Tue, Apr 24, 2018 at 8:34 AM, Alexandre Drouin notifications@github.com wrote:

Update: from what I see in the README, we could keep the R package here and install it using:

if(!require(devtools))install.package("devtools") devtools::install_github("aldro61/mmit/Rpackage")

That's probably the way to go, since it keep everything in one place and we don't need to manage multiple copies of the solver code.

— You are receiving this because you were assigned. Reply to this email directly, view it on GitHub https://github.com/aldro61/mmit/issues/19#issuecomment-383913830, or mute the thread https://github.com/notifications/unsubscribe-auth/AA478uBZChbBuylo52FWLRJtfXW4SQyWks5trxvkgaJpZM4Tgbtj .

aldro61 commented 6 years ago

I agree. Lets keep it this way.