Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

translated 'math.nextafter(x,y)' #1380

Open
wants to merge 1 commit into
base: 3.9
from
Open

translated 'math.nextafter(x,y)' #1380

wants to merge 1 commit into from

Conversation

@eviau
Copy link

eviau commented Aug 21, 2020

Bonjour,

Afin de me préparer à un atelier de traduction de la doc python-docs-fr, je voulais tester le processus. J'ai fait une très courte traduction de la description d'une seule fonction.

J'ai suivi les instructions au meilleur de mes connaissances.

Dites-moi ce qu'il en est 😃

Closes #1379 .

Copy link
Contributor

vpoulailleau left a comment

Merci pour cette contribution !

Si tu regardes le résultat de l'intégration continue (le 1 successful check), tu verras tout en bas que https://github.com/vpoulailleau/padpo se plaint d'un truc :

2020-08-21 20:10:30,083 [ERROR   ]            pofile.py(153):display_warnings     :: library/math.po: 384 [     NBSP     ] There should be a non-breakable space before ":": between ###Exemples### and ###:###

2020-08-21 20:10:30,083 [ERROR   ]            pofile.py(153):display_warnings     :: library/math.po: 388 [     NBSP     ] There should be a non-breakable space before ":": between ### math.inf) » augmente de valeur### and ###: vers l'infini positif.###

2020-08-21 20:10:30,083 [ERROR   ]            pofile.py(153):display_warnings     :: library/math.po: 393 [     NBSP     ] There should be a non-breakable space before ":": between ###, -math.inf) » baisse de valeur### and ###: vers l'infini négatif.###

2020-08-21 20:10:30,083 [ERROR   ]            pofile.py(153):display_warnings     :: library/math.po: 408 [     NBSP     ] There should be a non-breakable space before ":": between ###À consulter### and ###: « math.ulp ».###

En français, nous devons mettre des espaces insécables avant les deux points.

Je précise que je n'ai pas relu le reste (je vais me coucher 😉)

library/math.po Outdated Show resolved Hide resolved
library/math.po Outdated Show resolved Hide resolved
library/math.po Outdated Show resolved Hide resolved
library/math.po Outdated Show resolved Hide resolved
Copy link
Collaborator

awecx left a comment

Merci pour cette première contribution.
Je t'ai fait quelques remarques.
Quand des suggestions te sont faites, tu peux les « Add to batch » pour les grouper en un seul commit ; cela évite d'avoir une page « Conversation » trop chargée.

library/math.po Outdated Show resolved Hide resolved
library/math.po Outdated Show resolved Hide resolved

#: library/math.rst:243
msgid "``math.nextafter(x, math.copysign(math.inf, x))`` goes away from zero."
msgstr ""
"``math.nextafter(x, math.copysign(math.inf, x))`` augmente de valeur en "

This comment has been minimized.

@awecx

awecx Aug 22, 2020

Collaborator
Suggested change
"``math.nextafter(x, math.copysign(math.inf, x))`` augmente de valeur en "
"``math.nextafter(x, math.copysign(math.inf, x))`` augmente ou baisse de valeur, en "
>>> import math 
>>> x = -1
>>> math.nextafter(x, math.copysign(math.inf, x))
-1.0000000000000002

#: library/math.rst:245
msgid "See also :func:`math.ulp`."
msgstr ""
msgstr "À consulter : :func:`math.ulp`."

This comment has been minimized.

@awecx

awecx Aug 22, 2020

Collaborator

Nous mettons plus volontiers :

Suggested change
msgstr "À consulter : :func:`math.ulp`."
msgstr "Voir aussi : :func:`math.ulp`."
"Renvoie la prochaine valeur possible pour un flottant *x*, en se dirigeant "
"vers *y*."
Comment on lines 377 to 378

This comment has been minimized.

@awecx

awecx Aug 22, 2020

Collaborator

Ou bien

Suggested change
"Renvoie la prochaine valeur possible pour un flottant *x*, en se dirigeant "
"vers *y*."
"Renvoie la prochaine valeur possible pour un flottant *x*, « dans la direction » de "
"*y*."

Qu'en penses-tu ?

This comment has been minimized.

@eviau

eviau Aug 22, 2020

Author

peut-être: "Renvoie la prochaine valeur possible pour un flottant x, en se déplaçant vers y"?

This comment has been minimized.

@awecx

awecx Aug 24, 2020

Collaborator

Ou « Renvoie la valeur flottante consécutive à x, la plus proche de y » ?

This comment has been minimized.

@eviau

eviau Aug 24, 2020

Author

"Renvoie la valeur flottante consécutive à x, en se rapprochant de y" ?

This comment has been minimized.

@eviau

eviau Aug 24, 2020

Author

Pour être un peu pointilleuse, je ne sais pas si "successive" est plus appropriée que "consécutive": je pense que 'consécutive' suggère une progression dans le temps, ce qui n'est pas nécessairement ce qu'on souhaite ici. Qu'en penses-tu ?

This comment has been minimized.

@christopheNan

christopheNan Aug 24, 2020

Collaborator

On dit « deux entiers consécutifs », cela exprime l'ordre et pas forcément le temps.

This comment has been minimized.

@awecx

awecx Aug 24, 2020

Collaborator

« successif » ou « consécutif » me convient. C'est plus le « dans la direction de » et le « en se dirigeant » qui me chiffonnent (je ne trouve pas la doc originale très claire), ça jure un peu avec le reste de la doc du module qui est censé être rigoureux (c'est le module math après tout). Avec « le plus proche de » on s'éloigne de la source originale, mais le sens y est.
@christopheNan un avis ?

This comment has been minimized.

@christopheNan

christopheNan Aug 25, 2020

Collaborator

j'écrirais « dans la direction de ».

This comment has been minimized.

@eviau

eviau Aug 27, 2020

Author

Autre possibilité:
"En se dirigeant de x vers y, renvoie la valeur flottante consécutive à x".

On peut changer "se diriger" par l'une ou l'autre des suggestions précédentes.

This comment has been minimized.

@awecx

awecx Aug 27, 2020

Collaborator

Va pour « dans la direction de » si @christopheNan et moi sommes d'accord.

@eviau
Copy link
Author

eviau commented Aug 27, 2020

Merci pour vos commentaires ! 💐

Outre les discussions ouvertes, dont deux sont encore actives, y a-t-il d'autres changements à apporter ?

@awecx
Copy link
Collaborator

awecx commented Aug 27, 2020

Merci pour vos commentaires ! bouquet

Outre les discussions ouvertes, dont deux sont encore actives, y a-t-il d'autres changements à apporter ?

J'ai fermé les deux conversations concernant le verbe égaler. Il reste la dernière et c'est tout bon.

@eviau
Copy link
Author

eviau commented Aug 27, 2020

D'accord !

J'ai fait la mise-à-jour.

GitHub me refusait le push des changements effectués, je ne sais pas si j'ai bien fait de merger les autres changements effectués durant les derniers jours. Si cela ne convient pas, pour des raisons d'historique git ou autre, merci de me le mentionner, j'ajusterai en conséquence.

Merci pour vos commentaires et votre patience 😄

Copy link
Collaborator

christopheNan left a comment

Merci pour cette contribution. Quelques remarques.

