Evolution #346: Improve parsing goal performances
Yo, J'ai passé le parsing i18n en multi-thread + quelques grosses optimisations. Du coup, le fait de pouvoir modifier les clefs à la volée n'est plus possible car incompatible avec du multi-thread... Je pense que cela n'est pas très grave car cette option n'est pas utilisée (en tout cas par aucun des lutins). On pourrait imaginer repenser ce système si quelqu'un le veut. Les performances sont au rendez-vous (en strictMode le mode le plus consommateur) : +-------------------------------------------------+ | projet | (1) | (2) | avant | après | +--------------------+-----+-----+--------+-------+ | isis-fish | 184 | 594 | 2,76m | 13,3s | +--------------------+-----+-----+--------+-------+ | Observe-entities | 51 | 66 | 5,6s | 3,24s | +--------------------+-----+-----+--------+-------+ | Observe-services | 15 | 61 | 10,9s | 2,19s | +--------------------+-----+-----+--------+-------+ | Observe-validation | 155 | 155 | 2,2s | 1,1s | +--------------------+-----+-----+--------+-------+ | Observe-swing | 115 | 173 | 32,6s | 8,4s | +--------------------+-----+-----+--------+-------+ | vradi-services | 0 | 52 | 54,86s | 2,39s | +--------------------+-----+-----+--------+-------+ | vradi-swing | 56 | 116 | 16,29s | 4,56s | +-------------------------------------------------+ (1) : nombre de fichiers avec clef i18n (2) : nombre de fichiers scannés En fait, avec l'ancien mode plus y'a de fichiers, plus c'est long Pire, plus y'a de clefs i18n, plus c'est long... Je lance la release 1.2 :) -- Tony Chemit -------------------- tél: +33 (0) 2 40 50 29 28 email: chemit@codelutin.com http://www.codelutin.com
Le Sun, 28 Mar 2010 22:34:37 +0200, Tony Chemit <chemit@codelutin.com> a écrit :
Yo,
J'ai passé le parsing i18n en multi-thread + quelques grosses optimisations.
Du coup, le fait de pouvoir modifier les clefs à la volée n'est plus possible car incompatible avec du multi-thread...
Je pense que cela n'est pas très grave car cette option n'est pas utilisée (en tout cas par aucun des lutins).
Tu as une idée s'il y a d'autres utilisateurs qui auraient du coup pu etre impactés? Après, le gain est en effet enorme, donc quasiment necessaire, mais la petite regression est domage si d'autres que nous utiliser les changements à la volée. (d'ailleurs, si nous ne l'utilisions pas, pourquoi etait-elle présente? Ya un historique à ça?) -- Yannick
Yannick Martel wrote:
Le Sun, 28 Mar 2010 22:34:37 +0200, Tony Chemit <chemit@codelutin.com> a écrit :
Yo,
J'ai passé le parsing i18n en multi-thread + quelques grosses optimisations.
Du coup, le fait de pouvoir modifier les clefs à la volée n'est plus possible car incompatible avec du multi-thread...
Je pense que cela n'est pas très grave car cette option n'est pas utilisée (en tout cas par aucun des lutins).
Tu as une idée s'il y a d'autres utilisateurs qui auraient du coup pu etre impactés? Après, le gain est en effet enorme, donc quasiment necessaire, mais la petite regression est domage si d'autres que nous utiliser les changements à la volée. (d'ailleurs, si nous ne l'utilisions pas, pourquoi etait-elle présente? Ya un historique à ça?)
Sinon peut-être mettre une option pour désactiver le multi-thread et activer la modification des clés à la volée. Ca permettrait de pas avoir de régression ;)
Le Mon, 29 Mar 2010 12:22:22 +0200, Jean Couteau <couteau@codelutin.com> a écrit :
Sinon peut-être mettre une option pour désactiver le multi-thread et activer la modification des clés à la volée. Ca permettrait de pas avoir de régression ;) Désolé c'est pas une régression pour moi, juste une fonctionnalité enlevé.
Pouvoir paramétrer le mode multi-thread, c'est possible, mais cela va alourdir et complexifier le code. de plus les optimisations ne portent pas uniquement sur le multi-thread, mais aussi sur le fait qu'on n'autorise pas la modification à la volée des clef i18n détecté : donc incompatible avec le mode à la volée. Une chose est sure c'est que personne ici (sauf julien, moi et eric) l'a jamais lancé (et même connu je suppose ;)) Donc pour moi c'est pas une grosse perte... Dans tous les cas, je ne suis même pas sûr que le mode à la volée fonctionne toujours en 1.1 :)
_______________________________________________ I18n-devel mailing list I18n-devel@list.nuiton.org http://list.nuiton.org/cgi-bin/mailman/listinfo/i18n-devel
-- Tony Chemit -------------------- tél: +33 (0) 2 40 50 29 28 email: chemit@codelutin.com http://www.codelutin.com
Le Mon, 29 Mar 2010 12:18:29 +0200, Yannick Martel <martel@codelutin.com> a écrit :
Le Sun, 28 Mar 2010 22:34:37 +0200, Tony Chemit <chemit@codelutin.com> a écrit :
Yo,
J'ai passé le parsing i18n en multi-thread + quelques grosses optimisations.
Du coup, le fait de pouvoir modifier les clefs à la volée n'est plus possible car incompatible avec du multi-thread...
Je pense que cela n'est pas très grave car cette option n'est pas utilisée (en tout cas par aucun des lutins).
Tu as une idée s'il y a d'autres utilisateurs qui auraient du coup pu etre impactés? je pense pas que quelqu'un l'ai jamais utilisé
Après, le gain est en effet enorme, donc quasiment necessaire, mais la petite regression est domage si d'autres que nous utiliser les changements à la volée. (d'ailleurs, si nous ne l'utilisions pas, pourquoi etait-elle présente? Ya un historique à ça?)
L'historique était de pouvoir traduire les nouvelles clefs à la volée, mais cela n'est pas pertinent à mon goût, voila comment je procède : - j'écris directement dans le code la clef i18n - je fais la traduction directement ou je traduis après coup Dans tous cas, je ne mets jamais directement des traductions dans le code. -- Tony Chemit -------------------- tél: +33 (0) 2 40 50 29 28 email: chemit@codelutin.com http://www.codelutin.com
Le Mon, 29 Mar 2010 12:24:27 +0200, Tony Chemit <chemit@codelutin.com> a écrit :
Le Mon, 29 Mar 2010 12:18:29 +0200, Yannick Martel <martel@codelutin.com> a écrit :
Le Sun, 28 Mar 2010 22:34:37 +0200, Tony Chemit <chemit@codelutin.com> a écrit :
Yo,
J'ai passé le parsing i18n en multi-thread + quelques grosses optimisations.
Du coup, le fait de pouvoir modifier les clefs à la volée n'est plus possible car incompatible avec du multi-thread...
Je pense que cela n'est pas très grave car cette option n'est pas utilisée (en tout cas par aucun des lutins).
Tu as une idée s'il y a d'autres utilisateurs qui auraient du coup pu etre impactés? je pense pas que quelqu'un l'ai jamais utilisé
Après, le gain est en effet enorme, donc quasiment necessaire, mais la petite regression est domage si d'autres que nous utiliser les changements à la volée. (d'ailleurs, si nous ne l'utilisions pas, pourquoi etait-elle présente? Ya un historique à ça?)
L'historique était de pouvoir traduire les nouvelles clefs à la volée, mais cela n'est pas pertinent à mon goût, voila comment je procède :
- j'écris directement dans le code la clef i18n - je fais la traduction directement ou je traduis après coup
Dans tous cas, je ne mets jamais directement des traductions dans le code.
Hello, Je vais compléter, en fait ça permettait dans ISIS de rechercher toutes les les chaînes non traduite, il y en avait un paquet au début. Je trouve juste dommage de supprimer une tel option. Julien
participants (4)
-
Jean Couteau -
Julien Ruchaud -
Tony Chemit -
Yannick Martel