Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Traducción delibrary/itertools.po#451

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
clacri merged 20 commits intopython:3.8fromjcpmmx:traduccion-itertools
Aug 27, 2020

Conversation

jcpmmx
Copy link
Contributor

@jcpmmxjcpmmx commentedJun 26, 2020
edited
Loading

Closes#301

@jcpmmxjcpmmx marked this pull request as draftJune 26, 2020 01:40
@jcpmmxjcpmmx changed the titleTraducción itertools.poTraducción delibrary/itertools.poJun 26, 2020
@cmaureir
Copy link
Collaborator

@jcpmmx si necesitas ayuda, acá estamos! este finde estamos en un sprint de traducción asi que cualquier cosa nos dices!

@clacri
Copy link
Contributor

Hola@jcpmmx me gustaría revisar esta traducción ya que itertools es algo que empleo a menudo y me interesa mucho el tema :) como veo que está en draft prefiero preguntarte si ya aceptas review de cositas que puedas aplicar a lo que queda de traducción. Un saludo!

@jcpmmx
Copy link
ContributorAuthor

Gracias@cmaureir, justo acabo de leer esto.
Voy a revisar qué me falta y reporto mi progreso.

@jcpmmx
Copy link
ContributorAuthor

Hola@clacri, claro!
La tengo en draft porque aun no he terminado, pero bienvenidos los reviews parciales.

@cmaureir
Copy link
Collaborator

@jcpmmx te podemos ayudar en algo para continuar con el PR?

@jcpmmx
Copy link
ContributorAuthor

Gracias a todos.
Ya el PR está listo.

@jcpmmxjcpmmx marked this pull request as ready for reviewAugust 10, 2020 15:36
@clacri
Copy link
Contributor

Muy bien, a partir de mañana le voy echando vistacitos para revisarlo :) muchisimas gracias por tu trabajo

@jcpmmx
Copy link
ContributorAuthor

Vale, encantado en ayudar y perdón por la demora!

@clacri
Copy link
Contributor

hey@jcpmmx una primera cosita que puedes ir mirando es la solución de los errores del build por los fallos de pospell. Estás familiarizado con el sistema que utilizamos¿ en la carpeta dictionaries de nuestro repositorio hay un diccionario por cada fichero .po que se traduce, y contiene las palabras que son excepciones (tu sabes que son correctas asi q aunque no esten en el diccionario de español quieres que el chequeo las ignore). Si te fijas en tu build de travis, aun tendrías que o bien incluir las siguientes, o bien corregirlas porque son realmente errores:

library/itertools.po:31:Haskell
library/itertools.po:221:pred
library/itertools.po:233:seq
library/itertools.po:289:pred
library/itertools.po:305:itn
library/itertools.po:517:lengitud
library/itertools.po:633:paramétros
library/itertools.po:751:step
library/itertools.po:879:aue

@jcpmmx
Copy link
ContributorAuthor

@clacri gracias! No he podido hacer funcionarhunspell localmente en MacOS (al correrlo simplemente me muestra todas las palabras del.po) pero corregí las que me mostraste arriba.

@clacri
Copy link
Contributor

clacri commentedAug 10, 2020
edited
Loading

Bueno como ves ahora pasa el build :) lo del hunspell, que problema has tenido? quizás te podemos ayudar por aqui o por telegram. Has visto las instrucciones enhttps://github.com/python/python-docs-es/blob/3.8/.overrides/faq.rst ???? yo uso mac y lo conseguí configurar, quizás podemos verlo juntos.

@jcpmmx
Copy link
ContributorAuthor

@clacri pude instalar tantopospell comohunspell, pero al darle...

pospell -p dictionaries/library_itertools.txt -l es_ES library/itertools.po

simplemente me devuelve todas las palabras de mi traducción...

...library/itertools.po:997:usolibrary/itertools.po:997:variableslibrary/itertools.po:997:velocidadlibrary/itertools.po:997:volumenlibrary/itertools.po:997:y...

@cmaureir
Copy link
Collaborator

@clacri pude instalar tantopospell comohunspell, pero al darle...

pospell -p dictionaries/library_itertools.txt -l es_ES library/itertools.po

No estás generando el diccionario global,
solo ejecuta esto:

