Eric Kafe comments

Results 236 comments of


                                            Eric Kafe

Let tokenizers be installed via Pip

A smooth path forward could start by defining two collections in nltk_data, listing the contents of the two suggested PyPi bundles. The "essentials" bundle would correspond almost to the existing...

`nltk.util.elementtree_indent` should restrict parameter ranges in the documentation

Users who need a guard can implement it in one line around their own call.

Tag 3.9.1 points to commit which doesn't include #3309, which it was intended to ship

A solution could be to release 3.9.2.

Tag 3.9.1 points to commit which doesn't include #3309, which it was intended to ship

**Update on Resolution** This issue has been resolved, but the situation is slightly unusual: - **The PyPI package for `nltk==3.9.1` (`pip install nltk`) already includes the fix** for the WordNet...

Tag 3.9.1 points to commit which doesn't include #3309, which it was intended to ship

The situation is catastrophic for users who download the 3.9.1 version from Github with an initially blank nltk_data, because they have bug #3308. The current state, where the official tagged...

Tag 3.9.1 points to commit which doesn't include #3309, which it was intended to ship

Please test this experimental release: https://github.com/ekaf/nltk/releases/tag/3.9.1-fix1. It includes the latest commits and should solve this issue.

Avoid KeyError in langnames.py

Reverted to the most minimal edits

Avoid KeyError in langnames.py

This PR addresses issue #3403 by updating language code conversion functions to use `.get()` for dictionary lookups (specifically `bcp47.wiki_q` and `wiki_bcp47`). This prevents disruptive `KeyError` exceptions for unknown tags or...

Avoid KeyError in langnames.py

@stevenbird, this will take some time. This PR is not important enough to delay the release of 3.9.2.

Semcor License is likely invalid

For distributing SemCor as non-commercial "for academic use only", NLTK should theoretically be covered by the existing sublicense granted from Brown University to the Princeton University/WordNet project. Here's the reasoning:...