Progress bar en multi-threading

Version imprimable

Bonjour à tous

dans un programme, j'ai une portion de code qui effectue des copies de fichiers. Mon problème c'est que la GUI est figée pendant cette copie, et donc je me suis dit que ce serait pas mal d'ajouter une Progress Bar.
Après un 1er essai où la progress bar, elle aussi, reste figée pendant la copie, je me suis penchée sur le multi-threading. Mais là, je n'arrive à rien :/

Voilà un bout du 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
44
45
46
47
48
49
50
51
52
53
 
class Files_copy_setup(Tkinter.Tk):	#Copy the selected file(s) in local
	def __init__(self, parent):
		Tkinter.Tk.__init__(self, parent)
		self.parent = parent
		self.initialize()
 
	def initialize(self):
		self.grid()
		texte = Tkinter.Label(self, text = copied_files_text, fg = "white", bg = "blue")
		texte.grid(row = 0, sticky = 'EWN')
		j = 0
		for elt in input_values.files_to_detect:
			self.texte = Tkinter.Label(self, text=elt, fg = "black", bg = "red")
			self.texte.grid(row = 2 + j)
			j = j + 1
		button_continue = ttk.Button(self, text = u"Continue", command = self.OnButtonClickContinue)
		button_continue.grid(row = (len(input_values.files_to_detect) + 3), sticky = 'S')
		button_back = Tkinter.Button(self, text = u"Back", command = self.OnButtonClickBack)
		button_back.grid(row = (len(input_values.files_to_detect) + 4), sticky = 'S')
		button_kill = Tkinter.Button(self, text = u"Quit", command = self.OnButtonClickKill)
		button_kill.grid(row = (len(input_values.files_to_detect) + 5), sticky = 'S')
 
		self.alive = ttk.Progressbar(self, orient = "horizontal", length = 200, mode = "indeterminate")
		self.alive.grid(row = (len(input_values.files_to_detect) + 6), sticky = 'S')
		self.update()
		self.geometry("%dx%d+%d+%d" % (self.winfo_reqwidth(), self.winfo_reqheight(), (self.winfo_screenwidth() - self.winfo_reqwidth()) / 2, (self.winfo_screenheight() - self.winfo_reqheight()) / 2))
		self.resizable(False, False)
		self.geometry(self.geometry())
		self.mainloop()
 
	def OnButtonClickContinue(self):
 
		self.alive.start()
		input_values.completion = 1
 
		for i in input_values.files_to_detect:
			full_file_name2 = os.path.join(input_values.path_video, i)
			if not(os.path.isfile(full_file_name2)):
				thread_copy = threading.Thread(shutil.copy(os.path.join(input_values.path_video_remote, i), input_values.path_video))
				thread_copy.start()
 
		self.destroy()
 
	def OnButtonClickBack(self):
		app = Fourth_Variable_Choice_remote(None)
		input_values.completion = 0
		self.destroy()
 
	def OnButtonClickKill(self):
		input_values.completion = 0
		input_values.next_step_var = 1
		self.destroy()

Et comme je ne maitrise ni la Progress Bar, ni le threading, je sais pas du tout comment faire en sorte que ça fonctionne...

Si qq'un peut m'éclairer là-dessus, ce serait bien urbain :)

Bonjour,

Vous devez incrémenter* self.alive (.step(valeur)) dans la boucle de OnButtonClickContinue.
Soit vous le faite dans le for soit lorsque un thread se termine (plus réaliste).
Dans le cadre de for c'est simple (voir l'exemple)
Dans le cadre du thread il faudra en faire une classe.
Attention, évitez de toucher un objet Tk à partir du thread lui même.

@+

*
Un exemple. Ici Sample simule une progression, tout comme votre for dans la liste.
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 try: import tkinter as Tk from tkinter import ttk except: import Tkinter as Tk import ttk def Sample(val, event=None, ival=1): pb.step(ival) val += ival l.config(text=str(val)) if val != maxl: root.after(100, lambda: Sample(val)) else: l.config(text="FIN") btn.config(text="Quitter", command=root.quit) maxl = 500 root = Tk.Tk() root.title('test: ttk.Progressbar') l = Tk.Label(root) l.pack(padx=5, pady=5) pb = ttk.Progressbar(root, length=maxl, maximum=maxl) pb.pack(padx=5, pady=5) btn = Tk.Button(root, text="Lancer le test") btn.bind("<Button-1>", lambda e: Sample(0, e)) btn.pack(padx=5, pady=5) root.mainloop()
Edit: Oups. Pas fonction mais classe.

20/10/2011, 10h02
lestatbzh

OK, merci. Je suis pas sur d'avoir tout saisi, mais je vais essayer à partir de là ;)

