Discussion :

[Aelurus_]

Bonjour à tous,

j'ai une liste qui ressemble à cela :

Code :

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

['H1', 0.0154, 0.0154, 0.0154, 0.0154, 0.0154, 0.0154, 0.0154, 0.0154, 'H8', 0.0158, 0.0158, 'H12', 0.0151, 0.0151, 0.0151, 0.0151, 0.0151, 0.0151, 0.0151]

Mon objectif est d'isoler chaque nom avec les valeurs qui suivent dans des listes séparées. Pour cela je fais :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
values = ['H1', 0.0154, 0.0154, 0.0154, 0.0154, 0.0154, 0.0154, 0.0154, 0.0154, 'H8', 0.0158, 0.0158, 'H12', 0.0151, 0.0151, 0.0151, 0.0151, 0.0151, 0.0151, 0.0151]
capteurs_noms = []
values_capteurs = []
capteurs_num = -1
for i in range(len(values)):
    if type(values[i]) == str:
        capteurs_num = capteurs_num+1
        capteurs_noms.append(values[i])
        values_capteurs.append([values[i]]) 
    elif type(values[i]) == float:
        values_capteurs[capteurs_num].append(values[i])

Mon besoin est d'isoler le couple nom/valeurs pour utilisation par la suite. Y a-t-il plus simple plus "esthétique", au moins je pense le code correct dans le sens où il ne plante pas et donne un résultat attendu ^^.
Merci

[wiztricks]

Salut,

Coté plus esthétique, pensez fonction qui prends la liste en argument et qui retourne le tableau. Ce qui vous permettrait d'écrire:

Code :

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

capteurs_noms, values_capteurs = myfunction(values)

Et de cacher les détails dans myfunction.

Ca ne change pas grand chose sauf que si demain vous trouvez un meilleur algo. c'est juste dans myfonction que çà se passe.

Après pour trouver les index des items qui sont des caractères:

Code :

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

1
2
3
4
ix = []
for i, e in values:
    if isinstance(value, str):
          ix.append(i)

ou plus succinctement:ix = [ i for i, e in enumerate(values) id isinstance(e, str) ]Cela fait, on sait que les "bouts" commencent a peu près en ix[i] et se terminent à ix[i+1]... mais il manque la fin: ix.append(-1)... Et les débuts sont alors ix[:-1] et les fins ix[1:]. D'où:

Code :

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

1
2
3
values_capteurs = []
for a, b in zip(ix[:-1], ix[1:]):
    values_capteurs += L[a:b]

ou:

Code :

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

values_capteurs = [ values[a:b] for a, b in zip(ix[:-1], ix[1:]) ]

- W

[Aelurus_]

Bonjour Wiztricks,

pour le coté fonction c'est effectivement l'étape suivante comme suggérer, qui nous donnerait cela :

Code :

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
values = ['H1', 0.0154, 0.0154, 0.0154, 0.0154, 0.0154, 0.0154, 0.0154, 0.0154, 'H8', 0.0158, 0.0158, 'H12', 0.0151, 0.0151, 0.0151, 0.0151, 0.0151, 0.0151, 0.0151]
 
def isol_capteurs(values):
    capteurs_noms = []
    values_capteurs = []
    capteurs_num = -1
    for i in range(len(values)):
        if type(values[i]) == str:
            capteurs_num = capteurs_num+1
            capteurs_noms.append(values[i])
            values_capteurs.append([values[i]])        
        elif type(values[i]) == float:
            values_capteurs[capteurs_num].append(values[i])
    return (capteurs_noms, values_capteurs)
 
capteurs_noms, values_capteurs = isol_capteurs(values)

Pour la suite, j'ai peur que vous alliez un peu trop vite pour moi ^^. Ma première idée était effectivement en comptant le nombre de valeurs.

Je pense avoir un peu compris isinstance et zip. Je ne les avaient mais jamais fait tourné, merci pour ces lacunes ^^.

J'obtiens ceci :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
ix = []
L=[]
for i in range(len(values)):
    value = values[i]
    if isinstance(value, str):
          ix.append(i)
 
 
values_capteurs = []
a = capteurs_noms
b = values
for a, b in zip(ix[:-1], ix[1:]):
    values_capteurs += L[a:b]

