Question about BagOne

[pminervini]

I was checking the BagOne class, and on this line:

OpenNRE/opennre/model/bag_one.py

Line 46 in 0acd945

token, pos1, pos2, mask = self.sentence_encoder.tokenize(item)

the method sentence_encoder.tokenizer was being used in this way:

token, pos1, pos2, mask = self.sentence_encoder.tokenize(item)

However, aren't the position of pos1 and masks flipped? And, what if the masks entry is not being returned?
E.g. see here:

Different order of the returned values: https://github.com/thunlp/OpenNRE/blob/master/opennre/encoder/bert_encoder.py#L215

No masks returned:

OpenNRE/opennre/encoder/base_encoder.py

Line 154 in 0acd945

return indexed_tokens, pos1, pos2