@@ -418,6 +418,7 @@ et al. et autres, `à accorder
export exportation
expression expression
framework cadriciel
frozen package or set paquet ou ensemble figé

This comment has been minimized.

@christopheNan

christopheNan Aug 28, 2020

Collaborator
Suggested change
frozen package or set paquet ou ensemble figé
frozen (package or set) (paquet ou ensemble) figé
"Dans un monde d'affichages graphiques, on pourrait se demander \"pourquoi "
"s'embêter\" ? Il est vrai que les terminaux d'affichage à cellules de "
Comment on lines 83 to 84

This comment has been minimized.

@christopheNan

christopheNan Aug 28, 2020

Collaborator
Suggested change
"Dans un monde d'affichages graphiques, on pourrait se demander \"pourquoi "
"s'embêter\" ? Il est vrai que les terminaux d'affichage à cellules de "
"Dans un monde d'affichages graphiques, on pourrait se demander « pourquoi "
"s'embêter ? ». Il est vrai que les terminaux d'affichage caractères par "
@@ -68,6 +80,14 @@ msgid ""
"an X server. Another is tools such as OS installers and kernel "
"configurators that may have to run before any graphical support is available."
msgstr ""
"Dans un monde d'affichages graphiques, on pourrait se demander \"pourquoi "
"s'embêter\" ? Il est vrai que les terminaux d'affichage à cellules de "
"caractères sont une technologie obsolète, mais il existe des niches dans "

This comment has been minimized.

@christopheNan

christopheNan Aug 28, 2020

Collaborator
Suggested change
"caractères sont une technologie obsolète, mais il existe des niches dans "
"caractère sont une technologie obsolète, mais il existe des niches pour "
"Dans un monde d'affichages graphiques, on pourrait se demander \"pourquoi "
"s'embêter\" ? Il est vrai que les terminaux d'affichage à cellules de "
"caractères sont une technologie obsolète, mais il existe des niches dans "
"lesquelles la possibilité de faire des choses de fantaisie avec eux est "

This comment has been minimized.

@christopheNan

christopheNan Aug 28, 2020

Collaborator
Suggested change
"lesquelles la possibilité de faire des choses de fantaisie avec eux est "
"lesquelles la possibilité de faire des choses fantaisistes ainsi est "
"s'embêter\" ? Il est vrai que les terminaux d'affichage à cellules de "
"caractères sont une technologie obsolète, mais il existe des niches dans "
"lesquelles la possibilité de faire des choses de fantaisie avec eux est "
"encore précieuse. Une niche est sur les Unixes de petite taille ou "

This comment has been minimized.

@christopheNan

christopheNan Aug 28, 2020

Collaborator
Suggested change
"encore précieuse. Une niche est sur les Unixes de petite taille ou "
"encore précieuse. En exemple de niche, on peut citer les systèmes de type Unix de petite taille ou "

#: howto/curses.rst:170
msgid ""
"Windows are the basic abstraction in curses. A window object represents a "
"rectangular area of the screen, and supports methods to display text, erase "
"it, allow the user to input strings, and so forth."
msgstr ""
"Les fenêtres sont l'abstraction de base de *curses*. Un objet de fenêtre "
"représente une zone rectangulaire de l'écran et supporte des méthodes pour "

This comment has been minimized.

@christopheNan

christopheNan Aug 28, 2020

Collaborator
Suggested change
"représente une zone rectangulaire de l'écran et supporte des méthodes pour "
"représente une zone rectangulaire de l'écran qui gère des méthodes pour "
@@ -227,6 +336,12 @@ msgid ""
"newwin` function creates a new window of a given size, returning the new "
"window object. ::"
msgstr ""
"L'objet ``stdscr`` renvoyé par la fonction :func:`~curses.initscr` est un "
"objet de fenêtre qui couvre l'écran entier. De nombreux programmes peuvent "

This comment has been minimized.

@christopheNan

christopheNan Aug 28, 2020

Collaborator
Suggested change
"objet de fenêtre qui couvre l'écran entier. De nombreux programmes peuvent "
"objet fenêtre qui couvre l'écran entier. De nombreux programmes peuvent "
"n'avoir besoin que de cette fenêtre unique, mais vous pouvez diviser l'écran "
"en fenêtres plus petites, afin de les redessiner ou de les effacer "
"séparément. La fonction :func:`~curses.newwin` crée une nouvelle fenêtre "
"d'une taille donnée, retournant le nouvel objet fenêtre. : ::"

This comment has been minimized.

@christopheNan

christopheNan Aug 28, 2020

Collaborator
Suggested change
"d'une taille donnée, retournant le nouvel objet fenêtre. : ::"
"d'une taille donnée, renvoyant un nouvel objet fenêtre. ::"
@@ -237,6 +352,13 @@ msgid ""
"difference from most other computer applications, but it's been part of "
"curses since it was first written, and it's too late to change things now."
msgstr ""
"Notez que le système de coordonnées utilisé dans *curses* est inhabituel. "
"Les coordonnées sont toujours passées dans l'ordre *y,x*, et le coin "
"supérieur gauche d'une fenêtre est la coordonnée (0,0). Ceci rompt la "

This comment has been minimized.

@christopheNan