J'ai plus de mal avec la fonction zip, ou du moins dans son association avec ix. Pour l'instant je n'arrive pas associer les deux fonctions. Si j'ai bien compris il me reste récupérer mes ix. Bon je me mélange les pinceaux là. Dans la première boucle je récupère bien la position du nom, mais je n'arrive pas à l'exploiter avec la fonction zip. J'ai l'impression qu'il y a une double boucle que je n'arrive pas à voir.

[wiztricks]

Salut,

Pour la suite, j'ai peur que vous alliez un peu trop vite pour moi ^^.

L'avantage de la fonction, c'est qu'on attend d'elle un boulot qu'on peut tester.
Après ne vous prenez pas le chou avec le code que je vous ai suggéré, çà ne marchera pas mieux et, pire, vous n'allez pas être trop confiant avec des lignes que vous ne comprenez pas.

Code :

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

1
2
3
4
def isol_capteurs(values):
     ix = [ i for i, e in enumerate(values) id isinstance(e, str) ] + [-1]
     values_capteurs = [ values[a:b] for a, b in zip(ix[:-1], ix[1:]) ]
     return [ v[0] for v in values_capteurs], values_capteurs

- W

[Aelurus_]

Yesssss,
effectivement, votre code est largement plus court, mais il est trop costaud, trop concis ^^. Je ne vais pas l'utiliser, mais je garde en mémoire isinstance et zip. Je vois aussi qu'il offre une simplicité agréable. Votre déclaration de boucle me plait je vais tenter de l'appliquer.

Grand merci pour ces retours.

[danielhagnoul]

@wiztricks : le zip ne me donne pas la dernière série de valeurs.

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
datas = [
    'H1', 0.0154, 0.0154, 0.0154, 0.0154, 0.0154, 0.0154, 0.0154, 0.0154,
    'H8', 0.0158, 0.0158,
    'H12', 0.0151, 0.0151, 0.0151, 0.0151, 0.0151, 0.0151, 0.0151,
]
 
ix = [ i for i, e in enumerate(datas) if isinstance(e, str) ]
 
# debug
print(ix) # [0, 9, 12]
print(list(zip(ix[:-1], ix[1:]))) # [(0, 9), (9, 12)]
# fin debug
 
values_capteurs = [ datas[a:b] for a, b in zip(ix[:-1], ix[1:]) ]
values_capteurs.append(datas[ix[-1]:])
 
print(values_capteurs)
 
"""
[
    ['H1', 0.0154, 0.0154, 0.0154, 0.0154, 0.0154, 0.0154, 0.0154, 0.0154], 
    ['H8', 0.0158, 0.0158], 
    ['H12', 0.0151, 0.0151, 0.0151, 0.0151, 0.0151, 0.0151, 0.0151]
]
"""

[wiztricks]

Salut,

@wiztricks : le zip ne me donne pas la dernière série de valeurs.

C'est parce que vous avez oublié d'ajouter la fin: ix.append(-1).

- W

[Aelurus_]

Salut tous,

@wiztricks : le zip ne me donne pas la dernière série de valeurs.

Je confirme, je trouve effectivement la même différence, il manque une valeur à 'H12'. Cela vient de

Code :

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

if isinstance(e, str) ] + [-1]

. Mais je n'arrive pas, ne sais pas corriger ce décalage dans le code à wistricks.

Code :

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
values = [
        'H1', 0.0154, 0.0154, 0.0154, 0.0154, 0.0154, 0.0154, 0.0154, 0.0154,
        'H8', 0.0158, 0.0158,
        'H12', 0.0151, 0.0151, 0.0151, 0.0151, 0.0151, 0.0151, 0.0151
]
 
def isol_capteurs(values):
    capteurs_noms = []
    values_capteurs = []
    capteurs_num = -1
    for i in range(len(values)):
        if type(values[i]) == str:
            capteurs_num = capteurs_num+1
            capteurs_noms.append((values[i]))
            values_capteurs.append([values[i]])        
        elif type(values[i]) == float:
            values_capteurs[capteurs_num].append(values[i])
    return (capteurs_noms, values_capteurs)
 
capteurs_noms, values_capteurs = isol_capteurs(values)
 
print (capteurs_noms)
print (values_capteurs)
 
def isol_capteursW(values):
     ix = [ i for i, e in enumerate(values) if isinstance(e, str) ] + [-1]
     values_capteurs = [ values[a:b] for a, b in zip(ix[:-1], ix[1:]) ]
     return [ v[0] for v in values_capteurs], values_capteurs
 
capteurs_noms, values_capteurs = isol_capteursW(values)
 
print (capteurs_noms)
print (values_capteurs)
 
