Discussion :

[Stef31]

Bonjour à tous,
Désolé pour le titre, je ne sais pas résumé mon problème.
j'utilise le code ci dessous pour afficher sur un graphique toute les courbes ayant le même nom de fichiers.
les fichiers texte sont contenus dans un dossier sélectionnable.
Les fichiers sont composé d'un préfixe, "_", nom du fichier
le bouton "suivant" est censé passer directement au nom de fichier qui n'a pas encore été afficher et afficher toutes les courbes qui portent le même nom.
Je tourne en rond sans réussir à voir d’où vient mon erreur.

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
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
import os
import glob
import tkinter as tk
from tkinter import filedialog, messagebox, Radiobutton
import matplotlib.pyplot as plt
from matplotlib.backends.backend_tkagg import FigureCanvasTkAgg
 
 
class GraphicalComparison:
    def __init__(self, master):
        self.files_displayed_count = 0
        self.serial_numbers_displayed = []
        self.master = master
        self.master.title("Comparaison Graphique")
        self.file_list = []
        self.current_prefix = ""
        self.current_serial_number = ""
        self.current_file_index = 0
        self.selected_parameter = "S11"
        self.selected_format = tk.StringVar(value="N9918A")  # Format par défaut
 
        # Interface utilisateur
        self.format_label = tk.Label(master, text="Sélectionner le format:")
        self.format_label.pack()
        self.n9918a_button = Radiobutton(master, text="N9918A", variable=self.selected_format, value="N9918A")
        self.n9918a_button.pack()
        self.cmt_button = Radiobutton(master, text="CMT", variable=self.selected_format, value="CMT")
        self.cmt_button.pack()
 
        self.parameter_label = tk.Label(master, text="Sélectionner le paramètre:")
        self.parameter_label.pack()
        self.parameter_var = tk.StringVar(master)
        self.parameter_var.set("S11")
        self.parameter_menu = tk.OptionMenu(master, self.parameter_var, "S11", "S21", "S22")
        self.parameter_menu.pack()
 
        self.load_files_button = tk.Button(master, text="Charger les fichiers", command=self.load_files)
        self.load_files_button.pack()
 
        self.next_button = tk.Button(master, text="Suivant", command=self.show_next_graph)
        self.next_button.pack()
 
        self.fig, self.ax = plt.subplots()
        self.fig.set_size_inches(20, 12)
        self.canvas = FigureCanvasTkAgg(self.fig, master=master)
        self.canvas.get_tk_widget().pack()
 
    def load_files(self):
        folder_selected = filedialog.askdirectory()
        if folder_selected:
            self.file_list = glob.glob(os.path.join(folder_selected, '*.s2p'))
            self.show_next_graph()
 
    def show_next_graph(self):
        current_serial_number = self.get_current_serial_number()
 
        if current_serial_number in self.serial_numbers_displayed:
            total_files_count = len(self.file_list)
            displayed_files_count = self.files_displayed_count
            difference = total_files_count - displayed_files_count
            if difference != 0:
                messagebox.showinfo("Info",
                                    f"Comparaison terminée.\nNombre total de fichiers: {total_files_count}\nNombre de fichiers affichés: {displayed_files_count}\nDifférence: {difference}")
            else:
                messagebox.showinfo("Info", f"Comparaison terminée.")
            root.quit()
            return
        # Vérifiez si le numéro de série a déjà été affiché, s'il n'a pas été affiché, ajoutez-le à la liste
        if current_serial_number not in self.serial_numbers_displayed:
            print(current_serial_number)
            print(self.serial_numbers_displayed)
            print(self.current_file_index)
            print(len(self.file_list))
            self.serial_numbers_displayed.append(current_serial_number)
        if self.current_file_index < len(self.file_list):
            current_serial_number = self.get_current_serial_number()
            files_with_same_serial = [file_path for file_path in self.file_list if
                                      self.get_serial_number_from_file(file_path) == current_serial_number]
            self.clear_graph()
            legends = []
            for file_path in files_with_same_serial:
                prefix = self.get_prefix_from_file(file_path)
                data = self.load_data_from_file(file_path)
                if data:
                    x_data = [row[0] for row in data]
                    y_data = [row[1] for row in data]
                    self.ax.plot(x_data, y_data)
                    legends.append(prefix)
                    #Spec S11 & S22
 
                    # Sélectionner la colonne appropriée en fonction du choix de l'utilisateur
                    selected_parameter = self.parameter_var.get()
                    if selected_parameter == "S21":
                        # Ajouter Spec Bande 1 Up
                        red1_x_values = [2.025e9, 2.11e9]
                        red1_y_values = [-25, -25]
                        self.ax.plot(red1_x_values, red1_y_values, color='red', label='Courbe Rouge')
                        # Ajouter Spec Bande 2 Up
                        red2_x_values = [2.2e9, 2.29e9]
                        red2_y_values = [-25, -25]
                        self.ax.plot(red2_x_values, red2_y_values, color='red', label='Courbe Rouge')
                        # Ajouter Spec Bande 1 Down
                        red3_x_values = [2.025e9, 2.11e9]
                        red3_y_values = [-32, -32]
                        self.ax.plot(red1_x_values, red3_y_values, color='red', label='Courbe Rouge')
                        # Ajouter Spec Bande 2 Down
                        red4_x_values = [2.2e9, 2.29e9]
                        red4_y_values = [-32, -32]
                        self.ax.plot(red2_x_values, red4_y_values, color='red', label='Courbe Rouge')
                    else:
                        # Ajouter Spec Bande 1
                        red1_x_values = [2.025e9, 2.11e9]
                        red1_y_values = [-15, -15]
                        self.ax.plot(red1_x_values, red1_y_values, color='red', label='Courbe Rouge')
                        # Ajouter Spec Bande 2
                        red2_x_values = [2.2e9, 2.29e9]
                        red2_y_values = [-15, -15]
                        self.ax.plot(red2_x_values, red2_y_values, color='red', label='Courbe Rouge')
 
                    self.files_displayed_count += 1
 
            self.ax.set_title(f"Numéro de Série : {current_serial_number}")
            self.ax.set_xlabel("Fréq (GHz)")
            selected_parameter = self.parameter_var.get()
            if selected_parameter == "S11":
                y_label = "S11 (dB)"
            elif selected_parameter == "S21":
                y_label = "S21 (dB)"
            elif selected_parameter == "S22":
                y_label = "S22 (dB)"
            else:
                self.ax.set_ylabel("Ordonnée")
            self.ax.set_ylabel(y_label)
            self.ax.grid()
            self.ax.set_yticks(range(int(min(y_data)), int(max(y_data)) + 1, 5))
            self.ax.legend(legends)
            self.canvas.draw()
            self.current_file_index += 1
 
        if self.current_file_index >= len(self.file_list):
            print(self.current_file_index)
            print(len(self.file_list))
            total_files_count = len(self.file_list)
            displayed_files_count = self.files_displayed_count
            difference = total_files_count - displayed_files_count
            if difference != 0:
                messagebox.showinfo("Info",
                                    f"Comparaison terminée.\nNombre total de fichiers: {total_files_count}\nNombre de fichiers affichés: {displayed_files_count}\nDifférence: {difference}")
            else:
                messagebox.showinfo("Info", f"Comparaison terminée.")
            root.quit()
 
    def get_serial_number_from_file(self, file_path):
        prefix, serial_number = os.path.splitext(os.path.basename(file_path))[0].split('_')
        return serial_number
 
    def get_prefix_from_file(self, file_path):
        prefix, serial_number = os.path.splitext(os.path.basename(file_path))[0].split('_')
        return prefix
 
    def get_current_serial_number(self):
        file_path = self.file_list[self.current_file_index]
        return self.get_serial_number_from_file(file_path)
 
    def clear_graph(self):
        self.ax.clear()
        self.canvas.draw()
 
    def load_data_from_file(self, file_path):
        data = []
        try:
            with open(file_path, 'r') as file:
                lines = file.readlines()
                for line in lines:
                    values = line.split()
                    if len(values) >= 2:
                        try:
                            x_value = float(values[0])
                            # Sélectionner la colonne appropriée en fonction du choix de l'utilisateur
                            selected_parameter = self.parameter_var.get()
                            if selected_parameter == "S11":
                                y_value = float(values[1])
                            elif selected_parameter == "S21":
                                y_value = float(values[3])
                            elif selected_parameter == "S22":
                                y_value = float(values[7])
                            else:
                                y_value = []  # Gérer le cas où le choix de l'utilisateur n'est pas valide
 
                            data.append([x_value, y_value])
                        except ValueError:
                            print(f"Skipping line (invalid data): {line}")  # Débogage
        except Exception as e:
            print(f"Error loading data: {e}")  # Débogage
        return data
 
 
root = tk.Tk()
app = GraphicalComparison(root)
root.mainloop()

Merci de m'éclairer

[Sve@r]

Bonjour

j'utilise le code ci dessous pour afficher sur un graphique toute les courbes ayant le même nom de fichiers.

Donc là déjà je perçois mal la notion de "courbe ayant le même nom de fichier"...

le bouton "suivant" est censé passer directement au nom de fichier qui n'a pas encore été afficher et afficher toutes les courbes qui portent le même nom.
Je tourne en rond sans réussir à voir d’où vient mon erreur.

Ben déjà moi je pars de la base: le bouton "suivant" doit faire un truc, je vais voir ce qu'il fait.
Et là il appelle la méthode "show_next_graph()" qui commence par if current_serial_number in self.serial_numbers_displayed et dans ce cas ça quitte (donc déjà le comportement ne correspond pas à ce qui est décrit ici).
Ensuite j'affiche "current_serial_number" et "self.serial_numbers_displayed" et je vois que le second contient le premier. Donc ça quitte. Ensuite pourquoi le test est toujours vrai ben là c'est à toi d'investiguer (comment/pourquoi/qui modifie "self.serial_numbers_displayed" etc.)

Accessoirement écrire if truc puis ensuite if not truc... "else" ça ne te parle pas ??? Mais en plus ça ne sert à rien. Si if truc amène à quitter, alors si on ne quitte pas c'est que "truc" n'est pas arrivé et alors inutile de même tester quoi que ce soit d'autre...

[Stef31]

Drôle de façon d'apporter de l'aide

Accessoirement écrire if truc puis ensuite if not truc... "else" ça ne te parle pas ??? Mais en plus ça ne sert à rien.

Sans rire?
et accessoirement, ta réponse également.

[Sve@r]

Drôle de façon d'apporter de l'aide

Et encore, j'ai quand-même pris du temps pour ça même si tu n'en as visiblement rien à cirer. D'autres t'auraient carrément dit qu'avant d'écrire un code de 200 lignes on teste les parties importantes de façon indépendante pour éviter d'arriver à la fin à ce "rien ne marche je sais pas où ça foire"....

Sans rire?

Tout à fait sans rire. Je me base sur ce que tu nous montres (qu'ai-je d'autre à ma disposition ?). Si tu nous montres de la daube je présume que tu ne sais pas écrire autre chose. Et si tu sais écrire correctement mais que tu ne le fais pas c'est encore pire car ça veut dire que tu n'en as rien à carrer de ton code ; et dans ce cas ça m'ôte définitivement ce reste de remords car si tu ne te soucies pas de ton code on n'a pas nous à nous en soucier. D'ailleurs quand on vient s'inquiéter de son problème 2mn toutes les 24h c'est qu'effectivement il n'est pas si problématique que ça.

et accessoirement, ta réponse également.

Ok à demain.