Project

General

Profile

Evolution #1979

Remplacer "minidom" par "etree" pour la lecture des dictionnaires

Added by Joël Cuissinat almost 13 years ago. Updated about 12 years ago.

Status:
Fermé
Priority:
Normal
Assigned To:
Gwenael Remond
Category:
-
Start date:
08/22/2011
Due date:
% Done:

100%

Estimated time:
10.00 h
Spent time:
Distribution:
EOLE 2.3

Description

Il y a de fortes chances pour qu'on y gagne en performance...

lxml_parser.py View (14.3 KB) Joël Cuissinat, 01/03/2012 04:29 PM

lxml_parser.py View - Nouvelle version avec des optimisation reste un pb avec <value></value> (13.7 KB) Joël Cuissinat, 01/18/2012 05:06 PM

Associated revisions

Revision ec1b92ed (diff)
Added by Walid Nouh over 14 years ago

see #1979 Can hide ticket category to post-only users (to be improved)

Revision 6b401213 (diff)
Added by Walid Nouh over 14 years ago

see #1979 remove TicketCategory::dropdown and add a condition parameter to dropdownValue

Revision aabb787d (diff)
Added by Joël Cuissinat over 12 years ago

Ajout du parseur recodé avec lxml (fixes #1979)
  • creole/lxml_parser.py : nouveau parseur
  • creole/dom_parser.py : ancien parseur (déprécié)

Revision a9784f3d (diff)
Added by Joël Cuissinat over 12 years ago

Utilisation du parseur lxml (ref #1979 @3h)
  • creole/cfgparser.py : modification des imports

Revision d257ab2c (diff)
Added by Joël Cuissinat over 12 years ago

Ménage dans cfgparser.py (ref #1979 @30m)

Revision 0e92598e (diff)
Added by Bruno Boiget over 12 years ago

correction du parsing lxml (separateurs sans texte) + retour read_string pour Zéphir (ref #1979 @30m)

Revision 6a0b5aff (diff)
Added by Bruno Boiget over 12 years ago

lxml_parser : correction du calcul de l'attribut mode si pas de valeur dans le xml (ref #1979)

Revision 54951f27 (diff)
Added by Joël Cuissinat over 12 years ago

lxml_parser : encore une erreur sur le calcul des separators (ref #1979)

History

#1 Updated by Joël Cuissinat over 12 years ago

  • Assigned To set to Gwenael Remond

#2 Updated by Joël Cuissinat over 12 years ago

  • Target version changed from Mises à jour 2.3 - 02 RC to Mises à jour 2.2.3 - 03 RC

#3 Updated by Joël Cuissinat over 12 years ago

  • Target version changed from Mises à jour 2.2.3 - 03 RC to Mises à jour 2.3 - 02 RC

#4 Updated by Joël Cuissinat over 12 years ago

  • Target version changed from Mises à jour 2.3 - 02 RC to Mises à jour 2.3 - 02 Stable

#5 Updated by Joël Cuissinat over 12 years ago

  • Target version changed from Mises à jour 2.3 - 02 Stable to Mises à jour 2.3 - 03 RC
  • Distribution set to EOLE 2.3

#6 Updated by Joël Cuissinat over 12 years ago

  • Target version changed from Mises à jour 2.3 - 03 RC to Mises à jour 2.3.4 RC
  • Estimated time set to 10.00 h

#7 Updated by Joël Cuissinat over 12 years ago

En remplaçant le dom_parser.py par le fichier lxml_parser.py ci-joint, la différence n'est pas évidente :(
=> à tester sur un AmonEcole

#8 Updated by Gwenael Remond over 12 years ago

pour que ça gagne du temps c'est moins le fait d'utiliser lxml par rapport à xml.etree, c'est surtout qu'il y a plein de findall() qu'il faut remplacer par du XPath, et là ça devrait gagner du temps.

#9 Updated by Joël Cuissinat over 12 years ago

  • % Done changed from 0 to 20

Je comprends.
Mais c'est déjà un exploit en soi d'avoir réussi à changer le code et que ça marche toujours !
Enfin, pas tout à fait car le test unitaire : test_redefine.py plante :(

#11 Updated by Joël Cuissinat over 12 years ago

  • Status changed from Nouveau to Résolu
  • % Done changed from 20 to 100

#12 Updated by Fabrice Barconnière about 12 years ago

  • Status changed from Résolu to Fermé

Also available in: Atom PDF