Pourquoi mon if ne peut pas comparer une valeur

**pierrot10** · 21/04/2020, 00h06

Bonjour à tous

Mon but est de lire ce qui est envoyer dans mon serial.

Je recois des position GPS et cette commande me retourne toujours false alors qu'elle est true

Code python :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
 
print ("raw:", data) #prints raw data
print(data[0:6])
if data[0:6] == "$GPRMC":
    print('2')

print 2 n'est jamais afficher.

Par contre voici ce que me print data[0:6]

raw: b'$GPRMC,220135.00,A,4627.20773,N,00613.15008,E,0.632,,200420,,,A*7A\r\n'
b'$GPRMC'

il y a ce b. qu'est-ce qu'il veut dire, des bytes?

Merci

**Beginner.** · 21/04/2020, 00h29

Salut,

Envoyé par pierrot10

il y a ce b. qu'est-ce qu'il veut dire, des bytes?

Oui.

**Beginner.** · 21/04/2020, 00h33

Pour faire ce que tu veux une possibilité :

Code python :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
print ("raw:", data) #prints raw data
print(data[0:6])
if data[0:6].decode() == "$GPRMC":
    print('2')

Mais autant tout convertir : data = data.decode()

**pierrot10** · 21/04/2020, 00h44

Je pense avoir compris pourquoi,
parce que data est byte et la comparaison est un str

J'ai essayé ceci qui fonctionne

Code python :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
sentence = data[0:6]
    if sentence.decode('UTF-8') == "$GPRMC":

Mais je suis pas tout à fait convaincu. Je donne mon code dans le prochain message

**pierrot10** · 21/04/2020, 00h51

Voici mon code au complet

Comment je oeux faire pour quitter la boucle?

Je fais appelle à mon fonction

Code python :

Sélectionner tout - Visualiser dans une fenêtre à part

lat,lon,date = get_gps()

Code python :

Sélectionner tout - Visualiser dans une fenêtre à part

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
46
47
48
49
50
51
52
53
54
 
def get_gps():
   #GPIO.output(gps_power, 1)
    try:
        with serial.Serial('/dev/ttyS0', baudrate=9600, timeout=1) as ser:
            while True:
                data = ser.readline()
                #if data[0:6] == "b'$GPRMC":
                #    mnea  = pynmea2.parse(data)
                #    print(mnea.latitue)
 
                parseGPS(data) # J'aimerais bien un nouveau paramètre comme parseGPS(data,"GPRMC")
                print('1')
    except Exception as e:
        print('Error reading serial port:' + e)
 
def parseGPS(data):
    print ("raw:", data) #prints raw data
    print(data[0:6])
    sentence = data[0:6]
    if sentence.decode('UTF-8') == "$GPRMC":
        print('2')
        spdata = data.decode('UTF-8')
        sdata = spdata.split(",")
        if sdata[2] == 'V':
            print ('no satellite data available')
            return
        print('---Parsing GPRMC---')
        time = sdata[1][0:2] + ":" + sdata[1][2:4] + ":" + sdata[1][4:6]
        lat = decode(sdata[3]) #latitude
        dirLat = sdata[4]      #latitude direction N/S
        lon = decode(sdata[5]) #longitute
        dirLon = sdata[6]      #longitude direction E/W
        speed = sdata[7]       #Speed in knots
        trCourse = sdata[8]    #True course
        date = sdata[9][0:2] + "/" + sdata[9][2:4] + "/" + sdata[9][4:6]#date
 
        print("time : %s, latitude : %s(%s), longitude : %s(%s), speed : %s, True Course : %s, Date : %s" %  (time,lat,dirLat,lon,dirLon,speed,trCourse,date))
        return lat,lon,date
 
    #if str.find('GGA') > 0:
        #msg = pynmea2.parse(str)
        #msg.latitude
        #print "Timestamp: %s -- Lat: %s %s -- Lon: %s %s -- Altitude: %s %s -- Satellites: %s" % (msg.timestamp,msg.lat,msg.lat_dir,msg.lon,msg.lon_dir,msg.altitude,msg.altitude_units,msg.num_sa$
 
 
def decode(coord):
    #Converts DDDMM.MMMMM > DD deg MM.MMMMM min
    x = coord.split(".")
    head = x[0]
    tail = x[1]
    deg = head[0:-2]
    min = head[-2:]
    return deg + " deg " + min + "." + tail + " min"

Le problème des qu'il a trouver GPRMC, il continue a looper. et je pense que c'est parce que j'ai un WHILE True, donc il troune en boucle car elle n'est jamais rompue.
Donc, ou est-e que je met un break, dans parceGPS()?

Finalement pour que

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
lat,lon,date = get_gps()

prennent les valeurs trouvé, le return se fait bien parseGPS()?

