andreselizondo-adestech
andreselizondo-adestech
TF 2.2.0 has now been released. Testing should be done to make sure there were no breaking changes. https://github.com/tensorflow/tensorflow/releases
@emphasize Sorry for the delay. Indeed, adding a '/' solves the problem for `.pb` models, but that's not the case for Keras models. The solution to this would be to...
@MatthewScholefield I think I've finished the things that were pending. What's missing? How can I help?
@usimarit Hello! I'm interested in seeing support for the Streaming Conformer Transducer. Is there an ETA for this? How can I help? I'm no expert, but I might be able...
This PR is aimed to advance the TODO list on #14 .
@usimarit Hello! I've run into a problem. The Streaming Conformer Transducer (SCT) paper states that we need to convert the depthwise convolution inside the "ConvModel" to a "causal" depthwise convolution....
I noticed you're using a `DepthwiseConv2D` with `kernel_size=(32,1)`. Would you consider replacing this by a `SeparableConv1D` with `kernel_size=(32)`? We would also need to specify a value for `filters`, I'd guess...
Good news @usimarit, most of the initial work is done. The model is now trainable, though a few things are still missing. Please take a look and let me know...
> @andreselizondo-adestech We will have a big change in the repo structure as in the PR #177. Please be aware of that 😄 These changes will split the conformer file...
@usimarit I'm merging my changes into the refactored code, however.. there appears to be an issue using `SentencePiece` for training. Specifically at line [tensorflow_asr/featurizers/text_featurizers.py#L342](https://github.com/TensorSpeech/TensorFlowASR/blob/main/tensorflow_asr/featurizers/text_featurizers.py#L342). Seems like this function is nowhere...