"""sortie
['H1', 'H8', 'H12']
[['H1', 0.0154, 0.0154, 0.0154, 0.0154, 0.0154, 0.0154, 0.0154, 0.0154],
 ['H8', 0.0158, 0.0158],
 ['H12', 0.0151, 0.0151, 0.0151, 0.0151, 0.0151, 0.0151, 0.0151]]
['H1', 'H8', 'H12']
[['H1', 0.0154, 0.0154, 0.0154, 0.0154, 0.0154, 0.0154, 0.0154, 0.0154],
 ['H8', 0.0158, 0.0158],
 ['H12', 0.0151, 0.0151, 0.0151, 0.0151, 0.0151, 0.0151]]
"""

Salut,



C'est parce que vous avez oublié d'ajouter la fin: ix.append(-1).

- W

je tente de comprendre ou est cette fin ^^, si ce n'est pas +[-1].


fred1599, merci de ton retour. Par contre ton hypothèse de départ n'est pas entièrement fausse car c'est justement pour gérer ou ne pas gérer cette création en amont.

Les valeurs sont issues de manipulation expérimentale et proviennent d'un fichier. Les 'H1' et autres sont mis à la main dans la colonne des données. Cela permet de les séparer d'une manière assez simple avec une certaine vérification de la mesure récupérée. Cette boucle me permet de faire un seul fichier avec plusieures jeu de valeurs à traiter au lieu d'un fichier par jeu de valeurs.
Je reste persuader que ma logique de traitement est perfectible, et mon plus gros souci dans cet exercice est l'identification et la séparation de mes données.Les résultats obtenue sont juste avec une forme de données d'entrée. Je cherche effectivement à optimiser tant ma logique que le code qui en découlecar les deux sont liées, une bonne logique aide à codé et une meilleur maitrise du code aide la logique ^^.

Cependant ta version du dictionnaire est très intéressante, elle me ferait faire une certaine quantité de modification sur la suite des traitements qui sont construits sur des listes et tableaux. Les dictionnaires ne me sont pas très familiers non plus ^^.

Merci de ton exemple que je vais essayer de manipuler.

[wiztricks]

Salut,

Mais je n'arrive pas, ne sais pas corriger ce décalage dans le code à wistricks.

Quand ix.append(-1) est là, effectivement le -1 est une erreur.
Donc on reprend ce qu'on cherche à faire.
Si ix est la liste des positions des "str".

• ix[0], ix[1] définit le premier intervalle,
• ix[1], ix[2] définit le 2nd,
• etc...

Quand on arrive à la position du dernier str, il manque la fin de l'intervalle qui est len(values) (et non -1).

- W

[Aelurus_]

Jusque-là pas trop de soucis. Je comprends la nécessité du -1 qui donne la position de la dernière valeur quand ce n'est pas la dernière série. Ne comprenant qu'un petit votre syntaxe, je me dis qu'il y a quelque chose à faire avec la double incrémentation (i for i,e...) dans votre première boucle :

Code :

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

ix = [ i for i, e in enumerate(values) if isinstance(e, str) ] + [-1]

, mais ne comprend pas comme identifier le dernier tour en fait.
Ou alors c'est ici

Code :

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

values_capteurs = [ values[a:b] for a, b in zip(ix[:-1], ix[1:]) ]

, pour la derniere valeur il ne faut pas de ix[:-1] mais 0. Donc une autre lignes ou une modification par là ^^.

[wiztricks]

Ne comprenant qu'un petit votre syntaxe, je me dis qu'il y a quelque chose à faire avec la double incrémentation (i for i,e...) dans votre première boucle :

Code :

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

ix = [ i for i, e in enumerate(values) if isinstance(e, str) ] + [-1]

Ce n'est pas une double incrémentation, c'est une sorte de mélange entre: for e in values et for i in range(len(values)) pour avoir à la fois index et valeur. On peut écrire:

Code :

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

[ i for i in range(len(values)) if isinstance(values[i], str) ] + [len(values)]

Ou alors c'est ici

Code :

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

values_capteurs = [ values[a:b] for a, b in zip(ix[:-1], ix[1:]) ]

, pour la derniere valeur il ne faut pas de ix[:-1] mais 0. Donc une autre lignes ou une modification par là ^^.

On peut aussi écrire çà: [ value[ix[i]:ix[i+1]] for i in range(len(ix)-1) ]L'important est ce qu'on cherche à faire, pas forcément le Python avec lequel on le fait.

