Skip to content

Conversation

@MikaelMonjour
Copy link

En esperant que ce n'est pas trop mal fait

J'accepte les insultes ; )

Copy link
Collaborator

@Seluj78 Seluj78 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Relu rapidement, je ferai une plus grosse review plus tard

#: ../Doc/library/pickle.rst:2
msgid ":mod:`pickle` --- Python object serialization"
msgstr ""
msgstr ":mod:`pickle` --- Module Python de sérialisation en objet"
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Je comprends ca plutot comme Module de sérialisation d'objets Python
@JulienPalard, ton avis ?

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

pareil

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Merci de la reco, je me fais à l'outil donc je pensais qu'il fallait le laisser en anglais. Je retiens

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
msgstr ":mod:`pickle` --- Module Python de sérialisation en objet"
msgstr ":mod:`pickle` --- Module de sérialisation d'objets Python"
#: ../Doc/library/pickle.rst:10
msgid "**Source code:** :source:`Lib/pickle.py`"
msgstr ""
msgstr "**Source code:** :source:`Lib/pickle.py`"
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Unchanged ?

Suggested change
msgstr "**Source code:** :source:`Lib/pickle.py`"
msgstr "**Code source :** :source:`Lib/pickle.py`"
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Prise en main de l'outil toujours (je pensais que vous pouviez faire les modifs directement)
J'essaie de m'y mettre demain

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@MikaelMonjour du coup tu peux accepter directement dans github les propositions de modifications que on fait. CF https://help.github.com/en/articles/incorporating-feedback-in-your-pull-request
😄

#: ../Doc/library/pickle.rst:39
msgid "Relationship to other Python modules"
msgstr ""
msgstr "Relation par rapport aux autres modules python"
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
msgstr "Relation par rapport aux autres modules python"
msgstr "Relations aux autres modules Python"

Me semble plus clair et moins mot a mot

"The :mod:`pickle` module differs from :mod:`marshal` in several significant "
"ways:"
msgstr ""
"Le module :mod:`pickle`diffère du module :mod:`marshal` sur plusieurs "
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
"Le module :mod:`pickle`diffère du module :mod:`marshal` sur plusieurs "
"Le module :mod:`pickle` diffère du module :mod:`marshal` sur plusieurs "
"\"serialization\", \"marshalling,\" [#]_ or \"flattening\"; however, to "
"avoid confusion, the terms used here are \"pickling\" and \"unpickling\"."
msgstr ""
"Le module :mod:`pickle` implémente des protocole binaires pour permettre la "
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
"Le module :mod:`pickle` implémente des protocole binaires pour permettre la "
"Le module :mod:`pickle` implémente des protocoles binaires pour permettre la "
msgstr ""
"Le module :mod:`pickle` garde la trace des objets qu'il a déjà sérialisé, "
"pour faire en sorte que les prochaines références à cet objet ne soient plus "
"sérialisé à nouveau. :mod:`marshal` lui ne le fait pas. "
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sérialisées

Suggested change
"sérialisé à nouveau. :mod:`marshal` lui ne le fait pas. "
"sérialisées à nouveau. :mod:`marshal` ne le fait pas. "
"Python possède un module de bas niveau en sérialisation appelé :mod:"
"`marshal`, mais en général il est préférable d'utiliser :mod:`pickle` pour "
"sérialiser des objets Python. :mod:`marshal` existe principalement pour "
"supporter les fichiers Python finissant en :file:`.pyc`."
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
"supporter les fichiers Python finissant en :file:`.pyc`."
"supporter les fichiers Python :file:`.pyc`."
"Ça provoque des implications à la fois sur le partage d'objet et sur les "
"objets récursifs. Les objets récursifs sont des objets qui contiennent des "
"auto références. Celles-ci ne sont pas gérées par marshal. Essayer de lancer "
"des objets récursifs sur marshall va faire crasher votre interpréteur "
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
"des objets récursifs sur marshall va faire crasher votre interpréteur "
"une *marshallisation* d'objets récursifs va faire crasher votre interpréteur "
@deronnax
Copy link
Collaborator

@MikaelMonjour poke :)

@MikaelMonjour
Copy link
Author

MikaelMonjour commented Dec 20, 2018 via email

@JulienPalard
Copy link
Member

Bonjour @MikaelMonjour,

Oui c'est ta pull request, nous on ne peut faire que des suggestions, et c'est à toi de les accepter ou non.

Les suggestions sous forme de diff, tu peux les accepter depuis l'interface graphique en un clic. Les suggestions qu'on écrit en francais il faut aller réouvrir poedit, commit, et push. (pas besoin de réouvrir la pull request ou quoi que ce soit côté github).