christopheNan Aug 28, 2020

Collaborator
Suggested change
"supérieur gauche d'une fenêtre est la coordonnée (0,0). Ceci rompt la "
"supérieur gauche d'une fenêtre a pour coordonnées (0,0). Ceci rompt la "
@@ -245,6 +367,10 @@ msgid ""
"sizes. Legal coordinates will then extend from ``(0,0)`` to ``(curses.LINES "
"- 1, curses.COLS - 1)``."
msgstr ""
"Votre application peut déterminer la taille de l'écran en utilisant les "
"variables :data:`curses.LINES` et :data:`curses.COLS` pour obtenir les "
"tailles *y* et *x*. Les coordonnées légales s'étendront alors de ``(0,0)`` "

This comment has been minimized.

@christopheNan

christopheNan Aug 28, 2020

Collaborator
Suggested change
"tailles *y* et *x*. Les coordonnées légales s'étendront alors de ``(0,0)`` "
"tailles *y* et *x*. Les coordonnées licites s'étendront alors de ``(0,0)`` "
@eviau
Copy link
Author

eviau commented Aug 28, 2020

Merci pour les commentaires @christopheNan - le seul hic, c'est que ce sont des commentaires sur des changements effectués par d'autres, sur d'autres commits. Je m'explique.

Comme je disais plus haut, GitHub ne me laissait pas pousser mes nouveaux changements tel quel, il a fallu que je merge les changements faits sur upstream durant la semaine, par d'autres personnes. Je ne sais pas quelle est votre façon de procéder lorsque ce genre de changements avec l'historique git se produit - d'où mon commentaire.

Est-ce que quelqu'un pourrait clarifier la situation concernant la bonne façon, pour python-docs-fr, d'apporter des changements à la PR quant il y a eu des changemetns au upstream entre-temps ?

Merci pour votre patience ! Avoir la réponse me sera bien utile pour l'atelier 😄

@vpoulailleau
Copy link
Contributor

vpoulailleau commented Aug 28, 2020

@eviau J'avais mis des explications ici https://github.com/vpoulailleau/support-traduction-doc-python en m'étant inspiré de la doc

@eviau
Copy link
Author

eviau commented Aug 28, 2020

@eviau J'avais mis des explications ici https://github.com/vpoulailleau/support-traduction-doc-python en m'étant inspiré de la doc

Merci pour ce document !

Donc si je comprends bien, pour cette PR, je...

  • retourne en arrière dans mes commits
  • je refetch ma branch library-math uniquement
  • je refais les changements, si nécessaire
  • je re-commit
  • je re-push vers origin/library-math

et ça devrait être bon ?

@vpoulailleau
Copy link
Contributor

vpoulailleau commented Aug 29, 2020

@eviau C'est la diapo 3.4.2.8 (page 65)

L'idée c'est de retourner dans ta branche (library-math) et de faire un git pull qui va récupérer les modifications faites en ligne. Et effectivement il faudra un push après le commit.

Quand tu feches, ça ne modifie rien, ça récupère l'historique d'autres dépôts. Par contre git pull fait un fetch suivi d'un merge (ou rebase si on lui demande)

@eviau
Copy link
Author

eviau commented Aug 29, 2020

@vpoulailleau Merci pour le lien vers la diapo exacte !
J'ai l'impression d'avoir fait la bonne chose selon cette diapo ( 3.4.2.8 )... donc je ne sais pas trop quoi faire avec les changements requis sur les commits qui ne sont pas les miens ?

Je suis désolée pour la confusion 😅 mais je comprends de moins en moins comment ramener ma PR à un état merge-able...

@eviau eviau force-pushed the eviau:library-math branch from c9058df to 64d6e7c Aug 29, 2020
@eviau
Copy link
Author

eviau commented Aug 29, 2020

Je ne suis pas sûre de ce qui s'est passé avant, concernant les changements qui n'étaient pas les miens, mais ! j'ai fait un rebase des changements originaux que je voulais effectuer sur upstream/3.9.

J'en ai profité pour écraser les commits et en faire un seul.

Tenez-moi au courant :)

Copy link
Collaborator

christopheNan left a comment

Merci !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

4 participants
You can’t perform that action at this time.