- W

[fred1599]

Bonjour à tous,

j'ai une liste qui ressemble à cela :

Code :

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

['H1', 0.0154, 0.0154, 0.0154, 0.0154, 0.0154, 0.0154, 0.0154, 0.0154, 'H8', 0.0158, 0.0158, 'H12', 0.0151, 0.0151, 0.0151, 0.0151, 0.0151, 0.0151, 0.0151]

ça laisse à penser qu'avant cette construction et le mélange des genres, les sous listes auraient pu être construites à la volée.

La mise en place de sous listes me semblent pas non plus très pertinents, on aurait pu utiliser un dictionnaire avec comme clés les 'H...' et valeurs une liste de flottants, et cela aussi implique cette création avant d'en arriver à cette liste.

Sinon en partant de cette base tout de même,

Code :

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
datas = [    'H1', 0.0154, 0.0154, 0.0154, 0.0154, 0.0154, 0.0154, 0.0154, 0.0154,
    'H8', 0.0158, 0.0158,
    'H12', 0.0151, 0.0151, 0.0151, 0.0151, 0.0151, 0.0151, 0.0151,
]
 
last_key = None
d = {}
for value in datas:
    if isinstance(value, str):
        d[value] = []
        last_key = value
    elif last_key:
        d[last_key].append(value)
    else:
        raise KeyError("error in the first item")
 
print(d)
 
"""
{
  'H1': [0.0154, 0.0154, 0.0154, 0.0154, 0.0154, 0.0154, 0.0154, 0.0154],
  'H12': [0.0151, 0.0151, 0.0151, 0.0151, 0.0151, 0.0151, 0.0151],
  'H8': [0.0158, 0.0158]
}
"""

[Aelurus_]

En regardant ta boucle de plus prés j'en tire ceci :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
def isol_capteurs3(values):
    capteurs_noms = []
    values_capteurs = []
    capteurs_num = -1
    for value in values:
        if isinstance(value, str):
            capteurs_num = capteurs_num+1
            capteurs_noms.append((value))
            values_capteurs.append([value])        
        elif type(value) == float:
            values_capteurs[capteurs_num].append(value)
    return (capteurs_noms, values_capteurs)

Par intuition je dirais qu'il est plus léger en ressources car le "i" est enlevé, donc à minima une variable de moins. Ce n'est que mon avis car pas vérifié.

[Aelurus_]

ça laisse à penser qu'avant cette construction et le mélange des genres, les sous listes auraient pu être construites à la volée.

La mise en place de sous listes me semblent pas non plus très pertinents, on aurait pu utiliser un dictionnaire avec comme clés les 'H...' et valeurs une liste de flottants, et cela aussi implique cette création avant d'en arriver à cette liste.

Sinon en partant de cette base tout de même,

Code :

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
datas = [    'H1', 0.0154, 0.0154, 0.0154, 0.0154, 0.0154, 0.0154, 0.0154, 0.0154,
    'H8', 0.0158, 0.0158,
    'H12', 0.0151, 0.0151, 0.0151, 0.0151, 0.0151, 0.0151, 0.0151,
]
 
last_key = None
d = {}
for value in datas:
    if isinstance(value, str):
        d[value] = []
        last_key = value
    elif last_key:
        d[last_key].append(value)
    else:
        raise KeyError("error in the first item")
 
print(d)
 
"""
{
  'H1': [0.0154, 0.0154, 0.0154, 0.0154, 0.0154, 0.0154, 0.0154, 0.0154],
  'H12': [0.0151, 0.0151, 0.0151, 0.0151, 0.0151, 0.0151, 0.0151],
  'H8': [0.0158, 0.0158]
}
"""

Ton idée de dictionnaire me plait bien et colle bien à ce que je veux et il ton code tourne plus vite aussi. Je vais adapter la suite de mon code en ce sens. Désolé du retour tardif.
Merci pour cet exemple qui en plus me pousse à traiter les erreurs.

[danielhagnoul]

@wiztricks: en auscultant le code ci-dessous j'ai compris comment il fonctionne, merci pour cette zip idée.

Code Python :

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

1
2
3
def isol_capteurs(values):
    ix = [i for i, e in enumerate(values) if isinstance(e, str)] + [-1]
    return [values[a:b] for a, b in zip(ix[:-1], ix[1:])]

@fred1599: la construction, au départ, d'un dict au lieu d'une list est certainement la meilleure solution.