Salome HOME
Minor source update for OM compatibility
[modules/adao.git] / src / daComposant / daCore / Persistence.py
index 34bafb186feffe162b4b5dbcc47c57fdc6274e9c..29d7f71f5e961810f2468e8f9cd0cbc585ee3e9d 100644 (file)
@@ -1,6 +1,6 @@
 # -*- coding: utf-8 -*-
 #
-# Copyright (C) 2008-2023 EDF R&D
+# Copyright (C) 2008-2024 EDF R&D
 #
 # This library is free software; you can redistribute it and/or
 # modify it under the terms of the GNU Lesser General Public
@@ -30,10 +30,12 @@ __all__ = []
 import os, numpy, copy, math
 import gzip, bz2, pickle
 
-from daCore.PlatformInfo import PathManagement ; PathManagement()
-from daCore.PlatformInfo import has_gnuplot, PlatformInfo
-mfp = PlatformInfo().MaximumPrecision()
-if has_gnuplot:
+from daCore.PlatformInfo import PathManagement ; PathManagement()  # noqa: E702,E203
+from daCore.PlatformInfo import PlatformInfo
+lpi = PlatformInfo()
+mfp = lpi.MaximumPrecision()
+
+if lpi.has_gnuplot:
     import Gnuplot
 
 # ==============================================================================
@@ -45,8 +47,8 @@ class Persistence(object):
     __slots__ = (
         "__name", "__unit", "__basetype", "__values", "__tags", "__dynamic",
         "__g", "__title", "__ltitle", "__pause", "__dataobservers",
-        )
-    #
+    )
+
     def __init__(self, name="", unit="", basetype=str):
         """
         name : nom courant
@@ -89,16 +91,32 @@ class Persistence(object):
         """
         Stocke une valeur avec ses informations de filtrage.
         """
-        if value is None: raise ValueError("Value argument required")
+        if value is None:
+            raise ValueError("Value argument required")
         #
         self.__values.append(copy.copy(self.__basetype(value)))
         self.__tags.append(kwargs)
         #
-        if self.__dynamic: self.__replots()
+        if self.__dynamic:
+            self.__replots()
         __step = len(self.__values) - 1
-        for hook, parameters, scheduler in self.__dataobservers:
+        for hook, parameters, scheduler, order, osync, dovar in self.__dataobservers:
             if __step in scheduler:
-                hook( self, parameters )
+                if order is None or dovar is None:
+                    hook( self, parameters )
+                else:
+                    if not isinstance(order, (list, tuple)):
+                        continue
+                    if not isinstance(dovar, dict):
+                        continue
+                    if not bool(osync):  # Async observation
+                        hook( self, parameters, order, dovar )
+                    else:  # Sync observations
+                        for v in order:
+                            if len(dovar[v]) != len(self):
+                                break
+                        else:
+                            hook( self, parameters, order, dovar )
 
     def pop(self, item=None):
         """
@@ -136,8 +154,8 @@ class Persistence(object):
     def __str__(self):
         "x.__str__() <==> str(x)"
         msg  = "   Index        Value   Tags\n"
-        for i,v in enumerate(self.__values):
-            msg += "  i=%05i  %10s   %s\n"%(i,v,self.__tags[i])
+        for iv, vv in enumerate(self.__values):
+            msg += "  i=%05i  %10s   %s\n"%(iv, vv, self.__tags[iv])
         return msg
 
     def __len__(self):
@@ -157,7 +175,8 @@ class Persistence(object):
 
     def index(self, value, start=0, stop=None):
         "L.index(value, [start, [stop]]) -> integer -- return first index of value."
-        if stop is None : stop = len(self.__values)
+        if stop is None:
+            stop = len(self.__values)
         return self.__values.index(value, start, stop)
 
     # ---------------------------------------------------------
@@ -172,10 +191,11 @@ class Persistence(object):
                     if tagKey in self.__tags[i]:
                         if self.__tags[i][tagKey] == kwargs[tagKey]:
                             __tmp.append( i )
