Buffer Python Bluetooth

Version imprimable

Bonjour à tous,

Je cherche à allumer une led en fonction des commandes reçus par bluetooth, le code fonctionne, cependant j'ai quelque erreur quand le buffer est déja plein.
Voici mon code :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
import io
 
import serial
import RPi.GPIO as GPIO
 
 
PORT = "/dev/rfcomm0"
BAUDRATE = 9600
TIMEOUT = 0.1
ENCODING = "Utf-8"
LED_PIN = 22
 
try:
    GPIO.setmode(GPIO.BOARD)
    GPIO.setwarnings(False)
    GPIO.setup(LED_PIN, GPIO.OUT)
    etat = 0
 
    while True :
 
        ser = serial.Serial(PORT, BAUDRATE, timeout=TIMEOUT)
        buffer = io.BufferedReader(ser)
        with io.TextIOWrapper(buffer, encoding=ENCODING) as file:
 
            for line in file:
                # print(ser.in_waiting)
                # if ser.in_waiting > 0:
                    if len(line)>0 :
 
                        line = line.strip().lower()
                        print(line)
                        print("line :",line.encode("hex"))
 
                        if line == "on":
                            GPIO.output(LED_PIN, GPIO.HIGH)
                            etat = 1
                        elif line == "off":
                            GPIO.output(LED_PIN, GPIO.LOW)
                            etat = 0
        if etat == 1 : GPIO.output(LED_PIN, GPIO.HIGH)
 
finally:
    GPIO.cleanup()

Je veux utiliser la fonction ser_waiting, mais le problème est qu'elle me renvoie toujours 0...

Si certains ont des suggestions, merci d'avance.

09/05/2022, 16h49
wiztricks

Citation:

Envoyé par guillaume1937

Je cherche à allumer une led en fonction des commandes reçus par bluetooth, le code fonctionne, cependant j'ai quelque erreur quand le buffer est déja plein.

Elles ressemblent à quoi vos commandes? Car normalement ce devraient être des lignes (terminées par "fin-de-ligne")... et votre seul "buffer" devrait être la ligne lue.

- W
09/05/2022, 16h53
guillaume1937

Citation:

Envoyé par wiztricks

Elles ressemblent à quoi vos commandes? Car normalement ce devraient être des lignes (terminées par "fin-de-ligne")... et votre seul "buffer" devrait être la ligne lue.

- W

Les commandes c'est soit du "on" soit "off", grâce à la commande strip, on ne garde que le mot, mais je ne comprends pas pouquoi la fonction renvoie toujours 0

Code:

ser.in_waiting
09/05/2022, 18h40
wiztricks

Citation:

Envoyé par guillaume1937

Les commandes c'est soit du "on" soit "off", grâce à la commande strip, on ne garde que le mot

ça ne répond as à ma question qui était de savoir si les commandes se terminaient ou pas par "fin de ligne".

Citation:

Envoyé par guillaume1937

mais je ne comprends pas pouquoi la fonction renvoie toujours 0

Code:

ser.in_waiting

Si la logique est de lire tout ce qui arrive... pourquoi resterait-il des octets à lire?

- W
09/05/2022, 18h46
guillaume1937

Citation:

Envoyé par wiztricks

ça ne répond as à ma question qui était de savoir si les commandes se terminaient ou pas par "fin de ligne".

Si la logique est de lire tout ce qui arrive... pourquoi resterait-il des octets à lire?

- W

Oui les commandes se termine par \n, mais là le problème n'est pas dans la condition, puisque j'arrive à allumer la LED, c'est que parfois le programme ne prend pas en compte la commande que j'envoie, et je suppose qu'il y a un problème avec le buffer, et je ne comprends pas pourquoi la fonction waiting renvoie tout le temps 0 alors que ça ne devrait pas être le cas
09/05/2022, 19h50
wiztricks

Citation:

Envoyé par guillaume1937

je suppose qu'il y a un problème avec le buffer, et je ne comprends pas pourquoi la fonction waiting renvoie tout le temps 0 alors que ça ne devrait pas être le cas

Si les commandes reçues se terminent par une fin de ligne, il serait plus simple de les récupérer avec serial.readline(...) et de définir un timeout pour ne pas y rester coincé indéfiniment.
note: "plus simple" <=> un code plus facile à comprendre et à mettre au point.

