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

Translate file library/collections.abc.po #580

Open
wants to merge 25 commits into
base: 3.12
Choose a base branch
from

Conversation

mgiannopoulos24
Copy link
Contributor

@mgiannopoulos24 mgiannopoulos24 commented Jan 3, 2025

Ποιο issue κλείνει το συγκεκριμένο PR;

Closes #471

Περιγραφή του PR

Προστέθηκε η μετάφραση του collections.abc.po.

Λίστα Εργασιών

  • Προσθήκη λέξεων στο ευρετήριο main.txt.

@mgiannopoulos24 mgiannopoulos24 requested a review from a team as a code owner January 3, 2025 16:32
Copy link
Member

@skpanagiotis skpanagiotis left a comment

Choose a reason for hiding this comment

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

Καλή δουλειά, λίγο δύσκολο κείμενο με αρκετούς όρους δύσκουλους στη μετάφραση. Έχω κάνει κάποια suggestions. Μια παρατήρηση είναι η μετάφραση της λέξης co-routine.

library/collections.abc.po Outdated Show resolved Hide resolved
library/collections.abc.po Outdated Show resolved Hide resolved
library/collections.abc.po Outdated Show resolved Hide resolved
library/collections.abc.po Outdated Show resolved Hide resolved
library/collections.abc.po Outdated Show resolved Hide resolved
library/collections.abc.po Outdated Show resolved Hide resolved
library/collections.abc.po Outdated Show resolved Hide resolved
library/collections.abc.po Outdated Show resolved Hide resolved
library/collections.abc.po Show resolved Hide resolved
library/collections.abc.po Outdated Show resolved Hide resolved
@mgiannopoulos24
Copy link
Contributor Author

Να αλλάξω όλα τα co routines σε συρρουτίνες;

@skpanagiotis
Copy link
Member

Προσωπικά, δεν μου κολλάει και τόσο το συρρουτίνες. Επειδή ο όρος routine και συνεπώς coroutine είναι όροι της Python θα έλεγα μήπως το αφήναμε έτσι.

@mgiannopoulos24
Copy link
Contributor Author

Εντάξει θα τα αφήσω όλα coroutines.

Copy link
Member

@skpanagiotis skpanagiotis left a comment

Choose a reason for hiding this comment

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

Κάποια κενά που έχουν ξεφύγει. Κατά τα άλλα lgtm.

library/collections.abc.po Outdated Show resolved Hide resolved
library/collections.abc.po Outdated Show resolved Hide resolved
library/collections.abc.po Outdated Show resolved Hide resolved
library/collections.abc.po Outdated Show resolved Hide resolved
mgiannopoulos24 and others added 4 commits January 11, 2025 00:16
Co-authored-by: Panagiotis Skias <[email protected]>
Co-authored-by: Panagiotis Skias <[email protected]>
Co-authored-by: Panagiotis Skias <[email protected]>
Co-authored-by: Panagiotis Skias <[email protected]>
@mgiannopoulos24
Copy link
Contributor Author

Νόμιζα ότι το powrap τα κόβει, αλλά μάλλον όχι.

@skpanagiotis
Copy link
Member

skpanagiotis commented Jan 15, 2025

Θα το κάνω άλλο ένα review, αν και θα ήθελα να το κάνεις και κάποιος άλλο, και θα γίνει approve για να κλείσει. Πάντως μπράβο δύσκολο αρχείο 🎉

Copy link
Member

@gmargaritis gmargaritis left a comment

Choose a reason for hiding this comment

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

Πολύ καλή δουλεία 🤘

Αφήνω μερικές προτάσεις, LGTM overall!


#: library/collections.abc.rst:286
msgid "ABCs for read-only and mutable :ref:`sets <types-set>`."
msgstr ""
msgstr "ABCs για ανάγνωση μόνο και mutable :ref:`sets <types-set>`."
Copy link
Member

Choose a reason for hiding this comment

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

Εδώ δε μου κολλάει τόσο το ανάγνωση μόνο, γιατί πηγαίνει στο sets, i.e. ABCs για ανάγνωση μόνο sets.

Νομίζω θα προτιμούσα περισσότερο να μείνει read-only ως έχει, εκτός και αν χρησιμοποιούμε την έκφραση σε αντίστοιχες περιπτώσεις.

library/collections.abc.po Outdated Show resolved Hide resolved
@@ -633,13 +775,23 @@ msgid ""
"using mixins, inherit from both :meth:`Set` and :meth:`Hashable`, then "
"define ``__hash__ = Set._hash``."
msgstr ""
"Το mixin :class:`Set` παρέχει τη μέθοδο meth:`!_hash` για να υπολογίσει μια "
"τιμή κατακερματισμού για το σύνολο. Ωστόσο, η μέθοδος :meth:`~object."
Copy link
Member

Choose a reason for hiding this comment

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

Θα κρατούσα την αγγλική ορολογία εκτός και αν χρησιμοποιούμε τη συγκεκριμένη μετάφραση ευρέως

Suggested change
"τιμή κατακερματισμού για το σύνολο. Ωστόσο, η μέθοδος :meth:`~object."
"τιμή hash για το σύνολο. Ωστόσο, η μέθοδος :meth:`~object."

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Προσωπικά πιστεύω πως το τιμή κατακερματισμού είναι γενικά γνωστό ως έννοια οπότε θα το άφηνα ως έχει.

library/collections.abc.po Outdated Show resolved Hide resolved
library/collections.abc.po Outdated Show resolved Hide resolved

#: library/collections.abc.rst:286
msgid "ABCs for read-only and mutable :ref:`sets <types-set>`."
msgstr ""
msgstr "ABCs για ανάγνωση μόνο και mutable :ref:`sets <types-set>`."
Copy link
Member

Choose a reason for hiding this comment

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

Προτείνω να κρατήσουμε τη συνοχή στη μετάφραση mutable/immutable

Suggested change
msgstr "ABCs για ανάγνωση μόνο και mutable :ref:`sets <types-set>`."
msgstr "ABCs για ανάγνωση μόνο και μεταβλητά :ref:`sets <types-set>`."

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Στο αρχικό commit το είχα μεταφράσει στα Ελληνικά όμως ο Παναγιώτης μου είχε πει να το αφήσω όπως ήταν.

Comment λίγο πιο πάνω

Copy link
Member

Choose a reason for hiding this comment

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

My bad, δεν το είχα δει!

@skpanagiotis Βλέπω πως παρακάτω έχουμε immutable -> αμετάβλητα, οπότε θα έλεγα να κρατήσουμε είτε την αγγλική, είτε την ελληνική μετάφραση καθολικά για τις mutable/immutable


#: library/collections.abc.rst:262
msgid "ABCs for read-only and mutable :term:`sequences <sequence>`."
msgstr ""
msgstr "ABCs για ανάγνωση μόνο και mutable :term:`sequences <sequence>`."
Copy link
Member

Choose a reason for hiding this comment

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


#: library/collections.abc.rst:291
msgid "ABCs for read-only and mutable :term:`mappings <mapping>`."
msgstr ""
msgstr "ABCs για ανάγνωση μόνο και mutable :term:`mappings <mapping>`."
Copy link
Member

Choose a reason for hiding this comment

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

dictionaries/main.txt Outdated Show resolved Hide resolved
Co-authored-by: George Margaritis <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[3.12] Translate library/collections.abc.po
3 participants