Skip to content

Davis-Chung submission#1

Open
henrichung wants to merge 3 commits into
araiho:masterfrom
henrichung:master
Open

Davis-Chung submission#1
henrichung wants to merge 3 commits into
araiho:masterfrom
henrichung:master

Conversation

@henrichung

Copy link
Copy Markdown

Challenges 1 and 3 are in one file. Challenge 2 was done separately.

@araiho araiho left a comment

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.

Good job figuring out ordering and unique. There are a few places where naming is inconsistent or non-informative. The last question was not totally complete so for this assignment your grade is 3.5/4.

Comment thread exercise5_code.R
wages = read.csv(file="wages.csv", header=TRUE, stringsAsFactors = FALSE)

#Challenge question 1
test1 <- unique(wages[,c("gender","yearsExperience")])

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.

try to avoid names like 'test1' and 'test2.' It will make your code confusing when you have to come back to it someday.

Comment thread exercise5_code.R
#Challenge question 2


#Challenge question 3

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.

The key word in this question is effect. So, the correct answer would include the difference between the minimum difference between graduating high school and graduating college.

Comment thread Ex5_2_sorted_wages.R
head(sorted.wages, n = 1)

#Lowest Wage
tail(sort.wages, n = 1)

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.

I think that you switch names here. This will cause an error.

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.

3 participants