La fonction serial.in_waiting retourne 0 lorsqu'il n'y a rien en attente. Ce qui me semble cohérent avec le code qui a été écrit.

Citation:

Envoyé par guillaume1937

c'est que parfois le programme ne prend pas en compte la commande que j'envoie, et je suppose qu'il y a un problème avec le buffer,

Si vous pensez qu'il y a un problème avec le BufferedReader, virez le, il ne sert à rien.
Et si le problème est toujours là, vous saurez que ça vient peut être d'ailleurs.

- W
09/05/2022, 21h44
guillaume1937

Citation:

Envoyé par wiztricks

Si les commandes reçues se terminent par une fin de ligne, il serait plus simple de les récupérer avec serial.readline(...) et de définir un timeout pour ne pas y rester coincé indéfiniment.
note: "plus simple" <=> un code plus facile à comprendre et à mettre au point.

La fonction serial.in_waiting retourne 0 lorsqu'il n'y a rien en attente. Ce qui me semble cohérent avec le code qui a été écrit.

Si vous pensez qu'il y a un problème avec le BufferedReader, virez le, il ne sert à rien.
Et si le problème est toujours là, vous saurez que ça vient peut être d'ailleurs.

- W

Oui le bufferedReader ne sert pas à grand chose, c'était surtout pour essayer de mettre en œuvre un buzzer. Et justement pour le TIMEOUT, quelle valeur lui donner ? Parce que j'ai un peu fait au hasard.
09/05/2022, 23h26
wiztricks

Citation:

Envoyé par guillaume1937

Et justement pour le TIMEOUT, quelle valeur lui donner ? Parce que j'ai un peu fait au hasard.

Avec une valeur trop petite, ça va sortir en timeout trop tôt... trop grande, ça va attendre pour rien: à vous de voir (en testant).

- W
10/05/2022, 09h48
jurassic pork

Hello,

Citation:

Envoyé par guillaume1937

Les commandes c'est soit du "on" soit "off", grâce à la commande strip, on ne garde que le mot, mais je ne comprends pas pouquoi la fonction renvoie toujours 0

Code:

ser.in_waiting

Pourrait-on voir une ligne de commande complète ? Si il y a un crlf à la fin, plus la peine de gérer le buffer d'entrée, un readline est suffisant.
Ami calmant, J.P

Citation:

Envoyé par jurassic pork

Hello,

Pourrait-on voir une ligne de commande complète ? Si il y a un crlf à la fin, plus la peine de gérer le buffer d'entrée, un readline est suffisant.
Ami calmant, J.P

Qu'entendez vous par crlf ?

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
#!/usr/bin/env python
# -*- coding: latin-1 -*-
 
#!/usr/bin/env python3
 
import io
import serial
import RPi.GPIO as GPIO
import time
 
 
PORT = "/dev/rfcomm0"
BAUDRATE = 9200
TIMEOUT = 0.5
ENCODING = "Utf-8"
LED_PIN = 22
 
try:
    GPIO.setmode(GPIO.BOARD)
    GPIO.setwarnings(False)
    GPIO.setup(LED_PIN, GPIO.OUT)
    etat = 0
 
    while True :
 
        ser = serial.Serial(PORT, BAUDRATE, timeout=TIMEOUT)
        #print(ser.inWaiting())
        buffer = io.BufferedReader(ser)
        with io.TextIOWrapper(buffer, encoding=ENCODING) as file:
            for line in file:     
                #if ser.inWaiting()>= 0:
                    line = line.strip().lower()
                    print(line)
                    if len(line)>0 :
                        print("line :",line.encode("hex"))                   
                        if line == "on":
                            GPIO.output(LED_PIN, GPIO.HIGH)
                            etat = 1
                        elif line == "off":
                            GPIO.output(LED_PIN, GPIO.LOW)
                            etat = 0
        if etat == 1 : GPIO.output(LED_PIN, GPIO.HIGH)
 
finally:
    GPIO.cleanup()

Et j'ai parfois cette erreur :

Code:

1
2
3
4
5
6
7
  File "LED_buffer.py", line 30, in <module>
    for line in file:
  File "/usr/lib/python2.7/dist-packages/serial/serialutil.py", line 545, in readinto
    data = self.read(len(b))
  File "/usr/lib/python2.7/dist-packages/serial/serialposix.py", line 501, in read
    'device reports readiness to read but returned no data '