-                        elif isinstance(kwargs[tagKey],(list,tuple)) and self.__tags[i][tagKey] in kwargs[tagKey]:
+                        elif isinstance(kwargs[tagKey], (list, tuple)) and self.__tags[i][tagKey] in kwargs[tagKey]:
                             __tmp.append( i )
                 __indexOfFilteredItems = __tmp
-                if len(__indexOfFilteredItems) == 0: break
+                if len(__indexOfFilteredItems) == 0:
+                    break
         return __indexOfFilteredItems
 
     # ---------------------------------------------------------
@@ -184,7 +204,7 @@ class Persistence(object):
         __indexOfFilteredItems = self.__filteredIndexes(**kwargs)
         return [self.__values[i] for i in __indexOfFilteredItems]
 
-    def keys(self, keyword=None , **kwargs):
+    def keys(self, keyword=None, **kwargs):
         "D.keys() -> list of D's keys"
         __indexOfFilteredItems = self.__filteredIndexes(**kwargs)
         __keys = []
@@ -195,7 +215,7 @@ class Persistence(object):
                 __keys.append( None )
         return __keys
 
-    def items(self, keyword=None , **kwargs):
+    def items(self, keyword=None, **kwargs):
         "D.items() -> list of D's (key, value) pairs, as 2-tuples"
         __indexOfFilteredItems = self.__filteredIndexes(**kwargs)
         __pairs = []
@@ -232,7 +252,7 @@ class Persistence(object):
             __indexOfFilteredItems = [item,]
         #
         # Dans le cas où la sortie donne les valeurs d'un "outputTag"
-        if outputTag is not None and isinstance(outputTag,str) :
+        if outputTag is not None and isinstance(outputTag, str):
             outputValues = []
             for index in __indexOfFilteredItems:
                 if outputTag in self.__tags[index].keys():
@@ -272,18 +292,19 @@ class Persistence(object):
     # ---------------------------------------------------------
     def means(self):
         """
-        Renvoie la série, contenant à chaque pas, la valeur moyenne des données
+        Renvoie la série contenant, à chaque pas, la valeur moyenne des données
         au pas. Il faut que le type de base soit compatible avec les types
         élémentaires numpy.
         """
         try:
-            return [numpy.mean(item, dtype=mfp).astype('float') for item in self.__values]
+            __sr = [numpy.mean(item, dtype=mfp).astype('float') for item in self.__values]
         except Exception:
             raise TypeError("Base type is incompatible with numpy")
+        return numpy.array(__sr).tolist()
 
     def stds(self, ddof=0):
         """
-        Renvoie la série, contenant à chaque pas, l'écart-type des données
+        Renvoie la série contenant, à chaque pas, l'écart-type des données
         au pas. Il faut que le type de base soit compatible avec les types
         élémentaires numpy.
 
@@ -292,75 +313,95 @@ class Persistence(object):
         """
         try:
             if numpy.version.version >= '1.1.0':
-                return [numpy.array(item).std(ddof=ddof, dtype=mfp).astype('float') for item in self.__values]
+                __sr = [numpy.array(item).std(ddof=ddof, dtype=mfp).astype('float') for item in self.__values]
             else:
                 return [numpy.array(item).std(dtype=mfp).astype('float') for item in self.__values]
         except Exception:
             raise TypeError("Base type is incompatible with numpy")
+        return numpy.array(__sr).tolist()
 
     def sums(self):
         """
-        Renvoie la série, contenant à chaque pas, la somme des données au pas.
+        Renvoie la série contenant, à chaque pas, la somme des données au pas.
         Il faut que le type de base soit compatible avec les types élémentaires
         numpy.
         """
         try:
-            return [numpy.array(item).sum() for item in self.__values]
+            __sr = [numpy.array(item).sum() for item in self.__values]
         except Exception:
             raise TypeError("Base type is incompatible with numpy")
+        return numpy.array(__sr).tolist()
 
     def mins(self):
         """
-        Renvoie la série, contenant à chaque pas, le minimum des données au pas.
+        Renvoie la série contenant, à chaque pas, le minimum des données au pas.
         Il faut que le type de base soit compatible avec les types élémentaires
         numpy.
         """
         try:
-            return [numpy.array(item).min() for item in self.__values]
+            __sr = [numpy.array(item).min() for item in self.__values]
         except Exception:
             raise TypeError("Base type is incompatible with numpy")
+        return numpy.array(__sr).tolist()
 
     def maxs(self):
         """
-        Renvoie la série, contenant à chaque pas, la maximum des données au pas.
+        Renvoie la série contenant, à chaque pas, la maximum des données au pas.
         Il faut que le type de base soit compatible avec les types élémentaires
         numpy.
         """
         try:
-            return [numpy.array(item).max() for item in self.__values]
+            __sr = [numpy.array(item).max() for item in self.__values]
         except Exception:
             raise TypeError("Base type is incompatible with numpy")
+        return numpy.array(__sr).tolist()
 
     def powers(self, x2):
         """
-        Renvoie la série, contenant à chaque pas, la puissance "**x2" au pas.
+        Renvoie la série contenant, à chaque pas, la puissance "**x2" au pas.
         Il faut que le type de base soit compatible avec les types élémentaires
         numpy.
         """
         try:
-            return [numpy.power(item, x2) for item in self.__values]
+            __sr = [numpy.power(item, x2) for item in self.__values]
         except Exception:
             raise TypeError("Base type is incompatible with numpy")
+        return numpy.array(__sr).tolist()
 
     def norms(self, _ord=None):
         """
         Norm (_ord : voir numpy.linalg.norm)
 
-        Renvoie la série, contenant à chaque pas, la norme des données au pas.
+        Renvoie la série contenant, à chaque pas, la norme des données au pas.
         Il faut que le type de base soit compatible avec les types élémentaires
         numpy.
         """
         try:
-            return [numpy.linalg.norm(item, _ord) for item in self.__values]
+            __sr = [numpy.linalg.norm(item, _ord) for item in self.__values]
         except Exception:
             raise TypeError("Base type is incompatible with numpy")
+        return numpy.array(__sr).tolist()
+
+    def traces(self, offset=0):
+        """
+        Trace
+
+        Renvoie la série contenant, à chaque pas, la trace (avec l'offset) des
+        données au pas. Il faut que le type de base soit compatible avec les
+        types élémentaires numpy.
+        """
+        try:
+            __sr = [numpy.trace(item, offset, dtype=mfp).astype('float') for item in self.__values]
+        except Exception:
+            raise TypeError("Base type is incompatible with numpy")
+        return numpy.array(__sr).tolist()
 
     def maes(self, _predictor=None):
         """
         Mean Absolute Error (MAE)
         mae(dX) = 1/n sum(dX_i)
 
-        Renvoie la série, contenant à chaque pas, la MAE des données au pas.
+        Renvoie la série contenant, à chaque pas, la MAE des données au pas.
         Il faut que le type de base soit compatible avec les types élémentaires
         numpy. C'est réservé aux variables d'écarts ou d'incréments si le
         prédicteur est None, sinon c'est appliqué à l'écart entre les données
@@ -368,7 +409,7 @@ class Persistence(object):
         """
         if _predictor is None:
             try:
-                return [numpy.mean(numpy.abs(item)) for item in self.__values]
+                __sr = [numpy.mean(numpy.abs(item)) for item in self.__values]
             except Exception:
                 raise TypeError("Base type is incompatible with numpy")
         else:
@@ -378,16 +419,17 @@ class Persistence(object):
                 if numpy.asarray(_predictor[i]).size != numpy.asarray(item).size:
                     raise ValueError("Predictor size at step %i is incompatible with the values"%i)
             try:
-                return [numpy.mean(numpy.abs(numpy.ravel(item) - numpy.ravel(_predictor[i]))) for i, item in enumerate(self.__values)]
+                __sr = [numpy.mean(numpy.abs(numpy.ravel(item) - numpy.ravel(_predictor[i]))) for i, item in enumerate(self.__values)]
             except Exception:
                 raise TypeError("Base type is incompatible with numpy")
