-
Notifications
You must be signed in to change notification settings - Fork 51
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
General thoughts/suggestions on NLTK content #20
Comments
On 16 February 2015 at 20:27, Cameron McLean [email protected]
cheers "This is a profound psychological violence here. How can one even begin to On the Phenomenon of Bullshit Jobs, David Graeber |
A few comments based on experience/reflection from day 1 of ResBaz - feel free to discuss, discard, or modify as appropriate...
sent4
orsent7
that was defined elsewhere and invisible to the learner. Would it be better to have the user create two new lists from scratch, and then see how you can join and manipulate them?print
and others just type the variable name and have the interpreter display the contents. We should be consistent and (i suggest) always useprint variable
- this way learner get used to the idea of using lots of print statements to look inside their variables, a useful debugging skill.;
in (any) array and store/print anything before it - but were stumped because of lack of skills/practice yet - perhaps reword this example so it's clear we 'know' what is in the array and you want us to count and slice given a known content.sorted(whole_corpora)
orarray[8:]
type things - check that all examples are "runnable" if the user tries them on the loaded texts.[len(w) for w in text]
etc - I think it might better to write these out longform - especially as learners haven't been introduced to for loops yet etc.and
conditional which wasn't introduced earlier...w
for "word" and sometimes we useword
- be consistent. Perhaps the longer form is easier for learners to follow thanw
The text was updated successfully, but these errors were encountered: