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

average_word_length initialized? #16

Open
nrjones8 opened this issue Jan 13, 2014 · 1 comment
Open

average_word_length initialized? #16

nrjones8 opened this issue Jan 13, 2014 · 1 comment

Comments

@nrjones8
Copy link
Collaborator

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
Copy link
Collaborator

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.

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

2 participants