-
Notifications
You must be signed in to change notification settings - Fork 107
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
macro detection #27
base: main
Are you sure you want to change the base?
macro detection #27
Conversation
569fdae
to
4a922ad
Compare
Hi, I like the idea of being able to use macros! But could not get your code to work. I was trying to update it to the newest version as well, but ran into an issue. Likely because I'm not a Python pro. Can't seem to find where the Line 465 in 7b880d6
I was hoping to be able to merge @asamwow code with the latest from @ideasman42. Is there still hope for this to move forward, I'd be happy to lend as much of a hand as I can... |
@ecotechie hi, so glad for the interest! I figured macros would be pretty user specific, so I left the function mostly empty except for test code. |
Thanks @asamwow, I do see where the nerd_dictation_macro_process() function is being defined. What I don't see anywhere in the code is a I'm not sure what error you are referring to though, there is none, the function just isn't called, since it's not defined. Or, I'm a space cadet and am missing something very basic. |
Line 461 in 7b880d6
|
Ohh, I didn't know that was a way to define functions in python. 🤯 I'm going to try a bit more, but will likely step aside and let more python oriented devs sort this one out. I'm more of a PHP kinda guy 😄 If I can get something updated with the newer code to work, I'll do a pull request. Thanks for your patience... |
d56e61e
to
67eec6c
Compare
f72c0d1
to
5f3b3e9
Compare
2436242
to
72eef57
Compare
No description provided.