Skip to content
This repository has been archived by the owner on Jun 27, 2022. It is now read-only.

Minor revision for better formality and readability #90

Merged
merged 1 commit into from
Mar 19, 2018

Conversation

wla80
Copy link

@wla80 wla80 commented Feb 24, 2018

This commit is a part of my school assignment for revising technical document on github. Please feel free to provide me your feedback and hope the author can approve my pull request.

Thanks

@wla80 wla80 mentioned this pull request Feb 24, 2018
@floer32
Copy link
Collaborator

floer32 commented Feb 26, 2018

@wla80 Don't worry about the broken links by the way, for this repository I will handle it.

@floer32
Copy link
Collaborator

floer32 commented Feb 26, 2018

I like some of this revision! Some parts I want to change though, as I try to keep a certain style and tone in this guide.

What about this?

When you work on a real Machine Learning problem, you should focus your efforts on your domain knowledge and data before optimizing your choice of algorithms. You should strive for Do Simple Things until you have to increase complexity. You should not rush into neural networks because you think they're cool. To improve your model, get more data and use your knowledge of the problem to manipulate the data. You should only optimize the choice of alogrithms after you have gathered enough data, and you've processed it well.

@wla80
Copy link
Author

wla80 commented Feb 26, 2018

Hello @hangtwenty , thanks for your feedback. If you wish to maintain your style and tone, this looks good to me. I just uploaded the fix. Thank you

@floer32
Copy link
Collaborator

floer32 commented Mar 19, 2018

@wla80 Thank you for the follow up and sorry for the delay! Merging

@floer32 floer32 merged commit eef73ac into dive-into-machine-learning:master Mar 19, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants