Missing ic argument in res, jcn and lin similarity functions of wordnet

[aperezlebel]

Hello,

I just ran into a little bug when using res_similarity, jcn_similarity and lin_similarity functions. The ic argument is not passed to the corresponding methods of the Synset class.

Example:

import nltk
from nltk.corpus import wordnet as wn
from nltk.corpus.reader.wordnet import jcn_similarity, lin_similarity, res_similarity

nltk.download('wordnet')

# Random synsets
s1 = wn.synset_from_pos_and_offset('n', 7579787)
s2 = wn.synset_from_pos_and_offset('n', 2226429)

ic = {}
res_similarity(s1, s2, ic)
jcn_similarity(s1, s2, ic)
lin_similarity(s1, s2, ic)

Throws:
TypeError: 'bool' object is not subscriptable

[tomaarsen]

Well spotted! The verbose parameter on the calling side was filling the ic parameter on the function side. Alongside your changes I've ensured that verbose is only passed as a keyword argument. I've also fixed the pre-commit failing tests (caused by some ill-formatted code).

I appreciate it! I'll merge this once tests pass.