Translated howto/clinic up to Advanced part by glyg · Pull Request #187 · python/python-docs-fr

glyg · 2018-06-05T09:39:24Z

I'm not sure it is necessary to translate further, as this howto
is meant for core devs, who surely speak English...

I'm not sure it is necessary to translate further, as this howto is meant for core devs, who surely speak English...

JulienPalard · 2018-06-07T21:43:40Z

Hi thanks for your translation. What editor did you used? Your editor puts literal newlines in the translation every 80 characeters in the po files, which is really strange, can you please look at it?

glyg · 2018-06-08T06:09:11Z

ha that's emacs with PO mode (because I wrap lines in the editor for easier reading). I didn't know whether it was a problem. I'll look into removing them.
For reference, this is a known issue with po-mode:
https://www.emacswiki.org/emacs/PoMode

Will try po-mode+ and report.

glyg · 2018-06-08T07:11:54Z

Also for reference, sed was usefull:

sed  -i s/'\\n'/'\ '/g howto/clinic.po

JulienPalard · 2018-06-08T08:10:05Z

"Sed, c'est bien." :]

JulienPalard

Merci pour cette traduction, j'ai relevé quelques erreurs typographiques (des espaces qui manquent surtout), sinon la traduction semble bonne jusque là (j'ai relu les 3/4 je te laisse propager, ce sont les mêmes remarques qui reviennent).

JulienPalard · 2018-06-09T22:09:07Z

 "Argument Clinic usage."
 msgstr ""
+"Argument Clinic est un préprocesseur pour les fichiers C de CPython. "
+"Il permet d'automatiser les éléments standards lors de la rédaction du "


"éléments" ne m'aide pas à comprendre, quelque chose comme "tâches répétitives" à la place de "éléments standards" me semble plus clair.

JulienPalard · 2018-06-09T22:12:33Z

+"importante - on pourrait ré-écrire son générateur de code pour "
+"produire du code d'analyse d'argument adapté au mieux, plutôt que "
+"d'utiliser la bibliothèque d'analyse d'argument générique. On aurait "
+"ainsi l'analyse d'argument la plus rapide possible!)"


Il manque un espace insécable avant le point d'exclamation.

JulienPalard · 2018-06-09T22:13:02Z

 msgstr ""
+"De plus, Argument Clinic doit être suffisamment flexible pour "
+"s'accommoder d'approche différentes de l'analyse d'arguments. Il y a "
+"des fonctions dans python dont le traitement des arguments est très "


Une majuscule pour Python s'il vous plaît :]

JulienPalard · 2018-06-09T22:17:53Z

 "builtin.  With Argument Clinic, that's a thing of the past!"
 msgstr ""
+"Finalement, la motivation première d'Argument Clinic était de fournir "
+"des «signatures» pour l'introspection des composants natifs de "


Il manque deux espaces insécables : un après «, et un avant «.

JulienPalard · 2018-06-09T22:18:19Z

+"des «signatures» pour l'introspection des composants natifs de "
+"CPython. Précédemment, les fonctions d'introspection levaient une "
+"exception si vous passiez un composant natif. Grâce à Argument Clinic, "
+"ce comportement appartient au passé!"


Il manque un espace insécable avant le point d'exclamation.

JulienPalard · 2018-06-09T22:25:39Z

 #: ../Doc/howto/clinic.rst:154
 msgid "Let's dive in!"
-msgstr ""
+msgstr "En route!"


Espace insécable.

JulienPalard · 2018-06-09T22:28:01Z

 #: ../Doc/howto/clinic.rst:188
 msgid "Add the following boilerplate above the function, creating our block::"
-msgstr ""
+msgstr "Ajouter les lignes standard suivantes au dessus de votre fonction pour créer notre bloc: ::"


