Skip to content
New issue

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

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

Already on GitHub? # 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.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants