Skip to content

Sydney Harris Exercise10 submission#3

Open
sydneyharris wants to merge 2 commits into
qtran4:mainfrom
sydneyharris:main
Open

Sydney Harris Exercise10 submission#3
sydneyharris wants to merge 2 commits into
qtran4:mainfrom
sydneyharris:main

Conversation

@sydneyharris
Copy link
Copy Markdown

No description provided.

Comment thread Exercise10.R

#creates 2 columns in UW_MSU called UW_Total and MSU_Total with the cumulative sum of each team when a basket is scored regardless of team
UW_MSU$UW_Total <- c(cumsum(UW_scores))
UW_MSU$MSU_Total <- c(cumsum(MSU_scores))
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

You can use the for loops for cumulative sum as well. Checking this code:

load table of scoring

scoring=read.table("UWvMSU_1-22-13.txt",header=TRUE,sep="\t",stringsAsFactors=FALSE)

look at data

dim(scoring)
head(scoring)

preallocating matrix to store cumulative scores

cum_scores=matrix(NA,nrow(scoring)+1,3)
cum_scores[,1]=c(0,scoring[,1])
cum_scores[1,2:3]=0
colnames(cum_scores)=c("time","UW","MSU")

looping through individual scoring events

for(i in 1:nrow(scoring)){
if(scoring[i,2]=="UW"){
cum_scores[(i+1),2]=cum_scores[i,2]+scoring[i,3]
cum_scores[(i+1),3]=cum_scores[i,3]
}else{
cum_scores[(i+1),2]=cum_scores[i,2]
cum_scores[(i+1),3]=cum_scores[i,3]+scoring[i,3]
}
}

Comment thread Exercise10.R
print(number)
return('end of game')
}}
}
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

There should be a condition when i > 10 then the console will print out that this is out of guesses.

Copy link
Copy Markdown
Owner

@qtran4 qtran4 left a comment

Choose a reason for hiding this comment

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

2

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.

2 participants