-
Notifications
You must be signed in to change notification settings - Fork 266
Contresens de traduction dans heapq.po #1628
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
Comments
Bien vu ! Que pense-tu de #1629 ? Si c'est bon pour toi tu peux commenter dessus et on merge. |
Il faudra rétroporter vers 2.7 et les anciennes versions 3 (toutes impactées). |
Pour moi c'est bon, la modification proposée me semble correcte et répond tout à fait à mon problème. |
Automerge of PR #1629 by @christopheNan #1628 closes #1628 Merci pour la remontée du contresens.
Retroportage requis. |
C'est tout rétroporté \o/
(Si qq se demande pourquoi j'ai pas fait un for, c'est qu'avant chaque push je relisais :D) |
Et ça se botise en prenant en paramètres le commit à retroporter + la version cible ? |
Clairement ! Si ça arrive plus d'une fois par an ça s'automatise :) |
Le cas d'usage pourrait être toutes les traductions, non ? |
Tu veux dire un petit bot qui passe régulièrement : FROM=3.9
TO="2.7 3.5 3.6 3.7 3.8"
git switch $FROM
pomerge --from-files *.po */*.po
for branch in $TO; do
git switch $branch
pomerge --to-files *.po */*.po --no-overwrite
powrap -m
git add -u
git commit -m "Backporting from $FROM."
git push
done ? |
Oui. |
En lisant la doc de heapq de la bibliothèque, je pense qu'il y a un contresens de traduction en français (ligne 75 de heapq.po):
Ces deux points permettent d'aborder le tas comme une liste Python standard sans surprise : heap[0] est le plus petit élément tandis que heap.sort() ne modifie pas le tas !
Texte original en anglais:
These two make it possible to view the heap as a regular Python list without surprises: heap[0] is the smallest item, and heap.sort() maintains the heap invariant!
La traduction consière l'anglais 'invariant' comme un adjectif, alors qu'il doit être compris comme un nom. Plus simplement dit, heap.sort() conserve l'invariant de la liste (à savoir, heap[0] est le plus petit élément et heap[k]<=heap[2k+1] et heap[k]<=heap[2k+2]), mais ne conserve pas la liste invariante, elle peut être modifiée par l'appel à sort().
Un simple exemple montre que heap.sort() peut modifier le tas:
Je pense que la traduction correcte de la phrase pourrait être:
Ces deux points permettent d'aborder le tas comme une liste Python standard sans surprise : heap[0] est le plus petit élément tandis que heap.sort() conserve l'invariance du tas !
(l'expression "conservant l'invariance du tas" est utilisée plus bas dans la traduction).
The text was updated successfully, but these errors were encountered: