dwks / silvius

Kaldi-based speech recognition system + grammar
http://voxhub.io/silvius
BSD 2-Clause "Simplified" License
101 stars 28 forks source link

use map in CLIClickAutomator.transform_key; numbers fix #27

Closed jvanloov closed 6 years ago

jvanloov commented 6 years ago

I agree with your comment that CLIClickAutomator should use a map; commit 86f30c8 implements this.

It seems there is an issue with the recently merged "Support numbers..." commit. The numbers are composed correctly, but e.g. "twenty five" outputs as "xdotool key 25" instead of "xdotool 2 5". Changing the AST node to "sequence" (from "char") fixes this (commit effd698 implements this.

dwks commented 6 years ago

Thank you very much. The issue with the recent numbers code resulted from my merge, I like your fix for it though.