LED + Buzzer + Beeper

AliceDeh · May 22nd 2015

Grüße,

vorab der Code:

Python

import wiringpi2 as wiringpi  
from time import sleep       # allows us a time delay  


wiringpi.wiringPiSetup()	 # using wiringPi Libary

wiringpi.pinMode(0, 1)      # sets PIN 0 to output  
wiringpi.pinMode(1, 1)      # sets PIN 1 to output 
wiringpi.pinMode(2, 1)      # sets PIN 2 to output 
wiringpi.pinMode(3, 1)      # sets PIN 3 to output 
wiringpi.pinMode(4, 1)      # sets PIN 4 to output 
wiringpi.pinMode(6, 1)		# sets PIN 6 to output 
wiringpi.digitalWrite(0, 0) # sets port 0 to 0 (0V, off)  
wiringpi.digitalWrite(1, 0) # sets port 1 to 0 (0V, off)  
wiringpi.digitalWrite(2, 0) # sets port 2 to 0 (0V, off)  
wiringpi.digitalWrite(3, 0) # sets port 3 to 0 (0V, off)  
wiringpi.digitalWrite(4, 0) # sets port 4 to 0 (0V, off) 
wiringpi.digitalWrite(5, 0)	# sets port 4 to 0 (0V, off) 


wiringpi.pinMode(5, 0)      # sets PIN 5 to input (buzzer)


a = 0


def buzz(pitch, duration):
	period = 1.0
	delay = period / 2
	cycles = 1
	for i in range(cycles):
		wiringpi.digitalWrite(6, True)
		sleep(0.5)
		wiringpi.digitalWrite(6, False)
		sleep(0.5)


try:
	while True:
		if (wiringpi.digitalRead(5)):
			a = a + 1
			sleep(0.18)

			if (a == 1):
				wiringpi.digitalWrite(0, 1)
				pitch = 200
				duration = 1
				buzz(pitch, duration)

			if (a == 2):
				wiringpi.digitalWrite(1, 1)
				pitch = 400
				duration = 2
				buzz(pitch, duration)

			if (a == 3):
				wiringpi.digitalWrite(2, 1)
				pitch = 400
				duration = 3
				buzz(pitch, duration)

			if (a == 4):
				wiringpi.digitalWrite(3, 1)	
				pitch = 600
				duration = 4
				buzz(pitch, duration)

			if (a == 5):
				wiringpi.digitalWrite(4, 1)
				pitch = 200
				duration = 0.05
				buzz(pitch, duration)

			if (a == 6):
				wiringpi.digitalWrite(0, 0)
				pitch = 200
				duration = 0.05
				buzz(pitch, duration)

			if (a == 7):
				wiringpi.digitalWrite(1, 0)
				pitch = 200
				duration = 0.05
				buzz(pitch, duration)

			if (a == 8):
				wiringpi.digitalWrite(2, 0)
				pitch = 200
				duration = 0.05
				buzz(pitch, duration)

			if (a == 9):
				wiringpi.digitalWrite(3, 0)	
				pitch = 200
				duration = 0.05
				buzz(pitch, duration)


			if (a == 10):
				wiringpi.digitalWrite(4, 0)	
				pitch = 200
				duration = 0.05
				buzz(pitch, duration)
				a = 0

		else:  
			sleep(0.05)                      # delay 0.05s  

finally:  # when you CTRL+C exit, we clean up  
		wiringpi.digitalWrite(0, 0) # sets port 0 to 0 (0V, off)  
		wiringpi.digitalWrite(1, 0) # sets port 1 to 0 (0V, off)  
		wiringpi.digitalWrite(2, 0) # sets port 2 to 0 (0V, off)  
		wiringpi.digitalWrite(3, 0) # sets port 3 to 0 (0V, off)  
		wiringpi.digitalWrite(4, 0) # sets port 4 to 0 (0V, off)  
		wiringpi.pinMode(0, 0)      # sets PIN 0 back to input Mode 
		wiringpi.pinMode(1, 0)      # sets PIN 1 back to inp7ut Mode 
		wiringpi.pinMode(2, 0)      # sets PIN 2 back to input Mode 
		wiringpi.pinMode(3, 0)      # sets PIN 3 back to input Mode 
		wiringpi.pinMode(4, 0)      # sets PIN 4 back to input Mode 	