serial.serialutil.SerialException: device reports readiness to read but returned no data (device disconnected or multiple access on port?)

Oui mais au final j'avais également fait ce programme là et le problème était le même...
Peut être que ça vient du timeout ? j'ai essayé plusieurs valeurs, mais ça change juste le délais, et disons que je reçoit 8 commandes sur 9...

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
#!/usr/bin/env python
# -*- coding: latin-1 -*-
 
import serial
import RPi.GPIO as GPIO
import time
 
ser = serial.Serial('/dev/rfcomm0', 9600, timeout = 0.1)
GPIO.setmode(GPIO.BOARD) #Définit le mode de numérotation (Board)
GPIO.setwarnings(False) #On désactive les messages d'alerte
LED = 22
GPIO.setup(LED, GPIO.OUT) #Active le contrôle du GPIO  
etat = 0
 
while True:
 
        msg = (ser.readline().decode('utf-8')).strip().lower()
        #print(msg)
        #if len(msg) > 0 : print(msg)      
        if msg in ['on', 'ON']:
            print("LED on")
            GPIO.output(LED, GPIO.HIGH) #On allume
            etat = 1
        elif msg in ['off', 'OFF']:
            print("LED off")
            GPIO.output(LED, GPIO.LOW)
            etat = 0 
            if msg == "" : msg = 'off'
        elif  (msg == "" and etat ==1) : GPIO.output(LED, GPIO.HIGH)

10/05/2022, 10h39
jurassic pork

Citation:

Envoyé par guillaume1937

Qu'entendez vous par crlf ?

crlf = carriage return line feed = retour à la ligne = chr(13) + chr(10)
Si les trames que tu reçois possèdent ces valeurs à chaque ligne de commande alors le readline est la meilleure option pour lire.
10/05/2022, 10h45
guillaume1937

Citation:

Envoyé par jurassic pork

crlf = carriage return line feed = retour à la ligne = chr(13) + chr(10)
Si les trames que tu reçois possèdent ces valeurs à chaque ligne de commande alors le readline est la meilleure option pour lire.

Oui le \n, mais je le prends quand même en compte, comme dans les 2 programmes, mais que ce soit avec le buffer ou le readline j'ai le même problème.
10/05/2022, 11h02
jurassic pork

pas besoin de timeout avec le readline

Citation:

The timeout only applies to read() and does not apply to readline().
10/05/2022, 11h22
guillaume1937

Citation:

Envoyé par jurassic pork

pas besoin de timeout avec le readline

Pourtant quand j'enlève le timeout ou que je le fixe à 0, plus rien ne fonctionne...
10/05/2022, 11h33
jurassic pork

attention timeout = 0 c'est différent de pas de timeout ( None ou pas défini)
10/05/2022, 11h38
guillaume1937

Citation:

Envoyé par jurassic pork

attention timeout = 0 c'est différent de pas de timeout ( None ou pas défini)

Justement quand je ne définie pas, ou que je mets None, cela ne fonctionne plus.
10/05/2022, 11h49
jurassic pork

avec le readline ?
montre le code

Citation:

Envoyé par jurassic pork

avec le readline ?
montre le code

En tapant la commande minicom pour regarder ce qu'il se passe sur le port, je vois que quand je reçois bien la donnée sur mon programme, je ne vois rien sur le port, alors que si je la reçoit pas, elle s'affiche bien sur mon port.

code :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
#!/usr/bin/env python
# -*- coding: latin-1 -*-
 
#!/usr/bin/env python3
 
import io
import serial
import RPi.GPIO as GPIO
import time
 
 
PORT = "/dev/rfcomm0"
BAUDRATE = 9200
TIMEOUT = 0.05
ENCODING = "Utf-8"
LED_PIN = 22
 
try:
    GPIO.setmode(GPIO.BOARD)
    GPIO.setwarnings(False)
    GPIO.setup(LED_PIN, GPIO.OUT)
    etat = 0
 
    while True :
        ser = serial.Serial(PORT, BAUDRATE, timeout=TIMEOUT)
        #print(ser.inWaiting())
        #buffer = io.BufferedReader(ser)
        sio = io.TextIOWrapper(io.BufferedRWPair(ser, ser))
        sio.flush()
        line = sio.readline().decode('utf-8').strip().lower()
        if len(line)>0 :
            print(line)
            print("line :",line.encode("hex"))                   
            if line in ['on', 'on\r', 'on\n',('on\r\n')]:
                GPIO.output(LED_PIN, GPIO.HIGH)
                etat = 1
            elif line in ['off', 'off\r', 'off\n','off\r\n']:
                GPIO.output(LED_PIN, GPIO.LOW)
                etat = 0
        if etat == 1 : GPIO.output(LED_PIN, GPIO.HIGH)
 
finally:
    GPIO.cleanup()

10/05/2022, 11h56
wiztricks

Salut,

Citation:

Envoyé par guillaume1937

Oui mais au final j'avais également fait ce programme là et le problème était le même...
Peut être que ça vient du timeout ? j'ai essayé plusieurs valeurs, mais ça change juste le délais, et disons que je reçoit 8 commandes sur 9...

On ne peut pas reproduire les problèmes que vous rencontrez parce que nous n'avons pas les mêmes équipements pour dialoguer avec le port série.

Du coup, vous êtes un peu seul pour diagnostiquer ce qu'il se passe... et à pouvoir prendre des initiatives pour conforter vos diagnostics.

Et comme vous n'avez toujours pas simplifié votre code...

Bon courage.

- W
10/05/2022, 12h07
fred1599

Citation:

Envoyé par jurassic pork

pas besoin de timeout avec le readline

Au contraire...

Documentation

Citation:

Envoyé par guillaume1937

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
#!/usr/bin/env python
# -*- coding: latin-1 -*-
 
#!/usr/bin/env python3
 
import io
import serial
import RPi.GPIO as GPIO
import time
 
 
PORT = "/dev/rfcomm0"
BAUDRATE = 9200
TIMEOUT = 0.05
ENCODING = "Utf-8"
LED_PIN = 22
 
try:
    GPIO.setmode(GPIO.BOARD)
    GPIO.setwarnings(False)
    GPIO.setup(LED_PIN, GPIO.OUT)
    etat = 0
 
    while True :
        ser = serial.Serial(PORT, BAUDRATE, timeout=TIMEOUT)
        #print(ser.inWaiting())
        #buffer = io.BufferedReader(ser)
        sio = io.TextIOWrapper(io.BufferedRWPair(ser, ser))
        sio.flush()
        line = sio.readline().decode('utf-8').strip().lower()
        if len(line)>0 :
            print(line)
            print("line :",line.encode("hex"))                   
            if line in ['on', 'on\r', 'on\n',('on\r\n')]:
                GPIO.output(LED_PIN, GPIO.HIGH)
                etat = 1
            elif line in ['off', 'off\r', 'off\n','off\r\n']:
                GPIO.output(LED_PIN, GPIO.LOW)
                etat = 0
        if etat == 1 : GPIO.output(LED_PIN, GPIO.HIGH)
 
finally:
    GPIO.cleanup()

:aie: c'est un peu n'importe quoi le code dans ta boucle tu reprogrammes à chaque fois le port série et tu utilises sio. En plus tu fais le test de présence du on ou off à l'envers line in
Tu peux essayer ceci :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
try:
    GPIO.setmode(GPIO.BOARD)
    GPIO.setwarnings(False)
    GPIO.setup(LED_PIN, GPIO.OUT)
    etat = 0
    ser = serial.Serial(PORT, BAUDRATE)
    while True:
        line = str(ser.readline())
        print(line)                 
        if  "on" in line:
            GPIO.output(LED_PIN, GPIO.HIGH)
            etat = 1
        elif "off" in line:
                GPIO.output(LED_PIN, GPIO.LOW)
                etat = 0
        if etat == 1 : GPIO.output(LED_PIN, GPIO.HIGH)
 
finally:
    GPIO.cleanup()

10/05/2022, 15h50
guillaume1937

Problème réglé, merci à tous pour vos réponses !
10/05/2022, 16h18
wiztricks

Citation:

Envoyé par jurassic pork

:aie: c'est un peu n'importe quoi le code dans ta boucle tu reprogrammes à chaque fois le port série et tu utilises sio. En plus tu fais le test de présence du on ou off à l'envers

Ce n'est pas n'importe quoi: le PO a juste voulu modifier le code récupéré dans un autre forum.

- W