Skip to content

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