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

Chapter 7 typo #83

Open
rickiepark opened this issue Oct 18, 2018 · 5 comments
Open

Chapter 7 typo #83

rickiepark opened this issue Oct 18, 2018 · 5 comments
Labels
fixed-in-3rd-edition Tagged as fixed in the upcoming 3rd edition

Comments

@rickiepark
Copy link

rickiepark commented Oct 18, 2018

(p225) In the middle of page,
"argmax_i [0.2 \times i_0 + 0.2 \times i_0 + 0.6 \times i_1]" should be "argmax_i [0.2 \times i_0 + 0.2 \times i_0, 0.6 \times i_1]"

Sorry I misunderstood it.
"argmax_i [0.2 \times i_0 + 0.2 \times i_0 + 0.6 \times i_1]" is right. :)

@rickiepark rickiepark changed the title Chapter 6 typo Chapter 7 typo Oct 23, 2018
@rickiepark
Copy link
Author

(p234) In 3rd line of the first code block, print("Accuracy: ...") should be print("ROC AUC: ..."), because output text is ROC AUC: ....

@rickiepark
Copy link
Author

(p237) In 3rd line, "inside a GridSearch object" should be "inside a GridSearchCV object".

@rickiepark
Copy link
Author

(p221) In 2nd eq. in the page,
"1 if \sum_{i} C_{j}(x) \ge 0" should be "1 if \sum_{j} C_{j}(x) \ge 0"

@rickiepark
Copy link
Author

(p248) In step 3, \hat{y} eq. is ambiguous. I suggest \hat{y} = sign[\sum_{j=1}^{m} (\alpha_j \times predict(C_j, X))].

(p250) End of the 1st paragraph, "(step 4)" should be "(step 3)".

@rasbt
Copy link
Owner

rasbt commented Jun 16, 2019

(p234) In 3rd line of the first code block, print("Accuracy: ...") should be print("ROC AUC: ..."), because output text is ROC AUC: ....

Oh yes, definitely. Thanks!

(p237) In 3rd line, "inside a GridSearch object" should be "inside a GridSearchCV object".

Yes!

(p221) In 2nd eq. in the page,
"1 if \sum_{i} C_{j}(x) \ge 0" should be "1 if \sum_{j} C_{j}(x) \ge 0"

Thanks!

(p248) In step 3, \hat{y} eq. is ambiguous. I suggest \hat{y} = sign[\sum_{j=1}^{m} (\alpha_j \times predict(C_j, X))].

Yes, that's better

(p250) End of the 1st paragraph, "(step 4)" should be "(step 3)".

Yes

@rasbt rasbt added the fixed-in-3rd-edition Tagged as fixed in the upcoming 3rd edition label Jun 16, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fixed-in-3rd-edition Tagged as fixed in the upcoming 3rd edition
Projects
None yet
Development

No branches or pull requests

2 participants