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

Rewritten readability tests from nltk_contrib #5

Open
kmike opened this issue Oct 24, 2012 · 0 comments
Open

Rewritten readability tests from nltk_contrib #5

kmike opened this issue Oct 24, 2012 · 0 comments

Comments

@kmike
Copy link
Member

kmike commented Oct 24, 2012

Hi,

I wanted to use readability tests from nltk_contrib package. The code does not meet my requirements (like choosing between Dutch and English) and I have rewritten it.

I put it there so others could use the work as well. Maybe it could become a part of official nltk package? I would like to edit it so it meets the code standard of nltk.

Migrated from http://code.google.com/p/nltk/issues/detail?id=677

earlier comments
alex.rudnick said, at 2011-05-24T06:34:59.000Z:

Thanks for submitting updates! Could you outline the changes you made? Improvements are always welcome! Are you interested in contributing this as part of nltk_contrib? (could it maybe be merged in to the existing readability package, or would you prefer it to be separate?)

izidor.matusov said, at 2011-05-24T06:58:19.000Z:

I made these changes: 1) The original readability tests module of nltk_contrib contains code which is not related to readability tests. That code was removed. 2) Repaired a few lines so the code actually runs in the current version of Python. 3) Removed support for Dutch 4) Polished interface 5) Rewritten code, so pylint does not nag so much. Yes, I am interested. It could be merged but there are many radical changes so it could be a problem for someone who uses the previous readability package.


Migrated from nltk/nltk#161

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