@christaus
Copy link

christaus commented Jan 3, 2019 via email

msgstr ""
"Le module :mod:`pickle`n'est pas sécurisé contre les données erronées et "
"malicieusement construites. Ne jamais *unpickle* la donnée reçue à partir "
"d'une source non reconnue ou non authentique."
Copy link

@vvrav vvrav Feb 23, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
"d'une source non reconnue ou non authentique."
"d'une source non fiable ou non authentifiée."

(imho)

@deronnax
Copy link
Collaborator

salut @MikaelMonjour . Tu fais les modifs :) ?

"laquelle un flux d'octet (à partir d'un :term:`binary file` ou :term:`bytes-"
"like object`) est converti en retour en hiérarchie d'objet. *Pickling* (et "
"*unpickling*) sont alternativement connu sous le terme de \"sérialisation\" "
"ou de \"marshalling\". Cependant pour éviter la confusion les termes "
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
"ou de \"marshalling\". Cependant pour éviter la confusion les termes "
"ou de \"marshalisation\". Cependant pour éviter la confusion les termes "
"pyc` files."
msgstr ""
"Python possède un module de bas niveau en sérialisation appelé :mod:"
"`marshal`, mais en général il est préférable d'utiliser :mod:`pickle` pour "
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
"`marshal`, mais en général il est préférable d'utiliser :mod:`pickle` pour "
"`marshal`, mais en général il est préférable d'utiliser :mod:`pickle` pour "
"Shared objects remain shared, which can be very important for mutable "
"objects."
msgstr ""
"Ça provoque des implications à la fois sur le partage d'objet et sur les "
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
"Ça provoque des implications à la fois sur le partage d'objet et sur les "
"Cela a des implications à la fois sur le partage d'objet et sur les "
msgstr ""
"Ça provoque des implications à la fois sur le partage d'objet et sur les "
"objets récursifs. Les objets récursifs sont des objets qui contiennent des "
"auto références. Celles-ci ne sont pas gérées par marshal. Essayer de lancer "
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
"auto références. Celles-ci ne sont pas gérées par marshal. Essayer de lancer "
"auto références. Celles-ci ne sont pas supportées par marshal. Essayer de lancer "
"objets récursifs. Les objets récursifs sont des objets qui contiennent des "
"auto références. Celles-ci ne sont pas gérées par marshal. Essayer de lancer "
"des objets récursifs sur marshall va faire crasher votre interpréteur "
"python. Le partage d'objet s'effectue lorsque qu'il y a plusieurs références "
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
"python. Le partage d'objet s'effectue lorsque qu'il y a plusieurs références "
"Python. On parle de partage d'objet lorsque qu'il y a plusieurs références "
"des objets récursifs sur marshall va faire crasher votre interpréteur "
"python. Le partage d'objet s'effectue lorsque qu'il y a plusieurs références "
"au même objet à différents endroits dans la hiérarchie de l'objet en "
"sérialisation. Le module :mod:`pickle`récupères ce type d'objet seulement "
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
"sérialisation. Le module :mod:`pickle`récupères ce type d'objet seulement "
"sérialisation. Le module :mod:`pickle`récupère ce type d'objet seulement "
"de classes définies par les utilisateurs. :mod:`pickle` peut sauvegarder et "
"restaurer les instances de classes de manière transparente. Cependant la "
"définition de classe doit être importable et lancée dans le même module et "
"de la même manière que lors de son importation."
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Je crains que le terme importation ici soit ambigu, d'autant qu'à la ligne d'avant on parle de l'importabilité du module.
À voir s'il n'y a pas moyen de tourner ça de façon plus claire.

Copy link
Collaborator

@Seluj78 Seluj78 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

#: ../Doc/library/pickle.rst:2
msgid ":mod:`pickle` --- Python object serialization"
msgstr ""
msgstr ":mod:`pickle` --- Module Python de sérialisation en objet"
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
msgstr ":mod:`pickle` --- Module Python de sérialisation en objet"
msgstr ":mod:`pickle` --- Module de sérialisation d'objets Python"
#: ../Doc/library/pickle.rst:10
msgid "**Source code:** :source:`Lib/pickle.py`"
msgstr ""
msgstr "**Source code:** :source:`Lib/pickle.py`"
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@MikaelMonjour du coup tu peux accepter directement dans github les propositions de modifications que on fait. CF https://help.github.com/en/articles/incorporating-feedback-in-your-pull-request
😄

@JulienPalard
Copy link
Member

Mergé à la main, relu (tous les commentaires ont été pris en compte) et poussé sur la 3.7.

@MikaelMonjour
Copy link
Author

MikaelMonjour commented Mar 2, 2019 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

6 participants