J'ai essayé, mais j'y arrive pas du tout...
Là, la progress bar se déclenche une fois que la copie est finie, et le programme n'enchaine pas sur la suite de mon code...

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
 
	def OnButtonClickContinue(self):
		self.alive.start()
		input_values.completion = 1
		val = 0
		ival = 1
		maxl = 50
 
		for i in input_values.files_to_detect:
			full_file_name2 = os.path.join(input_values.path_video, i)
			self.alive.step(ival)
			val += ival
			if val != maxl:
				self.after(100)
			else:
				pass
			if not(os.path.isfile(full_file_name2)):
				shutil.copy(os.path.join(input_values.path_video_remote, i), input_values.path_video)
		self.alive.mainloop()
		self.destroy()

Bonjour,

C'est plutôt:

Code:

1
2
3
4
5
6
7
8
9
10
11
    def OnButtonClickContinue(self):
        input_values.completion = 1
        val = 0
        self.alive.config(maximun=len(input_values.files_to_detect))
        for i in input_values.files_to_detect:
            full_file_name2 = os.path.join(input_values.path_video, i)
            if not os.path.isfile(full_file_name2):
                shutil.copy(os.path.join(input_values.path_video_remote, i), input_values.path_video)
            val += 1
            self.alive.step(val)
        self.destroy()

@+

Edit: maxl est inutile ici.
Edit 2: Attention ! Lors de mes réponses je transforme vos tabulations par 4 espaces.

21/10/2011, 11h07
lestatbzh

C'est un peu mieux, mais j'ai toujours le même souci :
la progress bar reste figée pendant la copie de fichiers :/

Il va falloir que je passe par les threads je pense ;)

Merci encore, je vais tester 2-3 trucs et revenir ici si je sèche encore ;)
21/10/2011, 17h39
wiztricks

Salut,

Citation:

Envoyé par lestatbzh

C'est un peu mieux, mais j'ai toujours le même souci : la progress bar reste figée pendant la copie de fichiers :/

La progress bar doit être mise à jour après la copy de chaque fichier.
Passer par les threads ne permettra que d'avoir un IHM "reactif" pendant la copy.
=> Ce n'est pas inintéressant mais, cela ne fera peut être pas ce que vous vous attendez à...

- W
22/10/2011, 07h38
PauseKawa

Bonjour,

Citation:

Envoyé par wiztricks

=> Ce n'est pas inintéressant mais, cela ne fera peut être pas ce que vous vous attendez à...

C'est la vrais question en fait: souhaitez vous vraiment donner une indication sur l’avancement réel ou avoir un 'mouvement' qui permet de voir que cela 'tourne' ?

@+
02/11/2011, 09h45
lestatbzh

Pardon, j'ai été obligé de mettre mon projet de côté quelques temps...

Pour répondre à la question, je veux juste que l'user sache qu'une action est en train de se réaliser en tâche de fond.
Pour l'instant, ma fenêtre TkInter freeze complètement, c'est donc pourquoi j'aimerais une progress bar, sans qu'elle ne montre l'avancement exact, mais qui explicitement montre que qqch se passe ;)
Sachant qu'il n'y a jamais bcp de fichiers à copier, mais qu'ils sont susceptibles d'être gros, parfois jusqu'à 20-25Go. Donc updater après chaque copie ne serait pas assez explicite je pense.

Salut,

Citation:

Envoyé par lestatbzh

je veux juste que l'user sache qu'une action est en train de se réaliser en tâche de fond.

Une alternative (plus simple à coder) serait de transformer simplement le curseur de la souris en sablier. Ça se fait souvent et c'est parlant aussi, je trouve.
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 try:import Tkinter except:import tkinter as Tkinter def longTask(win): old_cursor = win['cursor'] win['cursor'] = 'watch' win.update() for x in range(100000000): pass win['cursor'] = old_cursor win = Tkinter.Tk() Tkinter.Button(win, text='EXECUTE', command=lambda win=win:longTask(win)).grid() win.geometry("800x600") win.mainloop()
A+

Pfeuh

02/11/2011, 10h55
lestatbzh

Effectivement, ça peut le faire aussi :)

Je vais tenter ça, merci
03/11/2011, 06h55
PauseKawa
Il faudrait avoir un indicateur minimum plutôt que range.
Si on utilise une liste avec un .append(False) à chaque lancement d'un thread et que l'on donne au thread l'index dans la liste il suffit de faire liste[index] = True après le shutil.copy.
Une simple fonction du style
Code:

1 2 3 4 5 def verif_fin(self): if all(liste): self.win['cursor'] = self.old_cursor else: self.after(10, self.verif_fin)
permets de ne pas bloquer l'IHM.

Ceci dit si l'on souhaite donner une vrais indication il est possible de travailler sur la taille des fichiers (os.path.getsize(path)/os.stat(f).st_size).
La somme étant la taille de la Progress Bar il suffit après de vérifier* celle ci.

*
.after() comme si dessus.
une liste
liste_copie.append((full_file_name2, os.path.getsize(full_file_name2)))
avec suppression de l'item lorsque celui-ci est fini de copier afin ne pas rechercher sa taille inutilement (os.path.getsize du fichier liste[index][0] == liste[index][1]).
Modifier la Progress Bar en fonction de la somme des tailles.
les méthodes en local serait sans doute un plus.
03/11/2011, 07h13
Luke spywoker

