Skip to content

Learning - #578

Merged
norvig merged 2 commits into
aimacode:masterfrom
Chipe1:learnnb
Jul 9, 2017
Merged

Learning#578
norvig merged 2 commits into
aimacode:masterfrom
Chipe1:learnnb

Conversation

@Chipe1

@Chipe1 Chipe1 commented Jul 8, 2017

Copy link
Copy Markdown
Contributor

Added section on Decision tree learner to notrbook
Added random forest code which is to be added in the 4th edition

@antmarakis

antmarakis commented Jul 8, 2017

Copy link
Copy Markdown
Collaborator

Nice addition. Is it possible to have some tests up?

Also, can you capitalize the header for the notebook section? Lastly, it seems that the Naive Bayes Classifier section has some issues in the "Examples" sub-section. You seem to have switched it from "Code" to "Markdown". Can you fix it?

@norvig

norvig commented Jul 9, 2017

Copy link
Copy Markdown
Member

This is looking very nice. But maybe the notebook is getting too long -- about 25 pages now -- maybe break out the neural part to a separate notebook.

@norvig
norvig merged commit b785561 into aimacode:master Jul 9, 2017
@Chipe1

Chipe1 commented Jul 9, 2017

Copy link
Copy Markdown
Contributor Author

@norvig Will the 4th edition go more in-depth with CNN, autoencoder etc. If so, should we use external libraries like TensorFlow in the separate notebook?

@Chipe1 Chipe1 mentioned this pull request Jul 9, 2017
dj5x5 pushed a commit to dj5x5/aima-python that referenced this pull request Jul 17, 2025
* Added decisiontreelearner to notebook

* Added RandomForest
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.

3 participants