deep-diver / EN-FR-MLT-tensorflow

English-French Machine Language Translation in Tensorflow
109 stars 52 forks source link

Purpose masks #5

Open Shiro-LK opened 5 years ago

Shiro-LK commented 5 years ago

Hello,

Thanks for your repository, it helps a lot ! I am trying to understand the purpose of the masks in your code. I assume the goal is to remove the padding from the loss function, but looking at the get_batches function, it seems that the array of lenghts returned has always the same value as you compute the lengths of the target pad sequence instead of the target sequence. Is it a mistake ? Or the masks has another goal ?