Luke Carlson

Results 4 comments of Luke Carlson

Hi are you still seeing these issues? (I've updated main since the original issue was opened)

Hey Pedro 1) looks like we ended up using two different names to refer to the same thing — we should probably rename them to both use `vocab_file` as the...

Because the test case for `MLP_MLX` passes, you could start by just merging in that class implantation? If you'd like to merge others, we'd want to make similar test cases...

Hey how's this one going? Feel free to reach out with any questions