Skip to content
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

Consume values in channel #1

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

jvtrigueros
Copy link

The some-external-input channel only has a buffer of 1, so whenever we type one thing on the text box, the handler function would block the UI. Executing the go-loop will consume from the channel indefinitely.

As an aside, the note is being played on the transducer, but perhaps it makes more sense that it gets played in the go-loop block as it feels like a side effect. Such that you're "pushing" notes onto the channel and playing the notes when "pulled".

@gzmask
Copy link
Owner

gzmask commented Mar 21, 2020

great insight and thanks for looking into this! I tested it and still doesn't work. In my scenario, I started the seesaw frame within emacs, and display the seesaw frame with the keypress listener. However the keypress event still goes back to emacs. Then I tested again starting the app with lein run, and the keypress event go back to terminal instead of triggering the handler function. I wonder what is the expected way to start a seesaw app.

@jvtrigueros
Copy link
Author

Granted, I tested it without overtone, on Windows, and IntelliJ so perhaps it worked by coincidence 😅

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants