IdentifiantMot de passe
Loading...
Mot de passe oublié ?Je m'inscris ! (gratuit)
Navigation

Inscrivez-vous gratuitement
pour pouvoir participer, suivre les réponses en temps réel, voter pour les messages, poser vos propres questions et recevoir la newsletter

Python Discussion :

Optimisation de code


Sujet :

Python

  1. #1
    Membre averti
    Homme Profil pro
    Ingénieur mécanique
    Inscrit en
    Octobre 2016
    Messages
    19
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Âge : 48
    Localisation : France, Indre et Loire (Centre)

    Informations professionnelles :
    Activité : Ingénieur mécanique
    Secteur : Industrie

    Informations forums :
    Inscription : Octobre 2016
    Messages : 19
    Par défaut Optimisation de code
    Bonjour,

    Après quelques mois pour apprendre le python, j'ai réussi à faire un code Python pour extraire des données d'un fichier XML (fichier Suunto, montre de sport avec GPS et fréquence cardiaque).
    Les données sont lues, traitées dans un dataframe, extrapolées, ....

    Le programme fonctionne mais j'essaye de l'optimiser sans trop de succès. Pouvez-vous svp jeter un coup d’œil sur mon code et me remonter les possibilités d’optimisation ? Je cherche à gagner du temps de calcul car ce soft, dans une future version, traitera plusieurs centaines de fichiers XML automatiquement.

    En passant de minidom à lxml, j'ai déjà gagné pas mal de temps.

    Merci d'avance.

    Ci-dessous le code :

    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
    # -*- coding: utf-8 -*-
    """
    Created on Sat Nov  5 23:01:26 2016
     
    @author: Remy
    """
     
    import pandas
    import matplotlib.pyplot as plt
    import numpy as numpy
    import logging
     
    from lxml import etree
     
    lignes = []
     
    def parseXML(nomfichier, ns):
        "Parser LXLM"
     
        tree = etree.parse(nomfichier)
        root = tree.getroot()
     
        for elem in root.iter(ns+'Sample'):
            try:
                UTCtime = elem.findtext(ns+'UTC')
            except ValueError:
                logging.info('%s, Erreur UTCtime', nomfichier)
                UTCtime = numpy.nan
     
            try:            
                time = elem.findtext(ns+'Time')
            except ValueError:
                logging.info('%s, Erreur Time', nomfichier)
                time = numpy.nan
     
            try:
                latitude = elem.findtext(ns+'Latitude')
            except ValueError:
                logging.info('%s, Erreur Latitude', nomfichier)
                latitude = numpy.nan
     
            try:
                longitude = elem.findtext(ns+'Longitude')
            except ValueError:
                logging.info('%s, Erreur Longitude', nomfichier)
                longitude = numpy.nan
     
            try:
                distance = elem.findtext(ns+'Distance')
            except ValueError:
                logging.info('%s, Erreur Distance', nomfichier)
                distance = numpy.nan
     
            try:
                speed = elem.findtext(ns+'Speed')
            except ValueError:
                logging.info('%s, Erreur Speed', nomfichier)
                speed = numpy.nan
     
            try:
                cadence = elem.findtext(ns+'Cadence')
            except ValueError:
                logging.info('%s, Erreur Cadence', nomfichier)
                cadence = numpy.nan
     
            try:
                altitude = elem.findtext(ns+'Altitude')
            except ValueError:
                logging.info('%s, Erreur Altitude', nomfichier)
                altitude = numpy.nan
     
            try:
                gpsaltitude = elem.findtext(ns+'GPSAltitude')
            except ValueError:
                logging.info('%s, Erreur GPSAltitude', nomfichier)
                gpsaltitude = numpy.nan
     
            try:
                heartrate = elem.findtext(ns+'HR')
            except ValueError:
                logging.info('%s, Erreur HR', nomfichier)
                heartrate = numpy.nan
     
            try:
                energy = elem.findtext(ns+'EnergyConsumption')
            except ValueError:
                logging.info('%s, Erreur EnergyConsumption', nomfichier)
                energy = numpy.nan
     
            try:
                temperature = elem.findtext(ns+'Temperature')
            except ValueError:
                logging.info('%s, Erreur Temperature', nomfichier)
                temperature = numpy.nan
     
            ligne = (UTCtime, time, latitude, longitude, distance, speed, cadence, altitude, gpsaltitude, heartrate, energy, temperature)                
            lignes.append(ligne)
     
    def main():
        "Programme principal"
        global df
     
        #Initialation du fichier Log
        logging.basicConfig(filename = 'test_parsing.log', filemode = 'w', level=logging.DEBUG)
     
        #Parser LXML
        namespace = '{http://www.suunto.com/schemas/sml}'
        nomfichier = 'D9F698461C000D00-2016-09-24T09_59_45-0.sml'
        parseXML(nomfichier, namespace)
     
        #Création d'un Dataframe Pandas avec le fichier parsé par lxml pour les données des Samples
        #Interpolation linéaire des données manquantes avec interpolate()
        #Les premières données manquantes sont recopiées en prenant les premières valides avec fillna()    
        #Les dernières données manquantes sont recopiées en prenant les dernières valides avec fillna() 
        colonnes = ['UTCtime', 'time', 'latitude', 'longitude', 'distance', 'speed', 'cadence', 'altitude', 'gpsaltitude', 'heartrate', 'energy', 'temperature']
        df = pandas.DataFrame(data = lignes, columns = colonnes)
        df.fillna(value=numpy.nan, inplace=True)
        df = df.interpolate(axis = 0).fillna(method = 'bfill').fillna(method = 'ffill')
     
        #On passe en "entier" les données qui peuvent être simplifiées
        #Les données en "float" sont arrondis à 2 décimales (hors latitude & longitude)   
        df['time'] = df['time'].astype(float).round(2)
        df['latitude'] = df['latitude'].astype(float)
        df['longitude'] = df['longitude'].astype(float)
        df['distance'] = df['distance'].astype(int)
        df["speed"] = df["speed"].astype(float).round(2)
        df["cadence"] = df["cadence"].astype(float).round(2)
        df['altitude'] = df['altitude'].astype(int).round(2)
        df['gpsaltitude'] = df['gpsaltitude'].astype(float).round(2)
        df['heartrate'] = df['heartrate'].astype(float).round(2)
        df['energy'] = df['energy'].astype(float).round(2)
        df['temperature'] = df['temperature'].astype(float).round(2)
     
        #Calcul de la moyenne des altitudes baro et GPS
        #L'altitude baro (plus stable) est recalée sur la moyenne GPS par offset
        altitudebaro = numpy.mean(df['altitude'])
        altitudegps = numpy.mean(df['gpsaltitude'])
        df['altitude'] = df['altitude'] + int(altitudegps - altitudebaro)
     
        #Graphs divers
        fig = plt.figure()
        plt.title('Tracé 2D')
        plt.plot(df['longitude'], df['latitude']) #, marker='o', markersize=1)
     
        fig = plt.figure()
        plt.title('Altitude Baro - GPS')
        plt.plot(df['altitude'])
        plt.plot(df['gpsaltitude'])
     
        fig = plt.figure()
        plt.title('Vitesse')
        plt.plot(df["speed"])
     
        fig = plt.figure()
        plt.title('Cadence')
        plt.plot(df['cadence'])
     
        fig = plt.figure()
        plt.title('Fréquence cardiaque')
        plt.plot(df['heartrate'])
     
    if __name__ == "__main__":
        main()

  2. #2
    Membre confirmé
    Homme Profil pro
    Ingénieur développement logiciels
    Inscrit en
    Octobre 2013
    Messages
    156
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Localisation : France

    Informations professionnelles :
    Activité : Ingénieur développement logiciels
    Secteur : Santé

    Informations forums :
    Inscription : Octobre 2013
    Messages : 156
    Par défaut
    Bonjour,

    Si le temps de traitement du fichier XML vous conviens, une manière d'optimiser ce code pour qu'il soit capable de traiter plusieurs centaines de fichiers de manière performante serai d'utiliser le module multiprocessing.
    Vous serai ainsi capable de traiter plusieurs fichiers en même temps.

    Au niveau de l'affichage graphique, vous utilisez matplotlib, personnellement, quand j'ai besoin de perf, j'utilise plus PyQtGraph, mais ça c'est selon le cas(A benchmarker pour être sûr) , et dans votre cas, c'est peut être pas une priorité.

    Aussi vous avez beaucoup de try except, si aucune exception c'est lancé, c'est performant, mais si vous avez beaucoup de chance que des exceptions soit lancées c'est peut être pas le mieux. Je vois que avec LXML, il est possible que spécifier une valeur par défaut dans findtext, peut être benchmarker aussi ces deux solutions pour choisir la plus performante.

    Code : Sélectionner tout - Visualiser dans une fenêtre à part
    1
    2
    3
    4
    5
    6
    7
    >>> from lxml import etree
    >>> node=etree.fromstring('<a><b>bum</b><b>ear</b><c/></a>')
    >>> node.findtext('b')
    'bum'
    >>> node.findtext('c')
    ''
    >>> node.findtext('c', default='Huh?')

  3. #3
    Membre émérite

    Homme Profil pro
    Ingénieur
    Inscrit en
    Août 2010
    Messages
    662
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Localisation : France

    Informations professionnelles :
    Activité : Ingénieur
    Secteur : Aéronautique - Marine - Espace - Armement

    Informations forums :
    Inscription : Août 2010
    Messages : 662
    Par défaut
    Salut,

    Pour savoir quoi optimiser vous pouvez essayer de profiler votre code en utilisant le module cProfile. Exemple:
    Code : Sélectionner tout - Visualiser dans une fenêtre à part
    1
    2
    3
    4
    5
    6
    7
    import cProfile
     
    profile = cProfile.Profile()
    profile.enable()
    main()
    profile.disable()
    profile.print_stats()
    La doc du module en question se trouvant ici.

    Vous devriez aussi penser à plus fragmenter votre code. La création du graph devrait selon moi être dans une fonction à part. De même certaines variables comme namespace devraient être placées en paramètre de la fonction main. Il faut éviter autant que possible les variables codées en durs. A la limite vous pouvez les placer en début de fichier en respectant la convention de nommage qui va bien: NAMESPACE = 'quelquechose'.

    Sinon si vous débutez c'est pas mal du tout. Continuez comme ça!

    Ju

  4. #4
    Membre averti
    Homme Profil pro
    Ingénieur mécanique
    Inscrit en
    Octobre 2016
    Messages
    19
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Âge : 48
    Localisation : France, Indre et Loire (Centre)

    Informations professionnelles :
    Activité : Ingénieur mécanique
    Secteur : Industrie

    Informations forums :
    Inscription : Octobre 2016
    Messages : 19
    Par défaut
    Merci pour les réponses.

    En utilisant le paramètre "default = numpy.nan" de findtext(), j'ai pu effectivement simplifier mes blocs "TRY - EXCEPT".

    Je commence aussi à jouer avec cProfile. Pas mal pour mesurer les améliorations.

    Si vous avez d'autres idées, je suis preneur.

    rmarion37

Discussions similaires

  1. optimiser le code d'une fonction
    Par yanis97 dans le forum MS SQL Server
    Réponses: 1
    Dernier message: 15/07/2005, 08h41
  2. Optimiser mon code ASP/HTML
    Par ahage4x4 dans le forum ASP
    Réponses: 7
    Dernier message: 30/05/2005, 10h29
  3. optimiser le code
    Par bibi2607 dans le forum ASP
    Réponses: 3
    Dernier message: 03/02/2005, 14h30
  4. syntaxe et optimisation de codes
    Par elitol dans le forum Langage SQL
    Réponses: 18
    Dernier message: 12/08/2004, 11h54
  5. optimisation du code et var globales
    Par tigrou2405 dans le forum ASP
    Réponses: 2
    Dernier message: 23/01/2004, 10h59

Partager

Partager
  • Envoyer la discussion sur Viadeo
  • Envoyer la discussion sur Twitter
  • Envoyer la discussion sur Google
  • Envoyer la discussion sur Facebook
  • Envoyer la discussion sur Digg
  • Envoyer la discussion sur Delicious
  • Envoyer la discussion sur MySpace
  • Envoyer la discussion sur Yahoo