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.

Sign up for GitHub

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

Jump to bottom

Revise typo, texts & apostrophes in linear-regression-scratch.md #76

Merged
merged 13 commits into from Dec 7, 2018
Merged

Revise typo, texts & apostrophes in linear-regression-scratch.md #76

merged 13 commits into from Dec 7, 2018

Conversation

bowen0701
Copy link
Contributor

@bowen0701 bowen0701 commented Dec 7, 2018 •
edited

Revise the following:

  • typo: "need iterate" -> "need to iterate"
  • texts: "To generate the training set" -> "To evaluate the trained model"
  • label notation in comments of mini-batch training scripts: "Y" -> "y"
  • redundant spaces
  • apostrophes: ’ -> '

Bowen Li and others added 13 commits December 6, 2018 19:18
Revise ref 2 typos & add figure titles
c7f5b43
Revise figure 11.22 title typo in how-to-contribute.md
cc11f8c
Resolve markdown's figure title error in how-to-contribute.md
c2f9819
Reverse to old figure title w/ figure no. in linear-regression-scratc…
4da2748 
…h.md
Remove figure numbering in how-to-contribute.md
a36f791
Merge remote-tracking branch 'upstream/master'
fed4008
Fix train loss error: train_l.mean() -> train_l.sum() in linear-regre…
f7e8a8b 
…ssion-scratch.md
@bowen0701
Merge pull request #1 from bowen0701/ch1
cf1fbc4 
Fix train loss error: train_l.mean() -> train_l.sum()
Merge branch 'master' of https://github.com/bowen0701/d2l-en
9b48598 
Pull master
Merge remote-tracking branch 'upstream/master'
58a9425 
Featch upstream master
Revise back to train_l.mean()
f6e6828
Revise typo, texts & apostrophes
ac2b2f7
@bowen0701
Merge pull request #2 from bowen0701/ch1
9e07cc9 
Revise typo, texts & apostrophes
@astonzhang astonzhang merged commit 1c68d56 into d2l-ai:master Dec 7, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@bowen0701 @astonzhang