Skip to content
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

Suppress pyplot output #63

Open
wants to merge 8 commits into
base: master
Choose a base branch
from
Open

Suppress pyplot output #63

wants to merge 8 commits into from

Conversation

raucher
Copy link

@raucher raucher commented Mar 10, 2017

Hi Parag, I have tweaked lectures from 0 to 3. Semicolons was added to pyplot statements suppressing an output and several DeprecationWarnings was removed.
I didn't run notebook with celebs (I don't have dataset), so run it if you want to make things more beautiful out of the box. Also printing learning cost yielded very long output (I have ATI GPU so maybe that's a reason), thus I changed it to print out only every 1K'th iteration, you may invent a variable for printing threshold and let student decide the right threshold for him (because students w/o nVidia may encounter the same problem).
In one word, please run changes and if all is fine, let me know, I'll finish lectures 4, 5 tomorrow!

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.

1 participant