+        return numpy.array(__sr).tolist()
 
     def mses(self, _predictor=None):
         """
         Mean-Square Error (MSE) ou Mean-Square Deviation (MSD)
         mse(dX) = 1/n sum(dX_i**2)
 
-        Renvoie la série, contenant à chaque pas, la MSE des données au pas. Il
+        Renvoie la série contenant, à chaque pas, la MSE des données au pas. Il
         faut que le type de base soit compatible avec les types élémentaires
         numpy. C'est réservé aux variables d'écarts ou d'incréments si le
         prédicteur est None, sinon c'est appliqué à l'écart entre les données
@@ -396,7 +438,7 @@ class Persistence(object):
         if _predictor is None:
             try:
                 __n = self.shape()[0]
-                return [(numpy.linalg.norm(item)**2 / __n) for item in self.__values]
+                __sr = [(numpy.linalg.norm(item)**2 / __n) for item in self.__values]
             except Exception:
                 raise TypeError("Base type is incompatible with numpy")
         else:
@@ -407,18 +449,19 @@ class Persistence(object):
                     raise ValueError("Predictor size at step %i is incompatible with the values"%i)
             try:
                 __n = self.shape()[0]
-                return [(numpy.linalg.norm(numpy.ravel(item) - numpy.ravel(_predictor[i]))**2 / __n) for i, item in enumerate(self.__values)]
+                __sr = [(numpy.linalg.norm(numpy.ravel(item) - numpy.ravel(_predictor[i]))**2 / __n) for i, item in enumerate(self.__values)]
             except Exception:
                 raise TypeError("Base type is incompatible with numpy")
+        return numpy.array(__sr).tolist()
 
-    msds=mses # Mean-Square Deviation (MSD=MSE)
+    msds = mses  # Mean-Square Deviation (MSD=MSE)
 
     def rmses(self, _predictor=None):
         """
         Root-Mean-Square Error (RMSE) ou Root-Mean-Square Deviation (RMSD)
         rmse(dX) = sqrt( 1/n sum(dX_i**2) ) = sqrt( mse(dX) )
 
-        Renvoie la série, contenant à chaque pas, la RMSE des données au pas.
+        Renvoie la série contenant, à chaque pas, la RMSE des données au pas.
         Il faut que le type de base soit compatible avec les types élémentaires
         numpy. C'est réservé aux variables d'écarts ou d'incréments si le
         prédicteur est None, sinon c'est appliqué à l'écart entre les données
@@ -427,7 +470,7 @@ class Persistence(object):
         if _predictor is None:
             try:
                 __n = self.shape()[0]
-                return [(numpy.linalg.norm(item) / math.sqrt(__n)) for item in self.__values]
+                __sr = [(numpy.linalg.norm(item) / math.sqrt(__n)) for item in self.__values]
             except Exception:
                 raise TypeError("Base type is incompatible with numpy")
         else:
@@ -438,11 +481,12 @@ class Persistence(object):
                     raise ValueError("Predictor size at step %i is incompatible with the values"%i)
             try:
                 __n = self.shape()[0]
-                return [(numpy.linalg.norm(numpy.ravel(item) - numpy.ravel(_predictor[i])) / math.sqrt(__n)) for i, item in enumerate(self.__values)]
+                __sr = [(numpy.linalg.norm(numpy.ravel(item) - numpy.ravel(_predictor[i])) / math.sqrt(__n)) for i, item in enumerate(self.__values)]
             except Exception:
                 raise TypeError("Base type is incompatible with numpy")
+        return numpy.array(__sr).tolist()
 
-    rmsds = rmses # Root-Mean-Square Deviation (RMSD=RMSE)
+    rmsds = rmses  # Root-Mean-Square Deviation (RMSD=RMSE)
 
     def __preplots(self,
                    title    = "",
@@ -451,28 +495,29 @@ class Persistence(object):
                    ltitle   = None,
                    geometry = "600x400",
                    persist  = False,
-                   pause    = True,
-                  ):
+                   pause    = True ):
         "Préparation des plots"
         #
         # Vérification de la disponibilité du module Gnuplot
-        if not has_gnuplot:
+        if not lpi.has_gnuplot:
             raise ImportError("The Gnuplot module is required to plot the object.")
         #
         # Vérification et compléments sur les paramètres d'entrée
-        if persist:
-            Gnuplot.GnuplotOpts.gnuplot_command = 'gnuplot -persist -geometry '+geometry
-        else:
-            Gnuplot.GnuplotOpts.gnuplot_command = 'gnuplot -geometry '+geometry
         if ltitle is None:
             ltitle = ""
-        self.__g = Gnuplot.Gnuplot() # persist=1
-        self.__g('set terminal '+Gnuplot.GnuplotOpts.default_term)
+        __geometry = str(geometry)
+        __sizespec = (__geometry.split('+')[0]).replace('x', ',')
+        #
+        if persist:
+            Gnuplot.GnuplotOpts.gnuplot_command = 'gnuplot -persist '
+        #
+        self.__g = Gnuplot.Gnuplot()  # persist=1
+        self.__g('set terminal ' + Gnuplot.GnuplotOpts.default_term + ' size ' + __sizespec)
         self.__g('set style data lines')
         self.__g('set grid')
         self.__g('set autoscale')
-        self.__g('set xlabel "'+str(xlabel)+'"')
-        self.__g('set ylabel "'+str(ylabel)+'"')
+        self.__g('set xlabel "' + str(xlabel) + '"')
+        self.__g('set ylabel "' + str(ylabel) + '"')
         self.__title  = title
         self.__ltitle = ltitle
         self.__pause  = pause
@@ -489,8 +534,7 @@ class Persistence(object):
               filename = "",
               dynamic  = False,
               persist  = False,
-              pause    = True,
-             ):
+              pause    = True ):
         """
         Renvoie un affichage de la valeur à chaque pas, si elle est compatible
         avec un affichage Gnuplot (donc essentiellement un vecteur). Si
@@ -530,7 +574,8 @@ class Persistence(object):
             self.__preplots(title, xlabel, ylabel, ltitle, geometry, persist, pause )
             if dynamic:
                 self.__dynamic = True
-                if len(self.__values) == 0: return 0
+                if len(self.__values) == 0:
+                    return 0
         #
         # Tracé du ou des vecteurs demandés
         indexes = []
@@ -543,8 +588,8 @@ class Persistence(object):
         #
         i = -1
         for index in indexes:
-            self.__g('set title  "'+str(title)+' (pas '+str(index)+')"')
-            if isinstance(steps,list) or isinstance(steps,numpy.ndarray):
+            self.__g('set title  "' + str(title) + ' (pas ' + str(index) + ')"')
+            if isinstance(steps, (list, numpy.ndarray)):
                 Steps = list(steps)
             else:
                 Steps = list(range(len(self.__values[index])))
@@ -553,7 +598,7 @@ class Persistence(object):
             #
             if filename != "":
                 i += 1
-                stepfilename = "%s_%03i.ps"%(filename,i)
+                stepfilename = "%s_%03i.ps"%(filename, i)
                 if os.path.isfile(stepfilename):
                     raise ValueError("Error: a file with this name \"%s\" already exists."%stepfilename)
                 self.__g.hardcopy(filename=stepfilename, color=1)
@@ -564,9 +609,10 @@ class Persistence(object):
         """
         Affichage dans le cas du suivi dynamique de la variable
         """
-        if self.__dynamic and len(self.__values) < 2: return 0
+        if self.__dynamic and len(self.__values) < 2:
+            return 0
         #
-        self.__g('set title  "'+str(self.__title))
+        self.__g('set title  "' + str(self.__title))
         Steps = list(range(len(self.__values)))
         self.__g.plot( Gnuplot.Data( Steps, self.__values, title=self.__ltitle ) )
         #
@@ -597,7 +643,7 @@ class Persistence(object):
         """
         try:
             if numpy.version.version >= '1.1.0':
-                return numpy.asarray(self.__values).std(ddof=ddof,axis=0).astype('float')
+                return numpy.asarray(self.__values).std(ddof=ddof, axis=0).astype('float')
             else:
                 return numpy.asarray(self.__values).std(axis=0).astype('float')
         except Exception:
@@ -656,8 +702,7 @@ class Persistence(object):
              geometry = "600x400",
              filename = "",
              persist  = False,
-             pause    = True,
-            ):
+             pause    = True ):
         """
         Renvoie un affichage unique pour l'ensemble des valeurs à chaque pas, si
         elles sont compatibles avec un affichage Gnuplot (donc essentiellement
@@ -686,34 +731,36 @@ class Persistence(object):
         """
         #
         # Vérification de la disponibilité du module Gnuplot
-        if not has_gnuplot:
+        if not lpi.has_gnuplot:
             raise ImportError("The Gnuplot module is required to plot the object.")
         #
         # Vérification et compléments sur les paramètres d'entrée
-        if persist:
-            Gnuplot.GnuplotOpts.gnuplot_command = 'gnuplot -persist -geometry '+geometry
-        else:
-            Gnuplot.GnuplotOpts.gnuplot_command = 'gnuplot -geometry '+geometry
         if ltitle is None:
             ltitle = ""
-        if isinstance(steps,list) or isinstance(steps, numpy.ndarray):
+        if isinstance(steps, (list, numpy.ndarray)):
             Steps = list(steps)
         else:
             Steps = list(range(len(self.__values[0])))
-        self.__g = Gnuplot.Gnuplot() # persist=1
-        self.__g('set terminal '+Gnuplot.GnuplotOpts.default_term)
+        __geometry = str(geometry)
+        __sizespec = (__geometry.split('+')[0]).replace('x', ',')
+        #
+        if persist:
+            Gnuplot.GnuplotOpts.gnuplot_command = 'gnuplot -persist '
+        #
+        self.__g = Gnuplot.Gnuplot()  # persist=1
+        self.__g('set terminal ' + Gnuplot.GnuplotOpts.default_term + ' size ' + __sizespec)
         self.__g('set style data lines')
         self.__g('set grid')
         self.__g('set autoscale')
-        self.__g('set title  "'+str(title) +'"')
-        self.__g('set xlabel "'+str(xlabel)+'"')
-        self.__g('set ylabel "'+str(ylabel)+'"')
+        self.__g('set title  "' + str(title)  + '"')
+        self.__g('set xlabel "' + str(xlabel) + '"')
+        self.__g('set ylabel "' + str(ylabel) + '"')
         #
         # Tracé du ou des vecteurs demandés
         indexes = list(range(len(self.__values)))
-        self.__g.plot( Gnuplot.Data( Steps, self.__values[indexes.pop(0)], title=ltitle+" (pas 0)" ) )
+        self.__g.plot( Gnuplot.Data( Steps, self.__values[indexes.pop(0)], title=ltitle + " (pas 0)" ) )
         for index in indexes:
-            self.__g.replot( Gnuplot.Data( Steps, self.__values[index], title=ltitle+" (pas %i)"%index ) )
+            self.__g.replot( Gnuplot.Data( Steps, self.__values[index], title=ltitle + " (pas %i)"%index ) )
         #
         if filename != "":
             self.__g.hardcopy(filename=filename, color=1)
@@ -743,55 +790,57 @@ class Persistence(object):
             raise TypeError("Base type is incompatible with numpy")
 
     # ---------------------------------------------------------
-    def setDataObserver(self, HookFunction = None, HookParameters = None, Scheduler = None):
+    def setDataObserver(self, HookFunction = None, HookParameters = None, Scheduler = None, Order = None, OSync = True, DOVar = None):
         """
-        Association à la variable d'un triplet définissant un observer
+        Association à la variable d'un triplet définissant un observer.
 
-        Le Scheduler attendu est une fréquence, une simple liste d'index ou un
-        range des index.
+        Les variables Order et DOVar sont utilisées pour un observer
+        multi-variable. Le Scheduler attendu est une fréquence, une simple
+        liste d'index ou un range des index.
         """
         #
         # Vérification du Scheduler
         # -------------------------
         maxiter = int( 1e9 )
-        if isinstance(Scheduler,int):      # Considéré comme une fréquence à partir de 0
+        if isinstance(Scheduler, int):                # Considéré comme une fréquence à partir de 0
             Schedulers = range( 0, maxiter, int(Scheduler) )