Il manque un espace avant ton :. Il existe deux bons moyens de traduire ::, soit un espace insécable suivi de ::, soit : ::. (dans le 2ème cas, c'est tout de même une bonne chose de mettre un espace insécable avant le premier :, alors autant utiliser le premier cas).

JulienPalard · 2018-06-09T22:29:13Z

 "paragraph consisting of a single 80-column line at the beginning of the "
 "docstring."
 msgstr ""
+"Si votre docstring ne contient pas de ligne «résumé», Argument Clinic "


Espaces insécables pour les «».

JulienPalard · 2018-06-09T22:29:27Z

 "docstring."
 msgstr ""
+"Si votre docstring ne contient pas de ligne «résumé», Argument Clinic "
+"va se plaindre. Assurons nous donc qu'il y en a une. La ligne «résumé» "


Espaces insécables.

JulienPalard · 2018-06-09T22:32:16Z

 "Python.  Check the name defined in the :c:type:`PyModuleDef` or :c:type:"
 "`PyTypeObject` as appropriate."
-msgstr ""
+msgstr "Le nom de la classe et du module doivent être les même que ceux vus par Python. Selon le cas, référez vous à :c:type:`PyModuleDef` ou :c:type:`PyTypeObject`"


À l'avenir c'est cool si tu "reflow" ton .po à 80 colonnes (mais effectivement pas avec des \n DANS la chaîne. Moi qui utilise emacs aussi, j'utilise le module pypi poindent juste avant de commit pour ça. Avoir tous la même limite de longueur de ligne permet de réduire les diffs (si chaque editeur reflow chaque fichier, tous les fichiers diffent en entier).

s/les même/les mêmes/
s/référez vous/référez-vous/

christopheNan

Merci pour la traduction.
Je mettrais un maximum de phrases au présent, même si souvent l'anglais (ou l'auteur) emploie le futur. Je trouve que c'est plus facile à lire.
Je traduirais les "should" par "devez" plutôt que "devriez".
Je n'ai pas terminé la relecture du document.
Cordialement

christopheNan · 2018-06-09T20:42:38Z

+"montre comment convertir votre première fonction en C de façon à ce "
+"qu'elle fonctionne avec Argument Clinic, avant d'introduire des usages "
+"plus avancés d'Argument Clinic. "



J'écrirais "votre première fonction C" car, à la première lecture, le "en" peut faire croire qu'il est rattaché à "convertir".

christopheNan · 2018-06-09T20:50:39Z

+"Argument Clinic est pour le moment considéré comme un outil interne à "
+"CPython.  Son utilisation n'est pas supportée pour des fichiers à "
+"l'extérieur de CPython, et ça compatibilité ascendante n'est pas "
+"garantie pour les versions futures. En d'autres termes, si vous êtes "


"Supporter" est un anglicisme. Je remplacerais par "Il n'est pas conçu pour gérer des fichiers ..."
s/ça compatibilité/sa compatibilité/

Merci pour vos remarques à tous les deux, je corrige. Sur ce paragraphe, je me demandais s'il fallait 'fixer" la référence à la version de python (remplacer 3.5 par 3.7 paraîtrai logique, mais alors il faudrait aussi le faire pour la version originale).

Effectivement c'est incohérent. Le plus important est d'ouvrir une issue sur bugs.python.org à ce sujet, après à toi de juger si tu corrige ou pas dans la traduction.

christopheNan · 2018-06-09T21:01:08Z

+"réglait pas leur problème - sans en créer de nouveaux. Il est donc de "
+"la première importance qu'Argument Clinic génère du code correct. On "
+"souhaiterai que le code soit aussi plus rapide. Au minimum, il ne doit "
+"pas introduire de régression sur la vitesse d'exécution. (Au bout du "


Je remplacerais "on souhaiterait" par "il est aussi souhaitable".
J'ajouterais "significative" à "régression sur la vitesse".

christopheNan · 2018-06-09T21:02:08Z

+"s'accommoder d'approche différentes de l'analyse d'arguments. Il y a "
+"des fonctions dans python dont le traitement des arguments est très "
+"étrange; le but d'Argument Clinic est de toutes les supporter. "



Je remplacerais "toutes les supporter" par "les gérer toutes".

christopheNan · 2018-06-09T21:03:06Z

+"des «signatures» pour l'introspection des composants natifs de "
+"CPython. Précédemment, les fonctions d'introspection levaient une "
+"exception si vous passiez un composant natif. Grâce à Argument Clinic, "
+"ce comportement appartient au passé!"


Il faut ajouter une espace insécable avant le point d'exclamation.

christopheNan · 2018-06-10T08:22:55Z

+"``[clinic]``, en enlevant tout le bazar qui en fait une chaîne de "
+"caractères correcte en C. Une fois que c'est fait, vous devriez "
+"avoir seulement le texte, aligné à gauche, sans ligne plus longue que "
+"80 caractères. (Argument Clinic préserve l'indentation à l'intérieur de la docstring.)"


Il me semble qu'en français, on met la parenthèse en incise dans la phrase et non comme phrase entière entre parenthèses.

christopheNan · 2018-06-10T08:23:16Z

 "``help()`` on your builtin in the future, the first line will be built "
 "automatically based on the function's signature.)"
 msgstr ""
+"Si l'ancienne docstring commençait par un ligne qui ressemble à une "


s/un ligne/une ligne/

christopheNan · 2018-06-10T08:24:27Z

 "docstring."
 msgstr ""
+"Si votre docstring ne contient pas de ligne «résumé», Argument Clinic "
+"va se plaindre. Assurons nous donc qu'il y en a une. La ligne «résumé» "


s/Assurons nous/Assurons-nous/

christopheNan · 2018-06-10T08:25:08Z

 "(Our example docstring consists solely of a summary line, so the sample code "
 "doesn't have to change for this step.)"
 msgstr ""
+"Dans notre exemple, la docstring consiste seulement en une ligne de "


s/consiste en/est composée de/ ?

christopheNan · 2018-06-10T08:26:48Z

+"Si c'est la première fois que ce module ou cette classe est utilisée "
+"avec Argument Clinic dans ce fichier C, vous devez déclarer votre "
+"module et/ou votre classe. Pour une bonne hygiène avec Argument "
+"Clinic, il vaut mieux faire ces déclaration quelque part en tête du "


Je traduirais "hygiene" par "bonnes pratiques".

christopheNan · 2018-06-10T20:24:39Z

 "change either."
 msgstr ""
+"Si vous exécutez de nouveau Argument Clinic sur ce même fichier, "
+"Argument Clinic supprimera la vieille sortie, et écriera la nouvelle "


s/ecriera/ecrira/ mais je mettrais plutôt le paragraphe au présent.

christopheNan · 2018-06-10T20:26:03Z

 "For the sake of clarity, here's the terminology we'll use with Argument "
 "Clinic:"
-msgstr ""
+msgstr "Par soucis de clarté, voilà la terminologie que nous emploierons :"


s/soucis/souci/ s'il n'y a que la clarté comme souci.

christopheNan · 2018-06-10T20:29:48Z

 "Python.  Check the name defined in the :c:type:`PyModuleDef` or :c:type:"
 "`PyTypeObject` as appropriate."
-msgstr ""
+msgstr "Le nom de la classe et du module doivent être les même que ceux vus par Python. Selon le cas, référez vous à :c:type:`PyModuleDef` ou :c:type:`PyTypeObject`"


s/les même/les mêmes/
s/référez vous/référez-vous/

christopheNan · 2018-06-10T20:31:43Z

 "convenience syntax intended to make porting old code into Argument Clinic "
 "easier."
 msgstr ""
+"Que fait «convertisseur»? Il établit à la fois le type de variable "


Il faut des espaces insécables avec les guillemets et avant le point d'interrogation.

christopheNan · 2018-06-10T20:33:43Z

 "easier."
 msgstr ""
+"Que fait «convertisseur»? Il établit à la fois le type de variable "
+"utilisé en C et la méthode pour la convertir la valeur Python en "


s/la convertir/convertir/

Je ne comprends pas la phrase corrigée.

christopheNan · 2018-06-10T20:37:37Z

 "file.  You'll need to include that in your original ``.c`` file, typically "
 "right after the clinic module block::"
 msgstr ""
+"Pour plus de visibilité, la plus part du code a été écrit dans un "


s/la plus part/la plupart/

christopheNan · 2018-06-10T20:39:06Z

 "macro defining the appropriate static :c:type:`PyMethodDef` structure for "
 "this builtin::"
-msgstr ""
+msgstr "Quatrièmement, à l'intérieur de la section sortie du bloc, vous trouverez une macro pré-processeur qui défini les structures statiques :c:type:`PyMethodDef` appropriées pour ce module natif : ::"


s/qui défini/qui définit/

christopheNan · 2018-06-10T20:40:07Z

+"fonction native. Supprimez le prototype de la fonction que vous "
+"modifiez, mais laissez l'accolade ouverte. Maintenant, supprimez tout "
+"le code d'analyse d'argument et les déclarations de toutes les "
+"variables auquel il assigne les arguments. Vous voyez que désormais, "


s/auquel/auxquelles/

christopheNan · 2018-06-10T20:40:42Z

+"variables auquel il assigne les arguments. Vous voyez que désormais, "
+"les arguments Python sont ceux de cette fonction  «impl»; si "
+"l'implémentation utilise des noms différents pour ces variables, "
+"corrigez les."


s/corrigez les/corrigez-les/

christopheNan · 2018-06-10T20:43:33Z

 "replace the existing static :c:type:`PyMethodDef` structure with the macro, "
 "*don't* add a comma to the end."
 msgstr ""
+"Notez que le corps de la macro contient un point virgule final. Donc, "


une virgule, non ?

Dans l'exemple donné en dessous de ce paragraphe, c'est bien un point-virgule à la fin, je ne sais pas si la doc est fausse ou si j'ai mal compris quelque chose

La phrase parle des macros générées par clinic, les *METHODDEF, ces macros ont une virgule a la fin, ce qui fait que quand on a plusieurs méthodes dans une struct PyMethodDef il n'est pas necessaire de séparer les macros *METHODDEF par des virgules, d'ailleurs dans l'exemple il ne met effectivement pas la virgule entre __PICKLE_PICKLER_DUMP_METHODDEF et
__PICKLE_PICKLER_CLEAR_MEMO_METHODDEF.

glyg · 2018-06-13T08:03:30Z

I still need to check for future tense over the doc, don't merge or read just yet

christopheNan · 2018-06-13T08:24:30Z

 "to the implementation.)"
 msgstr ""
+"Vous vous souvenez de la macro avec la structure :c:type:`PyMethodDef` pour "
+"cette fonction? Trouvez la structure :c:type:`PyMethodDef` existante pour "


Il manque une espace insécable avant le point d'interrogation.

christopheNan

Je propose quelques améliorations de style pour "l'alléger". À voir si tu les prends.

christopheNan · 2018-06-16T13:02:06Z

 "gets called at the bottom, with ``PyObject *args`` (and maybe ``PyObject "
 "*kwargs``) magically converted into the C variables and types you need."
 msgstr ""
+"Le premier objectif d'Argument Clinic est de prendre la responsabilité de "


s/prendre/assumer/ ?

Je mets 'prendre en charge' -- ça fait mieux passer le sens de 'take over responsability"

christopheNan · 2018-06-16T13:02:26Z

 msgstr ""
+"Le premier objectif d'Argument Clinic est de prendre la responsabilité de "
+"toute l'analyse d'arguments à l'intérieur de CPython. Cela signifie que si "
+"vous convertissez une fonction pour Argument Clinic, cette fonction ne "


s/pour/pour fonctionner avec/ ?

christopheNan · 2018-06-16T13:03:30Z

+"Le premier objectif d'Argument Clinic est de prendre la responsabilité de "
+"toute l'analyse d'arguments à l'intérieur de CPython. Cela signifie que si "
+"vous convertissez une fonction pour Argument Clinic, cette fonction ne "
+"devrait plus du tout analyser ses propres arguments. Le code généré par "


s/ne devrait plus du tout/n'a plus (besoin de| à)/ ?

christopheNan · 2018-06-16T13:03:58Z

+"toute l'analyse d'arguments à l'intérieur de CPython. Cela signifie que si "
+"vous convertissez une fonction pour Argument Clinic, cette fonction ne "
+"devrait plus du tout analyser ses propres arguments. Le code généré par "
+"Argument Clinic devrait être une « boîte noire » avec en entrée l'appel de "


s/devrait être/devient/ ?

christopheNan · 2018-06-16T13:04:25Z

+"vous convertissez une fonction pour Argument Clinic, cette fonction ne "
+"devrait plus du tout analyser ses propres arguments. Le code généré par "
+"Argument Clinic devrait être une « boîte noire » avec en entrée l'appel de "
+"CPython, et en sortie l'appel à votre code. Entre les deux ``PyObject "


Je mettrais une virgule après "Entre les deux".

christopheNan · 2018-06-16T13:11:36Z

 #: ../Doc/howto/clinic.rst:80
 msgid "Basic Concepts And Usage"
-msgstr ""
+msgstr "Concepts de base et utilisation"


Je mettrais les verbes encore au futur au présent dans cette section.

christopheNan · 2018-06-16T13:13:07Z

 "scenarios.  But these are advanced topics—let's do something simpler for "
 "your first function."
 msgstr ""
+"ou s'il y a de multiples appels à :c:func:`PyArg_ParseTuple`, vous devriez "


s/vous devriez choisir/choisissez plutôt/ ?

christopheNan · 2018-06-16T13:14:06Z

 #: ../Doc/howto/clinic.rst:188
 msgid "Add the following boilerplate above the function, creating our block::"
 msgstr ""
+"Ajouter les lignes standard suivantes au-dessus de votre fonction pour créer "


s/Ajouter/Ajoutez/ pour être cohérent avec les autres étapes.

christopheNan · 2018-06-16T13:15:06Z

 msgstr ""
+"Coupez la docstring et collez la entre les lignes commençant par "
+"``[clinic]``, en enlevant tout le bazar qui en fait une chaîne de caractères "
+"correcte en C. Une fois que c'est fait, vous devriez avoir seulement le "


le conditionnel me paraît superflu. Je passerais au présent : s/vous devriez/vous devez/

christopheNan · 2018-06-16T13:16:29Z

 "paragraph consisting of a single 80-column line at the beginning of the "
 "docstring."
 msgstr ""
+"Si votre docstring ne contient pas de ligne « résumé », Argument Clinic va "


Là encore, je passerais le futur au présent et j'enlèverais le conditionnel

JulienPalard · 2018-06-17T08:10:17Z

 msgstr ""
+"Quand vous déclarez une classe, vous devez aussi spécifier deux aspects de "
+"son type en C: la déclaration de type que vous utiliseriez pour un pointeur "
+"vers une instance de cette classe et un pointeur vers :c:le type:"


s/:c:le type:/:c:type:/

JulienPalard · 2018-06-17T08:10:44Z

 "only."
 msgstr ""
+"Si la fonction C existante appelle :c:func:`PyArg_ParseTuple` (et pas :\" "
+"\"c:func:`PyArg_ParseTupleAndKeywords`), alors tous ses arguments sont "


Petit souci de guillemets dans le :c:func:.

glyg · 2018-06-17T08:22:20Z

+"CPython, vous devez certainement pousser plus loin la conversion, en "
+"utilisant les concepts avancés que nous verrons plus loin dans ce document "
+"(comme ``return converters`` et ``self converters``). Mais concentrons nous "
+"pour le moment sur les choses simples."


Je remets au futur "vous devrez certainement", là c'est bien un futur je pense.

Pour information, la traduction m'a amené à créer cette [PR](python/cpython#108848) dans CPython. Reviewed-on: https://git.afpy.org/AFPy/python-docs-fr/pulls/187 Reviewed-by: Julien Palard <julien@palard.fr> Co-authored-by: Christophe Nanteuil <christophe.nanteuil@gmail.com> Co-committed-by: Christophe Nanteuil <christophe.nanteuil@gmail.com>

$@glyg$

Translated howto/clinic up to Advanced part

1508409

I'm not sure it is necessary to translate further, as this howto is meant for core devs, who surely speak English...

$@glyg$ glyg mentioned this pull request Jun 5, 2018

Working on howto/clinic.po #183

Closed

$@glyg$

removes spurious newlines

19733a5

$JulienPalard$

JulienPalard requested changes Jun 9, 2018

View reviewed changes

$christopheNan$

christopheNan reviewed Jun 10, 2018

View reviewed changes

$christopheNan$

christopheNan suggested changes Jun 10, 2018

View reviewed changes

glyg added 2 commits June 13, 2018 09:00

$@glyg$

Corrects errors pointed by @christopheNan and @JulienPalard

28cfc46

$@glyg$

merged

ab5e6b0

$christopheNan$

christopheNan reviewed Jun 13, 2018

View reviewed changes

$@glyg$

some more fixes

a41773a

$christopheNan$

christopheNan reviewed Jun 16, 2018

View reviewed changes

$@glyg$

fixes @christopheNan remarks

0f06b87

$JulienPalard$

JulienPalard requested changes Jun 17, 2018

View reviewed changes

$glyg$

glyg commented Jun 17, 2018

View reviewed changes

$@glyg$

hopefully final revision

b42b990

$@JulienPalard$

JulienPalard merged commit acd49a3 into python:3.6 Jun 17, 2018

Uh oh!

Uh oh!

Conversation

glyg commented Jun 5, 2018

Uh oh!

JulienPalard commented Jun 7, 2018

Uh oh!

glyg commented Jun 8, 2018 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

glyg commented Jun 8, 2018

Uh oh!

JulienPalard commented Jun 8, 2018

Uh oh!

JulienPalard left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

christopheNan left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

$@glyg$ glyg commented Jun 5, 2018

glyg commented Jun 8, 2018 •

edited

Loading

$@JulienPalard$ JulienPalard left a comment

$@christopheNan$ christopheNan left a comment

glyg commented Jun 13, 2018 •

edited

Loading

$@christopheNan$ christopheNan left a comment