Escuser le dérangement mais c'est quoi la ttk.Progressbar, je ne connait pas ce widget mais si c'est du python 3 c'est normal. J'ai essayer d'importer ttk mais python me dit:

Code:

ImportError: No module named ttk

et j'ai regarder dans la logithèque pas de ttk. :calim2:
03/11/2011, 08h27
pfeuh
Salut,

Citation:

Envoyé par Luke spywoker

c'est quoi la ttk.Progressbar

Je ne connaissais pas non plus, ça doit venir de là:
Code:

1 2 3 4 5 6 try: import tkinter as Tk from tkinter import ttk except: import Tkinter as Tk import ttk
Google ne sort pas grand chose sur ttk. PauseKawa, aurais tu un lien vers de la doc?

A+

Pfeuh
03/11/2011, 12h22
wiztricks

Salut,

ttk est une extension de tk qui apporte les 'styles' et de nombreux widgets "intéressants". Ces extensions sont pratiquement "inclues" dans tk mais restent préfixées par ttk pour ne pas casser les code existants.
Pour ce qui est de la doc (tk est une bibliothèque TCL utilisée aussi par Python, PERL, Ruby,...) regardez à http://www.tkdocs.com/
Cordialement,

- W
03/11/2011, 21h09
PauseKawa
Bonsoir,

Petites précisions quand même.

ttk, outre que le fait de fournir de nouveaux Widgets tels que le Notebook, apporte surtout des effets visuels tels que les thèmes, la transparence des fenêtres, l'anti-alliasing pour les fontes.
Il est construit comme tk mais vous comprendrais que ses effets ne sont pas applicables 'aux parents' (il aurait fallu refaire tout tk), d’où sa différentiation.

Il n'est disponible qu'a partir de Tcl/Tk 8.5* dont il utilise les évolutions.

Vous trouverez une petite description dans la doc Python sous Graphical User Interfaces with Tk pour la version 2.7 (prés version 3) puis 3.x..

Comme le dit wiztricks les docs en ligne sont plus généralistes que pour le wrapper T(t)kinter.

@+

* Pour tester votre version:
Code:

1 2 3 4 5 6 7 >>> Tkinter._test() >>> try: ... import Tkinter as Tk ... except: ... import tkinter as Tk ... >>> Tk._test()
04/11/2011, 15h32
lestatbzh
Citation:
Envoyé par PauseKawa

Il faudrait avoir un indicateur minimum plutôt que range.
Si on utilise une liste avec un .append(False) à chaque lancement d'un thread et que l'on donne au thread l'index dans la liste il suffit de faire liste[index] = True après le shutil.copy.
Une simple fonction du style

Code:

1 2 3 4 5 def verif_fin(self): if all(liste): self.win['cursor'] = self.old_cursor else: self.after(10, self.verif_fin)

permets de ne pas bloquer l'IHM.

Ceci dit si l'on souhaite donner une vrais indication il est possible de travailler sur la taille des fichiers (os.path.getsize(path)/os.stat(f).st_size).
La somme étant la taille de la Progress Bar il suffit après de vérifier* celle ci.

*
.after() comme si dessus.
une liste
liste_copie.append((full_file_name2, os.path.getsize(full_file_name2)))
avec suppression de l'item lorsque celui-ci est fini de copier afin ne pas rechercher sa taille inutilement (os.path.getsize du fichier liste[index][0] == liste[index][1]).
Modifier la Progress Bar en fonction de la somme des tailles.
les méthodes en local serait sans doute un plus.
J'ai laissé tomber les threads car j'y arrive pas du tout...
Mais utiliser la taille totale des fichiers à transférer est une bonne piste pour implémenter cette progress bar, merci ;)

:ccool: Merci pour ces précisions sur ttk mais je n'arrive pas a importer ttk depuis la version 2.6.5 de python, je connaissais déjà les widgets supplémentaires de Tkinter:
Code:

1 2 3 4 5 6 7 8 9 import tkFileDialog import tkSimpleDialog import tkColorChooser import tkMessageBox import ttk Traceback (most recent call last): File "<input>", line 1, in <module> ImportError: No module named ttk
Et je ne trouve pas de ttk dans la logithèque et dans la LTS d'Ubuntu la version 2.7 de python n'est plus dans la logithèque donc pas moyen de tester ttk... :calim2:

07/11/2011, 11h31
lestatbzh

bizarre...
moi je suis en Python 2.7 et j'importe ainsi :

Code:

import os, shutil, datetime, subprocess, shlex, sys, glob, re, Tkinter, ttk

et aucun souci :calim2:
07/11/2011, 11h52
wiztricks

Salut

ttk n'est livré avec les bibliothèques standards Python qu'à partir de 2.7 et 3.1
Pour 2.6, il devrait être possible d'installer pyttk

- W