You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@mxnet.apache.org by "Stephen Rawls (JIRA)" <ji...@apache.org> on 2019/01/14 07:17:00 UTC
[jira] [Commented] (MXNET-1295) Adding support for integer indexes
in SequenceMask, SequenceReverse, SequenceLast
[ https://issues.apache.org/jira/browse/MXNET-1295?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16741824#comment-16741824 ]
Stephen Rawls commented on MXNET-1295:
--------------------------------------
I am currently testing out a patch, will try to put in a PR in next few days.
> Adding support for integer indexes in SequenceMask, SequenceReverse, SequenceLast
> ---------------------------------------------------------------------------------
>
> Key: MXNET-1295
> URL: https://issues.apache.org/jira/browse/MXNET-1295
> Project: Apache MXNet
> Issue Type: Improvement
> Components: Apache MXNet Backend
> Reporter: Stephen Rawls
> Priority: Minor
>
> Currently the Sequence* family of operators, all of which take in an input array and optionally an array of valid sequence_length's, require the sequence_length array to have the same dtype as the input array.
> If users have their sequence_length array stored as an int32 array, which is natural for storing indexes, they must first convert/cast their array to float32 before being able to use it on float32 input arrays.
> Other similar operators, e.g. take, do not have this requirement.
> Additionally, it looks like the CTC operator is using an alternative implementation of SequenceMask that also doesn't have this limitation (but is not registered as an MxNet operator and so is unusable outside of the backend codebase).
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@mxnet.apache.org
For additional commands, e-mail: issues-help@mxnet.apache.org