search

monikkinom / ner-lstm

Named Entity Recognition using multilayered bidirectional LSTM
540 stars 183 forks source link

The `used` variable can be renamed to be more descriptive #8

Closed 0xnurl closed 7 years ago

0xnurl commented 7 years ago

First of all thanks so much for publishing this model, it is tremendously helpful.

One small suggestion: it takes a while to understand the purpose of the variable used (line 18). A more descriptive name for future readers may be something like words_used_in_sentence.

shreenibhar commented 7 years ago

Thanks for the edit. The var name has been changed.