awk 1 dict dictionaries/*.txt > dict.txtpospell -p dict.txt -l es_ES library/itertools.po

@jcpmmx
Copy link
ContributorAuthor

@cmaureir gracias por la respuesta.
Hice asi, pero tengo el mismo resultado :/

@cmaureir
Copy link
Collaborator

Si te das cuenta, el build en travis pasa, que hace exactamente eso, entonces hay algo que puede que no esté bien localmente en tu configuración.
Puedes intentar hacer un merge de upstream/3.8 en tu rama local:https://python-docs-es.readthedocs.io/es/3.8/faq.html#mantener-actualizada
y luego intentar esos pasos que te di, y nos envias una imagen o texto de todo lo que sale de ambos comandos.

Copy link
Contributor

@clacriclacri left a comment

Choose a reason for hiding this comment

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

Voy por el párrafo 45

Copy link
Contributor

@clacriclacri left a comment

Choose a reason for hiding this comment

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

Voy por el párrafo 96

Copy link
Contributor

@clacriclacri left a comment

Choose a reason for hiding this comment

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

voy por el párrafo 174


#: ../Doc/library/itertools.rst:90
msgid ""
"The following module functions all construct and return iterators. Some "
"provide streams of infinite length, so they should only be accessed by "
"functions or loops that truncate the stream."
msgstr ""
"Todas las funciones del siguiente módulo construyen y retornan iteradores. "
"Algunas proveen flujos infinitos, por lo que deberían ser sólo manipuladas "
"por funciones o bucles que cortan el flujo."

#: ../Doc/library/itertools.rst:96
Copy link
Contributor

Choose a reason for hiding this comment

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

Hola! No sé porqué, pero al revisar en paralelo tus cambios y el build de read the docs de tu traducción, parece que este párrafo no está traducido. Aqui el screenshot:

Screenshot 2020-08-16 at 10 14 14

quizás@cmaureir nos puede echar una mano porque en este caso desde luego no parece ser por el fuzzy.


#: ../Doc/library/itertools.rst:161
msgid ""
"See :func:`functools.reduce` for a similar function that returns only the "
"final accumulated value."
msgstr ""
"Para una función similar que retorne únicamente el valor final acumulado, "
"revisa :func:`functools.reduce`."

#: ../Doc/library/itertools.rst:166
Copy link
Contributor

Choose a reason for hiding this comment

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

aqui tambien parece haber algo raro en el rendering del .po en el read the docs
Screenshot 2020-08-16 at 10 20 23 solo aparece uno de los dos mensajes

Copy link
Contributor

@clacriclacri left a comment

Choose a reason for hiding this comment

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

voy por el párrafo 310

@jcpmmx
Copy link
ContributorAuthor

Gracias@clacri, tan pronto termines el review paso a revisarlo

Copy link
Contributor

@clacriclacri left a comment

Choose a reason for hiding this comment

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

voy por el 389

Copy link
Contributor

@clacriclacri left a comment

Choose a reason for hiding this comment

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

Hola! Ya terminé la revisión. Muy buen trabajo! Cuando hayas mirado mis comentarios y cambiado lo que consideres, me volveré a dar una pasada rapida sobre la version ya renderizada del read the docs. Muchísimas gracias por la traducción, es un módulo muy interesante

@jcpmmx
Copy link
ContributorAuthor

@clacri nueva versión!

@clacri
Copy link
Contributor

Muy bien, me lo reviso y te comento, posiblemente esta noche o mañana.

clacri
clacri previously approved these changesAug 23, 2020
@clacri
Copy link
Contributor

Lo he aprobado pero quiero revisar el build porque tras el merging con 3.8 parece que habia algun fallo.

@clacri
Copy link
Contributor

Vale, he visto los detalles del build en travis y es solo powrap, así que te lo voy a pasar yo misma para poder hacer el merge. Por cierto que si quieres ayuda para configurarte pre-commit para hacer los mismos checks que el build pero en local antes de hacer commit te echamos una manilla :)

@clacri
Copy link
Contributor

He pasado powrap pero al intentar hacer commit mi pre-commit ha encontrado que todas estas palabras no están en el diccionario:

library/itertools.po:41:álbegra
library/itertools.po:124:pfinal
library/itertools.po:140:elem
library/itertools.po:140:elem
library/itertools.po:140:elem
library/itertools.po:176:pfinal
library/itertools.po:233:elem
library/itertools.po:249:key
library/itertools.po:249:v
library/itertools.po:261:start
library/itertools.po:261:stop
library/itertools.po:305:it
library/itertools.po:305:it
library/itertools.po:428:adicionable
library/itertools.po:725:ignordos
library/itertools.po:725:milti
library/itertools.po:997:vectorizadas
library/itertools.po:41:álbegra
library/itertools.po:124:pfinal
library/itertools.po:140:elem
library/itertools.po:140:elem
library/itertools.po:140:elem
library/itertools.po:176:pfinal
library/itertools.po:233:elem
library/itertools.po:249:key
library/itertools.po:249:v
library/itertools.po:261:start
library/itertools.po:261:stop
library/itertools.po:305:it
library/itertools.po:305:it
library/itertools.po:428:adicionable
library/itertools.po:725:ignordos
library/itertools.po:725:milti
library/itertools.po:997:vectorizadas

¿puedes echarles un vistazo y corregirlas? No entiendo muy bien porque no te aparecían antes.

@clacriclacri dismissed theirstale reviewAugust 23, 2020 08:53

My pre-commit shows errors in some words that are not failing in the build

@jcpmmx
Copy link
ContributorAuthor

@clacri sigo sin poder ver los resultados depospell localmente, como reporté en#451 (comment)

El build esta pasando sin problema, aunque sigo viendo el error que ya habías notado enhttps://python-docs-es--451.org.readthedocs.build/es/451/library/itertools.html#itertools.accumulate

@clacri
Copy link
Contributor

No te preocupes, vamos a hacer una cosa, me lo traigo yo localmente tu PR y le paso lo que falte para poder hacer merge ya.

@clacriclacri merged commite87d971 intopython:3.8Aug 27, 2020
@jcpmmx
Copy link
ContributorAuthor

Gracias@clacri!
Se puede borrar el branch?

@clacri
Copy link
Contributor

Si, ahora ya está mergeado. A ver si yo hago limpieza de branches propias mañana que tengo demasiadas!! Enhorabuena y gracias por el esfuerzo!!!

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

@clacriclacriclacri approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Translatelibrary/itertools.po
3 participants
@jcpmmx@cmaureir@clacri

[8]ページ先頭

©2009-2025 Movatter.jp