#  PIN 5 is already an input, so no need to change anything

Display More

Wie ihr seht, wird mit jedem Tastendruck auf den Buzzer eine LED angeschaltet und der BiepTon wird ausgelöst. Dafür benutze ich die Funktion "Buzz". Buzz benötigt 'pitch' & 'duration', wie hier in diesem Code-Beispiel.

Code

if (a == 1):
				wiringpi.digitalWrite(0, 1)
				pitch = 200
				duration = 1
				buzz(pitch, duration)

			if (a == 2):
				wiringpi.digitalWrite(1, 1)
				pitch = 400
				duration = 2
				buzz(pitch, duration)

			if (a == 3):
				wiringpi.digitalWrite(2, 1)
				pitch = 400
				duration = 3
				buzz(pitch, duration)

			if (a == 4):
				wiringpi.digitalWrite(3, 1)	
				pitch = 600
				duration = 4
				buzz(pitch, duration)

Display More

Ich erhöhe immer den Pitch und Duration doch es kommt kein hörbarer Unterschied.
Hat dieser "nicht-eintretende" Effekt etwas mit dem:

Code

def buzz(pitch, duration):
	period = 1.0
	delay = period / 2
	cycles = 1
	for i in range(cycles):
		wiringpi.digitalWrite(6, True)
		sleep(0.5)
		wiringpi.digitalWrite(6, False)
		sleep(0.5)

period und delay erscheinen mir ziemlich ungenutzt. Und warum benutze ich 'cycles' wenn ich bei die Dauer mit 'duration' bestimme?
Kennt sich jemand mit der Funktion "Buzz" aus? Benötige eure Hilfe
[hr]
Ich versuche zu realisieren, dass bei jedem Tastendruck und eine weitere LED angeht der Ton umso höher wird pro angeschalteter LED.

Gruß

raspiprojekt · May 22nd 2015

Dein Ansinnen ist mir klar, die Funktio buzz passt nicht. Wie Du erkannt hast, hängt sind pitch und duration sinnlos. Einzig cycle und sleep bestimmen Länge und Höhe des buzzers. Was hast Du am GPIO 6 dran?

Code

def buzz(pitch, duration):

    for i in range(duration):#Hier müsstest Du ausprobieren wie hoch duration sein muß, ich würde aber lieber mit time arbeiten
        wiringpi.digitalWrite(6, True)
        sleep(pitch/1000) #die Dauer des sleep bestimmt die Frequenz pitch = 1 wäre hier 1kHz
        wiringpi.digitalWrite(6, False)
        sleep(pitch/1000)

Wo hast Du die Ursprungsfunktion her?

bootsmann · May 22nd 2015

Dann schau dir mal die Funktion genau an und überlege, wo denn dort die Werte, welche als Parameter im Funktionsaufruf in der Endlosschleife übergeben werden, verwendet werden.

Allgemein ist zu sagen, dass der gezeigte Code in den virtuellen Mülleimer gehört. Offensichtlich hapert es schon an den Grundlagen. Von wo sind denn diese Codeschnipsel übernommen worden?

PS:
Lies dich mal durch bzw arbeite es durch.
http://py-tutorial-de.readthedocs.org/de/python-3.3/

raspiprojekt · May 22nd 2015

Komme gerade aus dem Thread "Gelassenheit". Das war jetzt aber keine Hilfe!

AliceDeh · May 22nd 2015

