Skip to content

Updated Learning Notebook (outdated) #318

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

Closed
wants to merge 2 commits into from
Closed

Updated Learning Notebook (outdated) #318

wants to merge 2 commits into from

Conversation

antmarakis
Copy link
Collaborator

  • Removed MNIST section
  • Removed Numpy section
  • Added PluralityLearner section (Overview, Implementation and Example)
  • Updated kNN section (added Implementation and Example)

Tutorial on the kNN and PluralityLearner algorithms.
@norvig
Copy link
Collaborator

norvig commented Mar 6, 2017

I don't understand why you would remove MNIST? That was a very nice section.

@antmarakis
Copy link
Collaborator Author

antmarakis commented Mar 6, 2017

It is nice, but I don't think it does much to help understand the algorithms in learning. It is a "load-and-visualize" section, without a connection to the algorithms. I think that disconnection will confuse readers and possibly turn them away. If we keep it, we can do one of the following:

  1. Use it as the main dataset. Could be done, but it takes long to classify an image.

  2. Move it to the bottom of the page running the algorithms on its own, as a "real-world" example, comparing performances and such. The main dataset in that case is something simpler, like the iris one.

  3. Simply leave it as is and use another dataset for the examples below.

@norvig
Copy link
Collaborator

norvig commented Mar 7, 2017

I agree with your suggestion 2 -- move it to the bottom. Then we can warn students that it takes a while to run, and it doesn't stop anything below it from running.

@antmarakis antmarakis changed the title Updated Learning Notebook Updated Learning Notebook (outdated) Mar 7, 2017
@antmarakis antmarakis closed this Mar 7, 2017
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
pFad - Phonifier reborn

Pfad - The Proxy pFad of © 2024 Garber Painting. All rights reserved.

Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.


Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy