Skip to content

Team 3 pull request#3

Open
pmaletic wants to merge 104 commits intoquantargo:masterfrom
pmaletic:master
Open

Team 3 pull request#3
pmaletic wants to merge 104 commits intoquantargo:masterfrom
pmaletic:master

Conversation

@pmaletic
Copy link

@pmaletic pmaletic commented May 9, 2019

No description provided.


for(col in names(data)){

if(sum(is.na(data[,col]))>0){
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would prefer any here

#' @export

model <- function(input_data,target_name) {
dt_model<- rpart::rpart(as.formula(paste(target_name," ~ .")), data = input_data)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why combine the model fitting with plot?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants