I added the truncation logic in the __call__ of BatchConverter and added a truncate parameter in its constructor.
I checked that BatchConverter is accessed through the get_batch_converter method of Alphabet, therefore I added a truncate parameter to this method.
I accordingly modified all the calls to this method in the rest of the code.
In the test scripts, I set truncate=True since it is the default behavior.
As a side note: I've just noticed that the commit I made to sync my fork is listed in my PR. This is not intended, but I don't know how to remove it directly. If the PR gets approved I can eventually drop it while rebasing.
Fixes Issue #157
__call__
ofBatchConverter
and added atruncate
parameter in its constructor.BatchConverter
is accessed through theget_batch_converter
method ofAlphabet
, therefore I added atruncate
parameter to this method.truncate=True
since it is the default behavior.As a side note: I've just noticed that the commit I made to sync my fork is listed in my PR. This is not intended, but I don't know how to remove it directly. If the PR gets approved I can eventually drop it while rebasing.