Skip to content

vendrely submission#2

Open
kvendrel wants to merge 1 commit into
araiho:masterfrom
kvendrel:master
Open

vendrely submission#2
kvendrel wants to merge 1 commit into
araiho:masterfrom
kvendrel:master

Conversation

@kvendrel

Copy link
Copy Markdown

No description provided.

@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.

Great Job! 4/4

Comment thread challenge_1.R
#this will write the wages1list that contains the unique gender-yearsExperience
#combinations contained in the wages.csv file
wages1list=merge(uniqueFemales,uniqueMales,all=TRUE)
wages1list

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.

This could be shortened with

genderYears=data[,1:2]
unique_genderYears=unique(genderYears)

sorted=unique_genderYears[order(unique_genderYears[,1],unique_genderYears[,2]),]

But good effort! Nice use of merge.

Comment thread challenge_2.R
lowestEarner

message("the number of females in the top ten earners in this data set:")
nrow(TopTenFemales)

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.

Nice!

Comment thread challenge_3.R
wages3=wages[,3:4]

#make a list of only those who have graduated from HS (12 years of school)
HSgrads=wages3[wages3[,1]=="12",]

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.

Watch out naming things with uninformative names. If you come back later, you might not know what you were trying to do.

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