Tom Aarsen

Hello! ### Short description The [information_extraction](https://github.com/sebastianruder/NLP-progress/blob/master/english/information_extraction.md) page has illogical values for F1 scores. ### Assumptions This issue assumes that the F1 score is computed as: ``` F1 = 2 *...

Hello! ## Pull request overview * Add a radio button in the HTML renderer that allows viewing the time as percentages of the total time. * Small change in `Frame.js`...

Potential accidental capture of loop variable in Boxer

1

Hello! Within the last list comprehension in this return statement, the `cond` value will always take the value of the last element of `conds`: https://github.com/nltk/nltk/blob/4a130f1cdf642327378a4387b319cd2df6f79287/nltk/sem/boxer.py#L533-L537 This seems like it would...

In CI, refresh `nltk_data` cache if the hash of `index.xml` differs from the cached hash

4

Rather than simply caching `nltk_data` until the cache expires and it's forced to re-download the entire `nltk_data`, we should perform a check on the `index.xml` which refreshes the cache if...

GitHub Actions CI does not download third party dependencies

6

Hello! The Travis CI used to use https://github.com/nltk/nltk/blob/develop/tools/travis/third-party.sh to ensure that the Stanford CoreNLP, Parser, POSTagger and SENNA are downloaded before running the CI. The current Github Actions CI workflow...

Finishing up Stanford Deprecation

7

Hello! As some of you might be aware, several Stanford related classes have been deprecated back in 2017. They are the following: * [`nltk.tag.StanfordTagger`](https://github.com/nltk/nltk/blob/develop/nltk/tag/stanford.py#L31) * [`nltk.tag.StanfordPOSTagger`](https://github.com/nltk/nltk/blob/develop/nltk/tag/stanford.py#L139) * [`nltk.tag.StanfordNERTagger`](https://github.com/nltk/nltk/blob/develop/nltk/tag/stanford.py#L176) * [`nltk.parse.GenericStanfordParser`](https://github.com/nltk/nltk/blob/develop/nltk/parse/stanford.py#L28)...

Suggestions and discussions regarding `pre-commit`

9

Hello! Now that `pre-commit` has been in place since #2753, we should take a moment to reflect on it, and consider whether we want to remove or add certain hooks...

In the following locations: * https://github.com/nltk/nltk/blob/f989fe65d421e7ea4d1037a00f07eaeee3ad6a29/nltk/probability.py#L247-L249 * https://github.com/nltk/nltk/blob/f989fe65d421e7ea4d1037a00f07eaeee3ad6a29/nltk/probability.py#L1922-L1932 * https://github.com/nltk/nltk/blob/f989fe65d421e7ea4d1037a00f07eaeee3ad6a29/nltk/text.py#L607 We should by default merely return the matplotlib `ax` object, rather than showing the plot. That way, users can still...

Documentation typo fixed

1

Hello! ### Pull request overview * Resolved two typos in `README.rst` and `build_sphinx_docs.sh`. --- Note that `bash make_readme.sh` must be ran after this, as the README has been updated. I...

The lines https://github.com/CubieDev/TwitchMarkovChain/blob/c7b8639bf85ae129f4a1e3f0dd757ecdef75f098/Database.py#L226-L230 will pick a character and then select data that starts with that character. However, this means that the bot needs to learn sentences that start with any...