-        elif isinstance(Scheduler,range):  # Considéré comme un itérateur
+        elif isinstance(Scheduler, range):            # Considéré comme un itérateur
             Schedulers = Scheduler
-        elif isinstance(Scheduler,(list,tuple)):   # Considéré comme des index explicites
-            Schedulers = [int(i) for i in Scheduler] # map( long, Scheduler )
-        else:                              # Dans tous les autres cas, activé par défaut
+        elif isinstance(Scheduler, (list, tuple)):    # Considéré comme des index explicites
+            Schedulers = [int(i) for i in Scheduler]  # Similaire à map( int, Scheduler )  # noqa: E262
+        else:                                         # Dans tous les autres cas, activé par défaut
             Schedulers = range( 0, maxiter )
         #
         # Stockage interne de l'observer dans la variable
         # -----------------------------------------------
-        self.__dataobservers.append( [HookFunction, HookParameters, Schedulers] )
+        self.__dataobservers.append( [HookFunction, HookParameters, Schedulers, Order, OSync, DOVar] )
 
     def removeDataObserver(self, HookFunction = None, AllObservers = False):
         """
         Suppression d'un observer nommé sur la variable.
 
-        On peut donner dans HookFunction la meme fonction que lors de la
+        On peut donner dans HookFunction la même fonction que lors de la
         définition, ou un simple string qui est le nom de la fonction. Si
         AllObservers est vrai, supprime tous les observers enregistrés.
         """
-        if hasattr(HookFunction,"func_name"):
+        if hasattr(HookFunction, "func_name"):
             name = str( HookFunction.func_name )
-        elif hasattr(HookFunction,"__name__"):
+        elif hasattr(HookFunction, "__name__"):
             name = str( HookFunction.__name__ )
-        elif isinstance(HookFunction,str):
+        elif isinstance(HookFunction, str):
             name = str( HookFunction )
         else:
             name = None
         #
-        i = -1
+        ih = -1
         index_to_remove = []
-        for [hf, hp, hs] in self.__dataobservers:
-            i = i + 1
-            if name is hf.__name__ or AllObservers: index_to_remove.append( i )
+        for [hf, _, _, _, _, _] in self.__dataobservers:
+            ih = ih + 1
+            if name is hf.__name__ or AllObservers:
+                index_to_remove.append( ih )
         index_to_remove.reverse()
-        for i in index_to_remove:
-            self.__dataobservers.pop( i )
+        for ih in index_to_remove:
+            self.__dataobservers.pop( ih )
         return len(index_to_remove)
 
     def hasDataObserver(self):
@@ -803,15 +852,14 @@ class SchedulerTrigger(object):
     Classe générale d'interface de type Scheduler/Trigger
     """
     __slots__ = ()
-    #
+
     def __init__(self,
                  simplifiedCombo = None,
                  startTime       = 0,
                  endTime         = int( 1e9 ),
                  timeDelay       = 1,
                  timeUnit        = 1,
-                 frequency       = None,
-                ):
+                 frequency       = None ):
         pass
 
 # ==============================================================================
@@ -826,7 +874,7 @@ class OneScalar(Persistence):
     pour conserver une signification claire des noms.
     """
     __slots__ = ()
-    #
+
     def __init__(self, name="", unit="", basetype = float):
         Persistence.__init__(self, name, unit, basetype)
 
@@ -835,7 +883,7 @@ class OneIndex(Persistence):
     Classe définissant le stockage d'une valeur unique entière (int) par pas.
     """
     __slots__ = ()
-    #
+
     def __init__(self, name="", unit="", basetype = int):
         Persistence.__init__(self, name, unit, basetype)
 
@@ -845,7 +893,7 @@ class OneVector(Persistence):
     pas utiliser cette classe pour des données hétérogènes, mais "OneList".
     """
     __slots__ = ()
-    #
+
     def __init__(self, name="", unit="", basetype = numpy.ravel):
         Persistence.__init__(self, name, unit, basetype)
 
