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

Add buzzer sounds for brew steps #99

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

Conversation

cocothenut
Copy link

add buzzer-beep-sounds for brew-steps

buzzer-sounds beeps at end of steps

@Manuel83
Copy link
Owner

I started merging the other pull request and now this pull request has conflicts. Could you please resolve the conflicts?

Thanks a lot for providing this pull request. I'm looking forward to merge it.

@cocothenut
Copy link
Author

i had modify the files
must be work now

# if you dont want a beep sound comment out like : # cbpi.BoilStepEndBeep()
cbpi.BoilStepEndBeep()
self.next()
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Extra space here

# if you dont want a beep sound comment out like : # cbpi.MashStepEndBeep()
cbpi.MashStepEndBeep()
self.next()
self.next()
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Extra space here

@@ -11,7 +11,7 @@ class Buzzer(object):

# custom beep sounds

sound = ["H", 0.1, "L"]
sound = ["H", 0.1, "L", 0.1, "H", 0.1, "L", 0.1, "H", 0.1, "L"]
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Your "sound" may need to match the newer one.

@cocothenut
Copy link
Author

thanks for the help
seems my notepad++ makes different spaces by insert

may be it works now

@Manuel83
Copy link
Owner

Manuel83 commented Jul 24, 2017

One conflict missing

modules/base_plugins/brew_steps/init.py

The question is if I should add a central beep in the step logic instead of a beep in each step?

@cocothenut
Copy link
Author

Hi
I would add additional various buzzer-beep-sounds for various brew-steps

@Philusha1983
Copy link

Philusha1983 commented Jul 30, 2017 via email

@swimIan
Copy link
Contributor

swimIan commented Aug 9, 2017

Would love to make a hop dropper! Any leads...

@Philusha1983
Copy link

Philusha1983 commented Aug 9, 2017 via email

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.

4 participants