Quote from raspiprojekt
Dein Ansinnen ist mir klar, die Funktio buzz passt nicht. Wie Du erkannt hast, hängt sind pitch und duration sinnlos. Einzig cycle und sleep bestimmen Länge und Höhe des buzzers. Was hast Du am GPIO 6 dran?
Code
def buzz(pitch, duration):

    for i in range(duration):#Hier müsstest Du ausprobieren wie hoch duration sein muß, ich würde aber lieber mit time arbeiten
        wiringpi.digitalWrite(6, True)
        sleep(pitch/1000) #die Dauer des sleep bestimmt die Frequenz pitch = 1 wäre hier 1kHz
        wiringpi.digitalWrite(6, False)
        sleep(pitch/1000)
Wo hast Du die Ursprungsfunktion her?

An GPIO 6 ist mein Buzzer. Wenn der Strom an PIN 6 fließt geht die LED an.

PS: Übersprungfunktion?
[hr]

Quote from raspiprojekt
Dein Ansinnen ist mir klar, die Funktio buzz passt nicht. Wie Du erkannt hast, hängt sind pitch und duration sinnlos. Einzig cycle und sleep bestimmen Länge und Höhe des buzzers. Was hast Du am GPIO 6 dran?
Code
def buzz(pitch, duration):

    for i in range(duration):#Hier müsstest Du ausprobieren wie hoch duration sein muß, ich würde aber lieber mit time arbeiten
        wiringpi.digitalWrite(6, True)
        sleep(pitch/1000) #die Dauer des sleep bestimmt die Frequenz pitch = 1 wäre hier 1kHz
        wiringpi.digitalWrite(6, False)
        sleep(pitch/1000)
Wo hast Du die Ursprungsfunktion her?

Meinst du so?

Code

def buzz(pitch, duration):
	#period = 1.0 / pitch
	#delay = 1
	#cycles = 1
	duration = 500
	for i in range(duration):
		wiringpi.digitalWrite(6, True)
		sleep(pitch/1000)
		wiringpi.digitalWrite(6, False)
		sleep(pitch/1000)


try:
	while True:
		if (wiringpi.digitalRead(5)):
			a = a + 1
			sleep(0.18)

			if (a == 1):
				wiringpi.digitalWrite(0, 1)
				buzz(pitch, duration)

Display More

Ne oder? Geht nämlich nicht^^
[hr]
Habe ihn jetzt umgeschrieben:

Python

import wiringpi2 as wiringpi  
from time import sleep       # allows us a time delay  


wiringpi.wiringPiSetup()	 # using wiringPi Libary

wiringpi.pinMode(0, 1)      # sets PIN 0 to output  
wiringpi.pinMode(1, 1)      # sets PIN 1 to output 
wiringpi.pinMode(2, 1)      # sets PIN 2 to output 
wiringpi.pinMode(3, 1)      # sets PIN 3 to output 
wiringpi.pinMode(4, 1)      # sets PIN 4 to output 
wiringpi.pinMode(6, 1)		# sets PIN 6 to output 
wiringpi.digitalWrite(0, 0) # sets port 0 to 0 (0V, off)  
wiringpi.digitalWrite(1, 0) # sets port 1 to 0 (0V, off)  
wiringpi.digitalWrite(2, 0) # sets port 2 to 0 (0V, off)  
wiringpi.digitalWrite(3, 0) # sets port 3 to 0 (0V, off)  
wiringpi.digitalWrite(4, 0) # sets port 4 to 0 (0V, off) 
wiringpi.digitalWrite(5, 0)	# sets port 4 to 0 (0V, off) 


wiringpi.pinMode(5, 0)      # sets PIN 5 to input (buzzer)


a = 0


def buzz(pitch, duration):
	for i in range(duration):
		wiringpi.digitalWrite(6, True)
		sleep(1)
		wiringpi.digitalWrite(6, False)
		sleep(1)


