RASR FSA Builder #59
RASR FSA Builder #59
Conversation
|
I'm not sure about a test for this. It does require the librasr package, which I'm not sure if we should put it into the requirements and to initialize the |
|
You "could" add extra dependencies just for a test, including a small dummy lexicon and corpus for testing. While this would be preferable to have, I am not sure if this is within your time-scope. |
|
I would like to do it, actually. Having a small dummy corpus and lexicon is a good idea. However, the installation of |
i6_models/parts/fsa.py
Outdated
| fsa by simple left-multiplication and moving the tensors to a different device. | ||
| It can simply be passed to `i6_native_ops.fbw.fbw_loss` and `i6_native_ops.fast_viterbi.align_viterbi`. | ||
| :param num_states: the total number of all states S | ||
| :param edges: a [4, E] tensor of edges where each column is an edge consisting |
There was a problem hiding this comment.
Nitpick: E (I guess number of edges) is undefined here.
Co-authored-by: michelwi <[email protected]>
Co-authored-by: Albert Zeyer <[email protected]>
Co-authored-by: Albert Zeyer <[email protected]>
Components for building FSAs with the
librasrpython package and using them in a PyTorch training