@@ -854,7 +902,7 @@ class OneMatrice(Persistence):
     Classe de stockage d'une matrice de valeurs homogènes par pas.
     """
     __slots__ = ()
-    #
+
     def __init__(self, name="", unit="", basetype = numpy.array):
         Persistence.__init__(self, name, unit, basetype)
 
@@ -863,7 +911,7 @@ class OneMatrix(Persistence):
     Classe de stockage d'une matrice de valeurs homogènes par pas.
     """
     __slots__ = ()
-    #
+
     def __init__(self, name="", unit="", basetype = numpy.matrix):
         Persistence.__init__(self, name, unit, basetype)
 
@@ -874,7 +922,7 @@ class OneList(Persistence):
     "OneVector".
     """
     __slots__ = ()
-    #
+
     def __init__(self, name="", unit="", basetype = list):
         Persistence.__init__(self, name, unit, basetype)
 
@@ -891,7 +939,7 @@ class OneNoType(Persistence):
     volontairement, et pas du tout par défaut.
     """
     __slots__ = ()
-    #
+
     def __init__(self, name="", unit="", basetype = NoType):
         Persistence.__init__(self, name, unit, basetype)
 
@@ -905,7 +953,7 @@ class CompositePersistence(object):
     être ajoutés.
     """
     __slots__ = ("__name", "__StoredObjects")
-    #
+
     def __init__(self, name="", defaults=True):
         """
         name : nom courant
@@ -941,7 +989,8 @@ class CompositePersistence(object):
         """
         Stockage d'une valeur "value" pour le "step" dans la variable "name".
         """
-        if name is None: raise ValueError("Storable object name is required for storage.")
+        if name is None:
+            raise ValueError("Storable object name is required for storage.")
         if name not in self.__StoredObjects.keys():
             raise ValueError("No such name '%s' exists in storable objects."%name)
         self.__StoredObjects[name].store( value=value, **kwargs )
@@ -951,7 +1000,8 @@ class CompositePersistence(object):
         Ajoute dans les objets stockables un nouvel objet défini par son nom,
         son type de Persistence et son type de base à chaque pas.
         """
-        if name is None: raise ValueError("Object name is required for adding an object.")
+        if name is None:
+            raise ValueError("Object name is required for adding an object.")
         if name in self.__StoredObjects.keys():
             raise ValueError("An object with the same name '%s' already exists in storable objects. Choose another one."%name)
         if basetype is None:
@@ -963,7 +1013,8 @@ class CompositePersistence(object):
         """
         Renvoie l'objet de type Persistence qui porte le nom demandé.
         """
-        if name is None: raise ValueError("Object name is required for retrieving an object.")
+        if name is None:
+            raise ValueError("Object name is required for retrieving an object.")
         if name not in self.__StoredObjects.keys():
             raise ValueError("No such name '%s' exists in stored objects."%name)
         return self.__StoredObjects[name]
@@ -975,7 +1026,8 @@ class CompositePersistence(object):
         comporter les méthodes habituelles de Persistence pour que cela
         fonctionne.
         """
-        if name is None: raise ValueError("Object name is required for setting an object.")
+        if name is None:
+            raise ValueError("Object name is required for setting an object.")
         if name in self.__StoredObjects.keys():
             raise ValueError("An object with the same name '%s' already exists in storable objects. Choose another one."%name)
         self.__StoredObjects[name] = objet
@@ -984,7 +1036,8 @@ class CompositePersistence(object):
         """
         Supprime un objet de la liste des objets stockables.
         """
-        if name is None: raise ValueError("Object name is required for retrieving an object.")
+        if name is None:
+            raise ValueError("Object name is required for retrieving an object.")
         if name not in self.__StoredObjects.keys():
             raise ValueError("No such name '%s' exists in stored objects."%name)
         del self.__StoredObjects[name]
@@ -1019,7 +1072,8 @@ class CompositePersistence(object):
             usedObjs = []
             for k in objs:
                 try:
-                    if len(self.__StoredObjects[k]) > 0: usedObjs.append( k )
+                    if len(self.__StoredObjects[k]) > 0:
+                        usedObjs.append( k )
                 finally:
                     pass
             objs = usedObjs