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

Pcc01 #805

Open
wants to merge 100 commits into
base: community
Choose a base branch
from
Open

Pcc01 #805

wants to merge 100 commits into from

Conversation

eternity336
Copy link

Difficulty level (1-10): [1]
Estimated time spent (hours): [1]
Completed (yes/no): [Yes]
I stretched my coding skills (if yes what did you learn?): [The hardest part was figuring out GIT. I got it now though.]
Other feedback (what can we improve?): [I was trying to use pycharm. Would be nice to have a video on that as well.]

pybites and others added 30 commits May 8, 2017 10:07
another option is to parse twitter for netflix recommendations (API)

and to take it further learning some ML to make a recommendation engine (but requires more structured data and user input)
…ites#154)

I added a test to check for a mutated draw during validation.
* Only one test failing, but thats becuase of change in the tweety api

* Improved testing using mock for the api, not perfect

* Improved test with mock of tweepy.API.user_timeline

* Only modified tests - see issue 220

* Removed solution
# Conflicts:
#	01/NetSecParano/wordvalue-template.py
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.

None yet

5 participants