try:
	while True:
		if (wiringpi.digitalRead(5)):
			a = a + 1
			sleep(0.18)

			if (a == 1):
				wiringpi.digitalWrite(0, 1)
				pitch = 200
				duration = 1
				buzz(pitch, duration)

			if (a == 2):
				wiringpi.digitalWrite(1, 1)
				pitch = 400
				duration = 1
				buzz(pitch, duration)

			if (a == 3):
				wiringpi.digitalWrite(2, 1)
				pitch = 600
				duration = 1
				buzz(pitch, duration)

			if (a == 4):
				wiringpi.digitalWrite(3, 1)	
				pitch = 800
				duration = 1
				buzz(pitch, duration)

			if (a == 5):
				wiringpi.digitalWrite(4, 1)
				pitch = 1000
				duration = 1
				buzz(pitch, duration)

			if (a == 6):
				wiringpi.digitalWrite(0, 0)
				pitch = 800
				duration = 1
				buzz(pitch, duration)

			if (a == 7):
				wiringpi.digitalWrite(1, 0)
				pitch = 600
				duration = 1
				buzz(pitch, duration)

			if (a == 8):
				wiringpi.digitalWrite(2, 0)
				pitch = 400
				duration = 1
				buzz(pitch, duration)

			if (a == 9):
				wiringpi.digitalWrite(3, 0)	
				pitch = 200
				duration = 1
				buzz(pitch, duration)

			if (a == 10):
				wiringpi.digitalWrite(4, 0)	
				pitch = 200
				duration = 1
				buzz(pitch, duration)
				a = 0

		else:  
			sleep(0.05)                      # delay 0.05s  

finally:  # when you CTRL+C exit, we clean up  
		wiringpi.digitalWrite(0, 0) # sets port 0 to 0 (0V, off)  
		wiringpi.digitalWrite(1, 0) # sets port 1 to 0 (0V, off)  
		wiringpi.digitalWrite(2, 0) # sets port 2 to 0 (0V, off)  
		wiringpi.digitalWrite(3, 0) # sets port 3 to 0 (0V, off)  
		wiringpi.digitalWrite(4, 0) # sets port 4 to 0 (0V, off)  
		wiringpi.pinMode(0, 0)      # sets PIN 0 back to input Mode 
		wiringpi.pinMode(1, 0)      # sets PIN 1 back to inp7ut Mode 
		wiringpi.pinMode(2, 0)      # sets PIN 2 back to input Mode 
		wiringpi.pinMode(3, 0)      # sets PIN 3 back to input Mode 
		wiringpi.pinMode(4, 0)      # sets PIN 4 back to input Mode 	
#  PIN 5 is already an input, so no need to change anything

Display More

Die Tonlage bleibt dennoch unverändert

bootsmann · May 22nd 2015

Offensichtlich kommt der Code von hier

AliceDeh
1. du sollst Codewiederholung meiden. Dies wurde dir schon im anderen Strang mitgeteilt
2. ``if`` ist eine Anweisung und keine Funktion. Also lass die Klammern da weg.
3. schreibe aussagekräftige Name. ``a = 1`` was soll a sein?? In Python gibt es eine Namenskonvention an die man sich halten muss/soll
4. wie oben erwähnt, ist es besser, sich zuerst mit den Grundlagen vertraut zu machen, bevor man sich Hals über Kopf in grössere komplexere Projkte stürzt. Dazu noch ein bekannter Artikel.

Python

#!/usr/bin/env python
# coding: utf-8


from __future__ import print_function, division
import wiringpi2 as wiringpi




PINS = PORTS = range(6)




def setup_wringpi():
    wiringpi.wiringPiSetup()
    for pin in PINS:
        wiringpi.pinMode(pin, 1)
    for port in PORTS:
        wiringpi.digitalWrite(port, 0)




def buzz(pitch, duration):
    period = 1 / pitch
    delay = period / 2
    cycles = int(duration * pitch)
    for i in range(cycles):
        wiringpi.digitalWrite(6, True)
        wiringpi.delay(delay)
        wiringpi.digitalWrite(6, False)




def main():
    setup_wringpi()
    buzz(200, 1000)  # 1000 = 1sec laut Doku von wringpi




if __name__ == '__main__':
    main()

Display More

AliceDeh · May 22nd 2015

Quote from bootsmann
Offensichtlich kommt der Code von hier

