Traducción library/logging.config.po by lis17 · Pull Request #1645 · python/python-docs-es

lis17 · 2021-11-19T11:31:05Z

Closes issue #1420

lis17 · 2021-11-19T12:04:47Z

🤔

aroquega

Hola ✋🏽 . Lo del error de postell probablemente sea el param que comenté pero esa entrada y las relacionadas me parecen muy extrañas 🤔 .

He dejado algunas sugerencias y observaciones para que los revisas y me dices qué opinas 😄 . Todo lo que está en backticks (``) por lo general no deben traducirse porque hacen referencia a nombres de variables u otros nombres propios y he visto que ese patrón también estaba en la anterior traducción, pero sería bueno que lo también arregles eso.

He visto que has cambiado "manejador" por "gestor", a mi no me suena mucho esa palabra perp lo mejor es la consistencia dentro del archivo, y al menos hay 5 palabras que siguen con la traducción antigua de "manejador*".

Aparte de eso, los cambios y añadidos que has puesto me parecen bien 👍🏽.

aroquega · 2021-11-20T16:44:41Z

-"`ImportError` con un mensaje descriptivo adecuado. La siguiente es una lista "
-"(posiblemente incompleta) de condiciones que lanzará un error:"
+"`ImportError` con un mensaje descriptivo adecuado.  La siguiente es una "
+"lista (posiblemente incompleta) de condiciones que lanzará un error:"


creo que debería ser "de condiciones que lanzarán un error", de lo que leo, comprendo que las condiciones lanzan los errores si se cumplen, no la lista en sí. ¿Qué opinas?

Hola! Gracias por todas tus correcciones.. En este caso tiene sentido lo que dices.

aroquega · 2021-11-20T16:59:18Z

 "configuration.  :param encoding: The encoding used to open file when *fname* "
 "is filename."
 msgstr ""
+"Si se especifican como ``False``, los "


Hay algo muy raro con esta entrada @cmaureir . Esta es la que se toma para hacer la traducción, pero tiene un espaciado singular y su contenido se repite en las siguientes entradas.

si, extraño, pero igual es como está originalmente https://docs.python.org/3/library/logging.config.html?highlight=logging%20config#logging.config.fileConfig

Así es el texto en el rst:

:param disable_existing_loggers: If specified as ``False``, loggers which exist when this call is made are left enabled. The default is ``True`` because this enables old behaviour in a backward-compatible way. This behaviour is to disable any existing non-root loggers unless they or their ancestors are explicitly named in the logging configuration.

Así que los espacios que ves son los del comienzo del texto, con lo que mejor dejarlo igual.

aroquega · 2021-11-20T17:09:55Z

+"comportamientos antiguos                                  de una manera "
+"compatible con versiones anteriores. Este comportamiento consiste "
+"en                                 deshabilitar cualquier registrador no "
+"arraigado a menos que                                  ellos o sus "


¿podría ser "cualquier registrador que no sea raíz"? registrador arraigado al menos para mí no me suena al logger top de la jerarquía.

aroquega · 2021-11-20T17:18:20Z

-"puerto diferente). Para evitar el riesgo de que esto suceda, use el "
-"argumento ``verificar`` para :func:`listen` para evitar que se apliquen "
+"puerto diferente. Para evitar el riesgo de que esto suceda, use el argumento "
+"``verificar`` para :func:`listen` para evitar que se apliquen "


creo que verificar no debería ser traducido porque habla de un parámetro de la función listen. También se está perdiendo la palabra "escuchar", creo que una opción podría ser reformular esa parte de la oración como "use el argumento verify en la función :func:listen para escuchar y evitar ..."

Sí, esta duda la tuve al momento de traducir, pero como vi que ya se habían traducido varios términos así, seguí el patrón que ya había. Voy a intentar corregirlo en todo el texto, espero que no se me pase alguna.

Recomendarías que todas las palabras con comillas simples o dobles no sean traducidas? o depende de cada caso? es que me estoy encontrando con otras palabras en la misma situación y ya estoy dudando.

