Skip to content

Palakurthi-Blanc submission#5

Open
bpalakur wants to merge 15 commits into
araiho:masterfrom
bpalakur:master
Open

Palakurthi-Blanc submission#5
bpalakur wants to merge 15 commits into
araiho:masterfrom
bpalakur:master

Conversation

@bpalakur

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.

Remember to make sure your script is clean before your final pull request. And more code commenting would help me know which questions you are answering and how your code will work. 2.4/3

Comment thread wageanalysis.sh
cat wages.csv|sed 's/,/ /g'|sort -k 4 -n|tail -n 1
cat wages.csv|sed 's/,/ /g'|sort -k 4 -n|head -n 2|tail -n 1
cat wages.csv|sed 's/,/ /g'|sort -k 4 -n|tail -n 10| grep female| wc -l
<<<<<<< HEAD

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 need to delete these. They are references for merge conflicts and will cause an error when you run your script.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Sure, but we wanted to make sure that you knew we worked separately and merged conflicts as mentioned in our exercise hand out!

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

May I know why am I given 20% less?

Comment thread wageanalysis.sh

>>>>>>> fb07a0e23b3536c900e7e8d4a4f00c5f8f8570d8
#Returns the unique gender and years experience combinations
cat wages.csv | sed 's/,/ /g' | sort -k 1 -n -k 2 | sed 's/ /,/g' | cut -d , -f 1,2 | 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.

Is this your answer to 1?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Yes, it is!

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