Skip to content

Turning in the word frequency toolbox. #33

Open
LivKelley wants to merge 2 commits intosd16spring:masterfrom
LivKelley:master
Open

Turning in the word frequency toolbox. #33
LivKelley wants to merge 2 commits intosd16spring:masterfrom
LivKelley:master

Conversation

@LivKelley
Copy link

Yay!


This change is Review on Reviewable

@LucyWilcox
Copy link

Review status: 0 of 2 files reviewed at latest revision, 3 unresolved discussions.


Liv_WordFrequency.py, line 20 [r1] (raw file):
You shouldn't start looking at lines until after the header (which is standardized in Project Gutenberg)


Liv_WordFrequency.py, line 44 [r1] (raw file):
You pass this in so you don't need to call it too.


Liv_WordFrequency.py, line 57 [r1] (raw file):
This stuff could be in an if name equals main statement.


Comments from the review on Reviewable.io

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