Eric Kafe

Results 106 comments of Eric Kafe

Of the 2 doctests failures in File "nltk/parse/dependencygraph.py", only the first is caused by missing the dot binary from Graphviz package. The second failure reveals a genuine problem in the...

I used the following script to run all the tests in one batch: ``` #!/bin/sh #ln -s nltk PATH_to_nltk mkdir out for f in `find nltk/* -type f -name '*py'...

The two approaches differ in the way they handle bugs found outside the doctests: while 'python -m doctest' only prints a warning to stderr, 'pytest --doctest-modules nltk' prints the error...

After applying PR #3033 by @tomaarsen, "pytest --doctest-modules ./nltk/" works fine for finding which files have at least one error. For example, in the "tag" subdirectory: > nltk/tag/brill_trainer.py:126: DocTestFailure >...

For debugging, pytest's "--doctest-continue-on-failure" may be handy in order to obtain more complete figures: `pytest --doctest-modules --doctest-continue-on-failure ./nltk/tag/` > nltk/tag/brill_trainer.py:126: DocTestFailure > nltk/tag/brill_trainer.py:176: DocTestFailure > nltk/tag/brill_trainer.py:213: DocTestFailure > nltk/tag/hunpos.py:36: UnexpectedException...

I guess it will take several PRs before this issue is closed :) The list of errors is rather long, so it seems better to reduce it in small steps,...

Thanks @goodmami for your kind words about my recent updates in the standard wordnet module. It is indeed in better shape now, although the lack of compatibility of OMW with...

Concerning the maintenability claim, an important factor is the size of the codebase: the existing wordnet module counts 2301 lines, while the new Wn module is more than triple that...

Thanks @fcbond. I don't know enough about Wn's features, but I noticed that it doesn't seem to have a "sense key" notion, although sensekeys are the only reliable identifier across...

https://github.com/globalwordnet/cili/issues/9#issuecomment-929475375 mentions "a backlog of several years" in attributing the envisioned ILI identifiers. This leaves ample time to prepare the deprecation warnings.