search

uhh-lt / sensegram

Making sense embedding out of word embeddings using graph-based word sense induction
http://uhh-lt.github.io/sensegram
212 stars 50 forks source link

The -n param, the max number of edges, is ignored in the code #35

Open chanind opened 3 years ago

chanind commented 3 years ago

The -n param option specifies the number of edges in the ego graph, but it looks like in the code this param is accidentally ignored. This param is passed through to word_sense_induction.py#L66 as max_related, but that param is never read. It looks like the intention is to set the global var n to that value, and then use that in word_sense_induction.py#L56 to limit the size of the cluster. However, this is never set in the code and n remains set to None.