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

Contents of a starting kit #2

Open
rth opened this issue Oct 25, 2017 · 0 comments
Open

Contents of a starting kit #2

rth opened this issue Oct 25, 2017 · 0 comments

Comments

@rth
Copy link
Contributor

rth commented Oct 25, 2017

Continuing the conversation from 58c9c0b#commitcomment-25129227

Thanks for your responses @vlagani and @kegl !

I would make it simpler than what we have now, @EmanuelaBoros is working on it. At this point it crashes both in my computer and in travis https://travis-ci.org/ramp-kits/fake_news/jobs/292321109 so I can't improve it.

Yes, then probably fixing tests in the first step. Made a PR in #1 hoping to help with tests. Currenly the issue is that test.csv in pep8 branch is not correctly formatted (space separated instead of tab separated).

Since we will use this challenge starting this Friday in our DataCamp class, I don't want to give them much help (other than a working prototype :) )
We'll also need a decent notebook, by tomorrow evening.

I won't be able to help much before Friday, but will try to make some contributions next week.

Also, it would help if it the master branch was changed mostly via Pull Requests, as otherwise it's difficult to follow changes. Thanks!

cc @EmanuelaBoros

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

No branches or pull requests

1 participant