Travis Bartley

Wrap LSTM Outputs in own module class

11

Current LSTM outputs create ugly typing as mentioned in https://github.com/CUNY-CL/yoyodyne/pull/195. Replace with a dataclass wrapper to make eyes not bleed.

adding expert to hp ignore due to pickeling issue

6

Fixes: https://github.com/CUNY-CL/yoyodyne/issues/192 TQDM in the transducer's expert module doesn't pickle currently. So when doing multi-gpu training, pytorch gets annoyed. Since it's not really needed beyond training, I'm dropping it from...

Generalization of `expert` `teacher_forcing` and `monotonicity` across model architectures

5

Something I've been thinking about with expansion of library: a decent amount of the work we've been using involves application of inductive biases and teacher-prompted training to model architecture. Currently...

TQDM Error with multi GPU Transducer

1

Issue when running multi-gpu training with edit action transducer: ``` Traceback (most recent call last): File "/home/salamander/anaconda3/envs/sigmorphon2024/bin/yoyodyne-train", line 8, in sys.exit(main()) File "/home/salamander/anaconda3/envs/sigmorphon2024/lib/python3.10/site-packages/yoyodyne/train.py", line 390, in main model = get_model_from_argparse_args(args,...

Seen in https://github.com/CUNY-CL/yoyodyne/pull/140 , current setup for expert needing to learn vocab separate from dataloader is silly and inconsistent with other modules. Should change so the expert uses the vocab...

adding fixes so transducer can work again

1

Merges: https://github.com/CUNY-CL/yoyodyne/pull/233 https://github.com/CUNY-CL/yoyodyne/pull/197 Fixes: https://github.com/CUNY-CL/yoyodyne/issues/192 https://github.com/CUNY-CL/yoyodyne/issues/191 Dependent on: https://github.com/CUNY-CL/maxwell/pull/17 Summary: I fixed maxwell so TQDM isn't a property of the SED parameters anymore, this allows pickling of the expert module...

With https://github.com/CUNY-CL/yoyodyne/pull/247, most of the work in instantiating edit actions for the transducer is coming from the Index. It would make more sense to just make this inherited behavior from...