Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix Apply_04/US_Crime_Rates/Exercises_with_solutions Step 8 and Step 9 . #106

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

tanerant
Copy link

@tanerant tanerant commented Aug 2, 2020

Step 8 solution is in 1960-1-1 format instead of 1960.

Step 9 solution is 'crimes.idxmax (0)' instead of 'crime.idxmax (0)'.

… Step 9 .

Apply_04/US_Crime_Rates/Exercises_with_solutions.ipynb

Step 8 solution is in 1960-1-1 format instead of 1960.

Step 9 solution is 'crimes.idxmax (0)' instead of 'crime.idxmax (0)'.
Copy link

@Mandroide Mandroide left a comment

Choose a reason for hiding this comment

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

Solve the conflict between files.

@tanerant
Copy link
Author

tanerant commented Nov 7, 2020

Hi Mandroide . I think solved. Thanks for sharing this exercise.

@Mandroide
Copy link

Hi Mandroide . I think solved. Thanks for sharing this exercise.

Hi, I will share with you some advice. Whenever you open a pull request, avoid thrashing. You could perform a git reset --soft HEAD~2 to modify your commit history and commit the final solution to avoid thrashing and push force your commit.

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