Skip to content

Egan-Long submission#6

Open
Basmirkl wants to merge 15 commits into
mdoellma:masterfrom
BaileeEgan:master
Open

Egan-Long submission#6
Basmirkl wants to merge 15 commits into
mdoellma:masterfrom
BaileeEgan:master

Conversation

@Basmirkl

Copy link
Copy Markdown

No description provided.

Comment thread wages_challenge.sh
@@ -0,0 +1,28 @@
#Gets unique combinations of gender and yearsExperience, sorting by gender and then yearsExperience

@mdoellma mdoellma Oct 6, 2017

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 to see you committing often and with descriptive messages!
Please read the directions more carefully in the future.

@mdoellma mdoellma Oct 6, 2017

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.

2.75/3

Comment thread wages_challenge.sh
@@ -0,0 +1,28 @@
#Gets unique combinations of gender and yearsExperience, sorting by gender and then yearsExperience
echo "Unique combos of gender and yearsExp, sorted:"
cat wages.csv | grep -v "gender" | cut -d ',' -f 1,2 | sed 's/,/ /g' | sort -k1,1 -k2n | uniq

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.

correct, but should have been written to a file, according to directions given

Comment thread wages_challenge.sh
echo "Unique combos of gender and yearsExp, sorted:"
cat wages.csv | grep -v "gender" | cut -d ',' -f 1,2 | sed 's/,/ /g' | sort -k1,1 -k2n | uniq

#---Start of challenge 2---

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.

Correct, but returns the years schooling column as well. Please read directions fully.

Comment thread wages_challenge.sh

#---Start of challenge 3---
#Returns minimum wage for 12 years of education
minwage_12=$(cat wages.csv | grep -v "gender" | tr "," " " | grep -w "12" | sort -n -k4 | head -n 1 | cut -d " " -f4)

@mdoellma mdoellma Oct 6, 2017

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.

Close. You are finding lines with both 12 years schooling and 12 years experience. Use the cut tool to remove the years experience column before grep -w "12".
-0.25 pts.

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