**Beginner.** · 21/04/2020, 01h05

Envoyé par pierrot10

Comment je oeux faire pour quitter une boucle?

Ben en général on quitte une boucle quand la ou les conditions voulue(s) sont remplie(s)...

Dans ta boucle il y a

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

data = ser.readline()

mais combien y a-t-il de ligne à lire ?
Si il n'y en a qu'une alors pas besoin de la boucle while True...

Envoyé par pierrot10

Finalement pour que

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

lat,lon,date = get_gps()

prennent les valeurs trouvé, le return se fait bien parseGPS()?

Si tu tiens à faire comme ça alors dans la fonction get_gps il faut que tu remplaces parseGPS(data) par return parseGPS(data)...

Et pas besoin de la boucle car apparemment tu ne lis qu'une ligne...

PS : Corrige une erreur : remplace print('Error reading serial port:' + e) par print('Error reading serial port:', e)...

**wiztricks** · 21/04/2020, 08h58

Salut,

Envoyé par pierrot10

Je recois des position GPS et cette commande me retourne toujours false alors qu'elle est true

Code python :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
 
print ("raw:", data) #prints raw data
print(data[0:6])
if data[0:6] == "$GPRMC":
    print('2')

Pour répondre à votre question data[0:6] == b"$GPRMC".
Et si vous aviez compris ce que je vous racontais dans une autre discussion, vous sauriez que:
$GPRMC,220135.00,A,4627.20773,N,00613.15008,E,0.632,,200420,,,A*7A
est une trame NMEA 0183.

Elle utilise tous les caractères ASCII (et est donc encodée ASCII) et que les 2 derniers bytes (7E) sont un checksum des données qui sont entre les caractères '$' et '*' de la trame.

Ce qui signifie que vous avez un checksum à calculer pour vérifier l'intégrité des données reçues avant de pouvoir en faire quoi que ce soit.

- W

**pierrot10** · 21/04/2020, 10h23

Bonjour Wiztrick et beginner

Merci pour vos réponses et toutes vos suggestions.
D'abord, je m'excuse car j'ai pas vu vos réponses et il y a eu de l'avancement. Aussi apparemment, j'ai posté un message sans avoir vu celui de beginner.

"Historiquement", j'essayais de faire mon travail avec pynmea2 selon un tutorial. N'y arrivant pas à récupérer la trame venant de serial, je me suis penché sur la "solution manuelle", d'ou mes codes et mes fonctions.

La source du problème était bien

Tu compares des bytes à une string

et le decode() est bien la solution.

J'ai donc réussi à lire ma trame. Donc j'ai repris la solution avec pynmea2 car je trouve ceci plus simple et propre

Code python :

Sélectionner tout - Visualiser dans une fenêtre à part

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
 
get_gps("GPRMC")
def get_gps(sentence):
    #GPIO.output(gps_power, 1)
    try:
        with serial.Serial('/dev/ttyS0', baudrate=9600, timeout=1) as ser:
            while True:
                # read 20 lines from the serial output, if not leave
                for i in range(20):
                    data = ser.readline()
                    sdata=data[0:6].decode('UTF-8')
                    if sdata == "$" + sentence:
                        mnea  = pynmea2.parse(data.decode('UTF-8'))
                        print('DEBUG')
                        print(mnea)
                        print(mnea.latitude)
                        print(mnea.longitude)
                        print(mnea.time)
                        print(mnea.date)
                        return mnea.latitude, mnea.longitude
                        break
 
                 print('Cannt find NMEA sentence')
                 # Leave while
                 break
 
                # NOT USED
                # parseGPS(data)
 
    except Exception as e:
        print('Error reading serial port:' , e)

et j'ai appliqué la trouvaille à celle-ci, ce qui me donne quelque chose de plus simple et élégant, si je peux dire ainsi.

Au passage

PS : Corrige une erreur : remplace ...

Merci, c'est fait

De là, j'ai encore deux petites questions et une dernière pour Wiztricks

Q1: Est-ce que le break est indispensable après le return? Est-ce que le return suffit pour quitter la loop?
Q2: Est-ce qu'il y a plus élégant?

Elle utilise tous les caractères ASCII (et est donc encodée ASCII) et que les 2 derniers bytes (7E) sont un checksum des données qui sont entre les caractères '$' et '*' de la trame.

C'est intéressant,

et que les 2 derniers bytes (7E)

Tu veux dire 7A?

En fait c'est cela qui me permettrait de voir si la trame est valide ou pas?
Ce qu'il manque encore, c'est justement cela, car même s'il trouve $GPRMC, et que la trame n'est pas valide, je préfère qu'il me génère une exception, ou qu'il ne quitte pas le while, jusqu'à ce qu'il rencontre une trame valide, selon le type voulu.