is get_sequence_actual_length right?

liu-nlper / NER-LSTM-CRF

An easy-to-use named entity recognition (NER) toolkit, implemented the Bi-LSTM+CRF model in tensorflow.

347 stars 117 forks source link

Closed JimmyCai closed 6 years ago

JimmyCai commented 7 years ago

Hi liu,

when tensor in function get_sequence_actual_length(utils.py) is: [[-0.1,0.1,0]] it will return: [0] but actually, it is[2] so, it is wrong, right?

liu-nlper commented 6 years ago

Hi, you are right, but the argument "tensor" has no negative value.