Skip to content

Conversation

@antmarakis
Copy link
Collaborator

This PR unfortunately ballooned a bit, since I had to test multiple things at the same time. These are the changes:

  • Iris Visualization

  • Utilized notebook.py to hide matplotlib code from the notebook

  • Quick evaluation of learners

@antmarakis
Copy link
Collaborator Author

It seems that Travis is missing the mpl_toolkits module from matplotlib. Does anyone know how to add this? I assume I must add something to travis.yml.

@antmarakis
Copy link
Collaborator Author

I added the following line to travis.yml and the issue is now fixed:

pip install matplotlib

I would like someone to confirm this is something we want to do, since I am not that familiar with Travis.

@norvig norvig merged commit 0bb4069 into aimacode:master Jul 3, 2017
@norvig
Copy link
Collaborator

norvig commented Jul 3, 2017

I wish I understood Travis better myself, and I don't know the answer for sure, but it seems like it can't hurt to do pip install matplotlib

@antmarakis antmarakis deleted the patch-2 branch July 3, 2017 10:13
dj5x5 pushed a commit to dj5x5/aima-python that referenced this pull request Jul 17, 2025
…macode#568)

* Update learning.ipynb

* Update notebook.py

* Update .travis.yml
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