NoahCarnahan / plagcomps

Other
6 stars 1 forks source link

average_word_length initialized? #16

Open nrjones8 opened 10 years ago

nrjones8 commented 10 years ago

When running average_word_length, it still requires that we have an instance variable self.average_word_length_initialized referenced here: https://github.com/NoahCarnahan/plagcomps/blob/master/intrinsic/featureextraction.py#L265

I'm putting the instance variable back in for the moment (until we talk more about how to use avg(word) or however the function name will work. does this make sense @zachwooddoughty or am i missing a silly fix?

zachwooddoughty commented 10 years ago

Whoops, I forgot to mention this at our meeting this morning. average word length is now deprecated (though I didn't comment that), and you should use "avg(num_chars)" instead. I'll delete it tonight.