Skip to content

New branch#235

Open
bertramwooster wants to merge 38 commits into
rasbt:mainfrom
bertramwooster:new_branch
Open

New branch#235
bertramwooster wants to merge 38 commits into
rasbt:mainfrom
bertramwooster:new_branch

Conversation

@bertramwooster

Copy link
Copy Markdown

I have rectified multiple typos and simplified code in places. I have also used comments to make notebooks as self-explanatory as possible. Ciao.

@review-notebook-app

Copy link
Copy Markdown

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@rasbt

rasbt commented Nov 9, 2025

Copy link
Copy Markdown
Owner

Thanks for the PR, I appreciate it. I will go through it carefully in the next months and go through it one by one.

@rasbt

rasbt commented May 21, 2026

Copy link
Copy Markdown
Owner

Thanks a lot for these. I don't want to directly merge this mega-PR into main because several of these are suggestions, and it would diverge too much from the print book, but I am incorporating them one by one into an upcoming new edition.

  • Ch03 Instead of adding OvR for compatibility with newer sklearn versions, change to multinomial
  • Ch04 SBS class updates
  • Ch05 Use np.linalg.eigh instead of np.linalg.eig

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