Question of the parameter 'sinusoid'

[CrystalWLH]

I want to know why using the parameter 'sinusoid'? And could I not set the parameter ‘maxlen’ and using varying length of the text between batch?

[alphadl]

@CrystalWLH Because this repository is an earlier Transformer version, the original version is implemented with Tensorflow, see Kyubyong/transformer