creo que las palabras con comillas (") no son necesario, pero con las comillas para atrás (``) por lo general sí, aunque habría que ver el documento traducido, porque ahí se pueden ver los snippets de código o la signatura de la función.

Cambio hecho

aroquega · 2021-11-20T17:22:59Z

 "condensed format.  If your formatter requires different or extra "
 "configuration keys, you should use :ref:`logging-config-dict-userdef`."
 msgstr ""
+"Una clave opcional ``class`` indica el nombre de los formateadores de clase "


creo que debería ser "indica el nombre de la clase del formateador"

aroquega · 2021-11-20T17:46:44Z

+"gestores, que debe aparecer en la sección ``[gestores]``. Estos nombres "
+"deben aparecer en la sección ``[gestores]`` y tener las secciones "


lo mismo que el comentario anterior, [handlers] no debería traducirse. También la entrada anterior habla de una entrada nivel```, eso se quedó de la anterior traducción pero sería bueno si lo pusieras como level`` (sin traducción), por que hace referencia a un nombre del equema.

Cambio hecho!

aroquega · 2021-11-20T17:49:02Z

-"nivel '' se interpreta como para los registradores, y ``NOTSET`` se entiende "
-"como 'registrar todo'."
+"La entrada ``class`` indica la clase del gestor (según lo determinado por :"
+"func:`eval` en el espacio de nombres del paquete ``logging``). El `` nivel "


nivel no debería traducirse, y existe le faltan backticks (``) al final. Eso hace que no se renderize bien en la documentación.

aroquega · 2021-11-20T17:49:25Z

@@ -1267,7 +1282,7 @@ msgid ""
 "and have a corresponding section in the configuration file."
 msgstr ""
 "La entrada ``formateador`` indica el nombre clave del formateador para este "


formateador no debería ser traducido.

aroquega · 2021-11-20T17:49:51Z

-"manejador. Si está en blanco, se utiliza un formateador predeterminado "
+"gestor. Si está en blanco, se utiliza un formateador predeterminado "
 "(``logging._defaultFormatter``). Si se especifica un nombre, debe aparecer "
 "en la sección ``[formateadores]`` y tener una sección correspondiente en el "


formatters no debería ser traducido.

aroquega · 2021-11-20T17:51:13Z

 #: ../Doc/library/logging.config.rst
 msgid "param encoding"
-msgstr ""
+msgstr "param encoding"


posiblemente esta entrada sea la que cause error pero es muy raro porque está repetida, esta entrada no debería estar.

lis17 · 2021-11-22T12:42:52Z

La traducción de handler como gestor la encontré en la memoria de traducción. Si no es la más adecuada deshago el cambio y la dejo como manejador. Ya me comentas algo.

aroquega · 2021-11-22T12:59:29Z

@lis17 Hola ✋🏽 , creo que sí es adecuado usar la palabra "gestor" pero aún hay palabras que siguen con la vieja traducción de "manejador". Creo que debes cambiar esas palabras restantes por "gestor".

aroquega

Hola, tengo tres observaciones, la traducción sigue bien 👍🏽 😄 , con esas observación y un powrap creo que estaría. Aunque igual habría que ver si pospell lanza un error por lo de "param" 🤔 .

aroquega · 2021-11-24T14:00:30Z

 #: ../Doc/library/logging.config.rst:2
 msgid ":mod:`logging.config` --- Logging configuration"
-msgstr ":mod:`logging.config` --- Configuración de registro"
+msgstr ":mod:'logging.config' --- Configuración de registro"


falta cambiar estas comillas (') por backticks (``) porque se está renderizando mal

aroquega · 2021-11-24T14:06:34Z

-"contiene espacios o caracteres no alfanuméricos.  Si un valor de índice "
-"consta solo de dígitos decimales, se intentará acceder utilizando el valor "
-"entero correspondiente, volviendo al valor de cadena si es necesario."
+"diccionario con clave ``handlers``, la cadena``'cfg://handlers."


debe haber un espacio entre una letra y las backtics (``), si no, va a haber un problema en la renderización.

aroquega · 2021-11-24T14:07:31Z

+"diccionario con clave ``handlers``, la cadena``'cfg://handlers."
+"email``resolvería al diccionario con clave ``email`` en el diccionario "
+"``handlers``, y así sucesivamente.  La cadena ``'cfg: //handlers.email."
+"toaddrs [1]``resolvería a ``'dev_team.domain.tld'`` y la cadena ``'cfg://"


un espacio entre los backticks y "resolvería".

Gracias! cómo puedo saber que lo que falla son estos pequeños detalles? es por el tipo de error que lanza el CI?

hola, lo del powrap si es por el error que se lanza, y lo de los espacios si se nota cuando revisas la documentación generado.

aroquega · 2021-11-24T15:41:38Z

Hola también me acuerdo que antes aparecía otro error aparte del powrap, lo de la palabra param. Hay dos entradas que tienen la palabra "param", no vi bien cuál entrada causaba el error, pero cuando se solucione lo de powrap quizás vuelva a aparecer.

aroquega · 2021-11-25T03:01:05Z

hola, debes pasarle powrap con powrap library/logging.config.po

lis17 · 2021-11-25T11:04:03Z

hola, debes pasarle powrap con powrap library/logging.config.po

Hola. Lo he hecho en cada uno de los commits, este es del último:

Lo que no entiendo es el error que arroja cuando hago git add, que lo puedes ver en la captura de pantalla.

aroquega · 2021-11-26T04:30:22Z

Hola, según mi entendimiento, no es un error como tal o no debería causar problemas 🤔 . Es porque los saltos de línea en Windows (CRLF) son diferentes a los Linux (LF), en tu máquena se van a quedar los archivos con el salto de línea de windows (CRLF), pero cuando se suban, se van a subir con los saltos de línea de Linux.

No creo que ese sea el error aunque tampoco estoy muy seguro. Quizás sea un error del powrap.

aroquega · 2021-11-26T22:21:43Z

Hola, y después de hacer el powrap que te sale al hacer powrap --check --quiet library/logging.config.po? y ¿estás usando un entorno virtual? no me acuerdo muy bien pero creo que había problemas con powrap en windows, no me acuerdo bien. Pero creo que otra opción sería abrirlo con el poedit y guardarlo, para que el poedit haga el wrapping 😅.

lis17 · 2021-11-29T14:28:42Z

Hola, y después de hacer el powrap que te sale al hacer powrap --check --quiet library/logging.config.po? y ¿estás usando un entorno virtual? no me acuerdo muy bien pero creo que había problemas con powrap en windows, no me acuerdo bien. Pero creo que otra opción sería abrirlo con el poedit y guardarlo, para que el poedit haga el wrapping 😅.

Hola. No estoy usando entorno virtual, en otras ocasiones me ha dado problemas aunque puedo intentarlo de nuevo si es necesario. Ya me había comentado @cmaureir de como hacer el powrap con poedit, he ajustado los finales de renglón a 79, antes lo tenía a 75. Cualquier cosa que tenga que intentar estoy lista!

Acabo de activar el entorno virtual, hice powrap de nuevo:

cmaureir · 2021-11-29T19:49:41Z

ojo que solo debes hacer powrap library/logging.config.po
la opción --check --quiet no hace nada, solo te dice "sí, necesita powrap" o "no, no necesita powrap"

aroquega · 2021-11-29T20:05:47Z

hola @cmaureir quería saber si en local le decía "sí, necesita powrap" o "no", pero el powrap no hace rewrap 😭. @lis17 creo que tienes que hacerlo con poedit como te comentaron a ver si con eso se soluciona.

cmaureir · 2021-12-02T08:00:12Z

Son los problemas de siempre y windows lamentablemente :(

cmaureir · 2021-12-02T08:03:26Z

Hola @lis17 hice un powrap de tu cambio.
Si quisieras hacer otro cambio en ese Pull Request, recuerda primero hacer un git pull origin traduccíon-library-logging.config para que puedas tener en tu ordenador, el cambio que acabo de subir.

…cíon-library-logging.config' of https://github.com/lis17/python-docs-es into traduccíon-library-logging.config

lis17 · 2021-12-03T12:38:36Z

@aroquega , @cmaureir ahora el error es de pospell de nuevo. Ya me decís si tengo que intentar algo más.

cmaureir · 2021-12-06T22:00:05Z

@aroquega , @cmaureir ahora el error es de pospell de nuevo. Ya me decís si tengo que intentar algo más.

solo tienes que agregar la palabra "param" al diccionario del archivo. Si no existe, lo creas dentro de "dictionaries/" y le llamas library_logging_config.txt

lis17 · 2021-12-07T16:35:47Z

Al fin!!! objetivo conseguido!!!!!!

aroquega · 2021-12-07T21:16:27Z

@lis17 por qué lo cerraste? qué pasó?

cmaureir · 2021-12-07T21:51:30Z

@lis17 tienes que esperar que se haga el merge jaja y así se cierra solo el PR

cmaureir · 2021-12-07T22:13:11Z

Ahora si, gracias @lis17 🎉

* Traducción library/logging.config.po * traducción library/logging.config.po corrección pospell * corrección de comentarios PR library/logging.config * corrigiendo renderización * powrap library/logging.config.po * powrap con poedit * se agrega param al diccionario Co-authored-by: Cristián Maureira-Fredes <Cristian.Maureira-Fredes@qt.io>

lis17 · 2021-12-09T10:52:28Z

@lis17 por qué lo cerraste? qué pasó?

Sorry! Lo hice sin saber, como pasó todas las revisiones pensé que era lo correcto. Errores de novata ☺️

lis17 · 2021-12-09T10:54:15Z

@lis17 tienes que esperar que se haga el merge jaja y así se cierra solo el PR

Ya lo sé para la próxima 😆😆, sigo aprendiendo en cada paso. Gracias @aroquega @cmaureir por la ayuda!!

lis17 · 2021-12-09T10:54:55Z

Ahora si, gracias @lis17 🎉

👏🏼👏🏼👏🏼

lis17 added 2 commits November 19, 2021 12:26

Traducción library/logging.config.po

46d1370

traducción library/logging.config.po corrección pospell

c7ea5ec

aroquega suggested changes Nov 20, 2021

View reviewed changes

corrección de comentarios PR library/logging.config

6b90425

aroquega suggested changes Nov 24, 2021

View reviewed changes

corrigiendo renderización

6e1df35

powrap library/logging.config.po

f3f6888

lis17 added 3 commits December 3, 2021 13:06

powrap con poedit

873da46

Mgit pull origin traduccíon-library-logging.configerge branch 'traduc…

74857de

…cíon-library-logging.config' of https://github.com/lis17/python-docs-es into traduccíon-library-logging.config

powrap con poedit

34952bd

se agrega param al diccionario

362dbe9

lis17 closed this Dec 7, 2021

cmaureir reopened this Dec 7, 2021

aroquega approved these changes Dec 7, 2021

View reviewed changes

cmaureir approved these changes Dec 7, 2021

View reviewed changes

cmaureir merged commit 3cb437f into python:3.10 Dec 7, 2021

		"gestores, que debe aparecer en la sección ``[gestores]``. Estos nombres "
		"deben aparecer en la sección ``[gestores]`` y tener las secciones "

Uh oh!

Conversation

lis17 commented Nov 19, 2021

Uh oh!

lis17 commented Nov 19, 2021

Uh oh!

aroquega left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

aroquega Nov 20, 2021 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

lis17 commented Nov 22, 2021

Uh oh!

aroquega commented Nov 22, 2021

Uh oh!

aroquega left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

aroquega commented Nov 24, 2021

Uh oh!

aroquega commented Nov 25, 2021

Uh oh!

lis17 commented Nov 25, 2021

Uh oh!

aroquega Nov 20, 2021 •

edited

Loading

lis17 commented Nov 29, 2021 •

edited

Loading