AliceDeh
1. du sollst Codewiederholung meiden. Dies wurde dir schon im anderen Strang mitgeteilt
2. ``if`` ist eine Anweisung und keine Funktion. Also lass die Klammern da weg.
3. schreibe aussagekräftige Name. ``a = 1`` was soll a sein?? In Python gibt es eine Namenskonvention an die man sich halten muss/soll
4. wie oben erwähnt, ist es besser, sich zuerst mit den Grundlagen vertraut zu machen, bevor man sich Hals über Kopf in grössere komplexere Projkte stürzt. Dazu noch ein bekannter Artikel.
Python
#!/usr/bin/env python
# coding: utf-8


from __future__ import print_function, division
import wiringpi2 as wiringpi




PINS = PORTS = range(6)




def setup_wringpi():
    wiringpi.wiringPiSetup()
    for pin in PINS:
        wiringpi.pinMode(pin, 1)
    for port in PORTS:
        wiringpi.digitalWrite(port, 0)




def buzz(pitch, duration):
    period = 1 / pitch
    delay = period / 2
    cycles = int(duration * pitch)
    for i in range(cycles):
        wiringpi.digitalWrite(6, True)
        wiringpi.delay(delay)
        wiringpi.digitalWrite(6, False)




def main():
    setup_wringpi()
    buzz(200, 1000)  # 1000 = 1sec laut Doku von wringpi




if __name__ == '__main__':
    main()
Display More
Display More

Sehr sauberer Code auf jeden Fall, aber dennoch erscheint ein Fehler:

'Traceback (most recent call last):
File "led_5_reihe_taster_six.py", line 35, in <module>
main()
File "led_5_reihe_taster_six.py", line 31, in main
buzz(200, 1000) # 1000 = 1sec laut Doku von wringpi
File "led_5_reihe_taster_six.py", line 25, in buzz
wiringpi.delay(delay)
TypeError: in method 'delay', argument 1 of type 'unsigned int'
'

Quote

if __name__ == '__main__':
main()

Den versteh ich gar nicht Hier geht es auch nur um den Pieper.
In deinem Skript wurde mein Buzzer (Taster) ignoiert.
[hr]
Nochmal:

Python

import wiringpi2 as wiringpi  
from time import sleep       # allows us a time delay  


wiringpi.wiringPiSetup()	 # using wiringPi Libary

wiringpi.pinMode(0, 1)      # sets PIN 0 to output  
wiringpi.pinMode(1, 1)      # sets PIN 1 to output 
wiringpi.pinMode(2, 1)      # sets PIN 2 to output 
wiringpi.pinMode(3, 1)      # sets PIN 3 to output 
wiringpi.pinMode(4, 1)      # sets PIN 4 to output 
wiringpi.pinMode(6, 1)		# sets PIN 6 to output 
wiringpi.digitalWrite(0, 0) # sets port 0 to 0 (0V, off)  
wiringpi.digitalWrite(1, 0) # sets port 1 to 0 (0V, off)  
wiringpi.digitalWrite(2, 0) # sets port 2 to 0 (0V, off)  
wiringpi.digitalWrite(3, 0) # sets port 3 to 0 (0V, off)  
wiringpi.digitalWrite(4, 0) # sets port 4 to 0 (0V, off) 
wiringpi.digitalWrite(5, 0)	# sets port 4 to 0 (0V, off) 


wiringpi.pinMode(5, 0)      # sets PIN 5 to input (buzzer)


a = 0


def buzz(pitch, duration):
    period = 1 / pitch
    delay = 1000
    cycles = 1
    for i in range(cycles):
        wiringpi.digitalWrite(6, True)
        wiringpi.delay(delay)
        wiringpi.digitalWrite(6, False)


try:
	while True:
		if (wiringpi.digitalRead(5)):
			a = a + 1
			sleep(0.18)

			if a == 1:
				wiringpi.digitalWrite(0, 1)
				buzz(200, 1000)

			if (a == 2):
				wiringpi.digitalWrite(1, 1)
				buzz(400, 1000)

			if (a == 3):
				wiringpi.digitalWrite(2, 1)
				buzz(600, 1000)

			if (a == 4):
				wiringpi.digitalWrite(3, 1)	
				buzz(800, 1000)

			if (a == 5):
				wiringpi.digitalWrite(4, 1)
				buzz(pitch, duration)

			if (a == 6):
				wiringpi.digitalWrite(0, 0)
				buzz(pitch, duration)

			if (a == 7):
				wiringpi.digitalWrite(1, 0)
				buzz(pitch, duration)

			if (a == 8):
				wiringpi.digitalWrite(2, 0)
				buzz(pitch, duration)

			if (a == 9):
				wiringpi.digitalWrite(3, 0)	
				buzz(pitch, duration)

			if (a == 10):
				wiringpi.digitalWrite(4, 0)	
				buzz(pitch, duration)
				a = 0

		else:  
			sleep(0.05)                      # delay 0.05s  

finally:  # when you CTRL+C exit, we clean up  
		wiringpi.digitalWrite(0, 0) # sets port 0 to 0 (0V, off)  
		wiringpi.digitalWrite(1, 0) # sets port 1 to 0 (0V, off)  
		wiringpi.digitalWrite(2, 0) # sets port 2 to 0 (0V, off)  
		wiringpi.digitalWrite(3, 0) # sets port 3 to 0 (0V, off)  
		wiringpi.digitalWrite(4, 0) # sets port 4 to 0 (0V, off)  
		wiringpi.pinMode(0, 0)      # sets PIN 0 back to input Mode 
		wiringpi.pinMode(1, 0)      # sets PIN 1 back to inp7ut Mode 
		wiringpi.pinMode(2, 0)      # sets PIN 2 back to input Mode 
		wiringpi.pinMode(3, 0)      # sets PIN 3 back to input Mode 
		wiringpi.pinMode(4, 0)      # sets PIN 4 back to input Mode 	
#  PIN 5 is already an input, so no need to change anything

Display More

Mein Buzzer funktioniert und die LED's leuchten, doch ändert der Buzzer sein Tonlage nicht.

Wer kann mir helfen?

bootsmann · May 22nd 2015

AliceDeh

Dann nimm anstelle von ``wiringpi.delay(delay)`` ``time.sleep(delay)``
Ich kenn das wiringpi2 Modul nicht. Hab nur schnell auf github ein paar Beispiele angeschaut...

AliceDeh · May 22nd 2015

Hier nochmal dein Code:

Python

#!/usr/bin/env python
# coding: utf-8


from __future__ import print_function, division
import wiringpi2 as wiringpi




PINS = PORTS = range(6)




def setup_wringpi():
    wiringpi.wiringPiSetup()
    for pin in PINS:
        wiringpi.pinMode(pin, 1)
    for port in PORTS:
        wiringpi.digitalWrite(port, 0)




def buzz(pitch, duration):
    period = 1 / pitch
    delay = period / 2
    cycles = int(duration * pitch)
    for i in range(cycles):
        wiringpi.digitalWrite(6, True)
        time.sleep(delay)
        wiringpi.digitalWrite(6, False)




def main():
    setup_wringpi()
    buzz(200, 1000)  # 1000 = 1sec laut Doku von wringpi




if __name__ == '__main__':
    main()

Display More

Der erneute Fehler:

Traceback (most recent call last):
File "led_5_reihe_taster_six.py", line 35, in <module>
main()
File "led_5_reihe_taster_six.py", line 31, in main
buzz(200, 1000) # 1000 = 1sec laut Doku von wringpi
File "led_5_reihe_taster_six.py", line 25, in buzz
time.sleep(delay)
NameError: global name 'time' is not defined

Funktioniert leider auch nicht.
Komische Funktion .. man übergibt 'pitch' und es wird einfach nichts geändert. Das kenne ich so aus dem Unterricht auch nicht ..
Muss mir wohl oder übel ein neues Projekt einfallen lassen. Meine Idee scheint nicht umsetzbar zu sein.

Danke euch trzdem für die Hilfe bis dahin