programminghistorian / ph-submissions

The repository and website hosting the peer review process for new Programming Historian lessons
http://programminghistorian.github.io/ph-submissions
138 stars 114 forks source link

Ticket de revisión: "Introducción a la codificación de textos en TEI - Parte 2" #386

Closed jenniferisasi closed 2 years ago

jenniferisasi commented 3 years ago

Programming Historian en español ha recibido la siguiente propuesta de lección: "Introducción a la codificación de textos en TEI - Parte 2" por parte de Nicolás Vaughan (@nivaca). La editora de esta lección será @jenniferisasi. Esta lección es la segunda parte de una secuencia de dos, y se encuentra en estos momentos en fase de revisión y puede leerse en:

Vista previa http://programminghistorian.github.io/ph-submissions/es/lecciones/introduccion-a-tei-2

Archivo https://github.com/programminghistorian/ph-submissions/blob/gh-pages/es/lecciones/introduccion-a-tei-2.md

Imágenes https://github.com/programminghistorian/ph-submissions/tree/gh-pages/images/introduccion-a-tei

Por favor, utiliza los números de línea proporcionados en la vista previa, si eso ayuda a señalar mejor tus comentarios. Pero puedes estructurar tu revisión como mejor te parezca.

Otros miembros de nuestra comunidad también están invitados a ofrecer sus comentarios de una manera constructiva; los comentarios deberán publicarse en este hilo de conversación, por lo que se recomienda haber leído nuestra guía para revisores y tener en cuenta nuestra política contra el acoso (ver más abajo). No se aceptarán más comentarios por parte de la comunidad tras la publicación de la segunda revisión formal a fin de que el traductor pueda empezar a trabajar en los cambios solicitados. Cuando esto ocurra, publicaré un anuncio aquí.

Asimismo, me comprometo a mantener la conversación abierta a todo el mundo en GitHub. Pero si alguno de los participantes quiere ponerse en contacto en privado conmigo, puede escribirme un correo electrónico. También es posible contactar con nuestra ombudperson.

Política contra el acoso

Esta es una declaración de los principios de Programming Historian en español y establece las expectativas para el tono y el estilo de toda la correspondencia entre los revisores, autores, editores y colaboradores de nuestros foros públicos.

El objetivo de The Programming Historian en español es ofrecer un entorno abierto en el que la comunidad de participantes sean libres para analizar ideas, realizar preguntas, sugerir cambios, y pedir aclaraciones; también queremos que sea un espacio libre de acoso y hostigamento para todo el mundo con independencia de su género, identidad, orientación sexual, diversidad funcional, apariencia física, tamaño corporal, raza, edad, religión o conocimientos informáticos. No se tolerará ningún tipo de acoso o ataque ad hominem. Los participantes que violen esta regla podrán ser expulsados del proceso editorial a discreción del equipo editorial. Si presencias o sientes que has sido víctima de algún tipo de acoso, por favor, contacta con nuestros ombudsperson Silvia Gutiérrez De la Torre.

--

Con este ticket se cierra el preliminar #351 y en el que se decidió partir en dos una lección completa demasiado larga.

jenniferisasi commented 3 years ago

@nivaca quedó abierto así el Issue para la revisión de la segunda parte de tu lección. En los próximos días dejaré yo los primeros comentarios como editora (teniendo en cuenta los de la primera parte para no repetir) y, a la vez, empiezo a buscar revisores. ¡Gracias por tu paciencia!

nivaca commented 3 years ago

@jenniferisasi ¡Excelente! Quedo pendiente. Gracias.

rivaquiroga commented 3 years ago

@jenniferisasi, @nivaca, como las imágenes y archivos de ambas lecciones estaban todos juntos en una misma carpeta, tuve que separarlas para la publicación de la primera parte. Lo correspondiente a esta parte ahora está en intro-a-tei-2. @nivaca, mientras Jennifer hace la primera revisión, te pediría si puedes modificar los nombres de las imágenes teniendo en cuenta lo que aparece acá: https://programminghistorian.org/es/guia-para-autores#figuras-e-im%C3%A1genes. Como el nombre de esta lección termina con número, agreguémosle un guion para que no sea confuso (por ejemplo intro-a-tei-2-01.png).

nivaca commented 3 years ago

@rivaquiroga Ahora mismo lo hago.

nivaca commented 3 years ago

@rivaquiroga En el directorio images (y también en assets) encuentro dos subdirectorios: introduccion-a-tei e introduccion-a-tei-1. ¿Quieres que cambie los nombres de estos subdirectorios (introduccion-a-tei-1 y introduccion-a-tei-2)?

rivaquiroga commented 3 years ago

@nivaca, sí, eso mismo.

nivaca commented 3 years ago

@rivaquiroga Listo, ya cambié los nombres de imágenes según lo requerido.

rivaquiroga commented 3 years ago

¡Muchas gracias!

jenniferisasi commented 3 years ago

Gracias @rivaquiroga! no me había fijado yo en eso :/

jenniferisasi commented 3 years ago

@nivaca paso por aquí para dejar constancia de que estoy trabajando en la lectura de la lección para darte las primeras pautas editoriales. Voy despacio por los compromisos laborales a la vuelta de vacaciones, pero ahí voy. Espero tenerla pronto.

jenniferisasi commented 3 years ago

Llegué. Se me ha hecho larga de revisar puesto que, como ves, apunto numerosos cambios de estilo, sobre todo, que mejorarán la fluidez del texto para los lectores. Aquí van:

Comentario general:

Detalles de estilo y añadiduras que propongo:

Como siempre, quedo pendiente a cualquier duda por si no he puesto algo claramente o si no estás de acuerdo con alguno de los cambios sugeridos.

nivaca commented 3 years ago

Gracias por tus comentarios, @jenniferisasi. Mañana mismo empiezo la corrección.

nivaca commented 3 years ago

Hola @jenniferisasi. Gracias de nuevo por la revisión. Hiciste una lectura muy juiciosa. Ya corregí todo lo que señalaste.

jenniferisasi commented 3 years ago

Perfecto @nivaca. Gracias por introducir los cambios pertinentes. Me pongo entonces a iniciar el proceso de revisión por pares. Como siempre, cuando tenga alguna novedad (espero pronto) lo haré por aquí.

jenniferisasi commented 3 years ago

@dxvidmr (pendiente de recibir su usuario de GitHub para que vea este mensaje) ha aceptado actuar de primer revisor para esta segunda parte de introducción a TEI. Siempre con la posibilidad de dar aviso de cualquier situación que lo impida, se compromete a entregar la revisión antes o para el 30 de septiembre.

David, como creo que es tu primera revisión para nosotros, te invito a que eches un vistazo a nuestra Guía para revisores antes de ponerte manos a la obra. Yo quedo a disposición de responder cualquier duda que surja durante el proceso.

dxvidmr commented 3 years ago

Buenas, me paso solamente a saludar. Gracias nuevamente por invitarme a la revisión, @jenniferisasi. Me encantó la primera parte de la lección, así que será un placer revisar esta segunda entrega, @nivaca. Os contactaré si me surgen dudas.

nivaca commented 3 years ago

Gracias, @dxvidmr. Quedo pendiente de tus comentearios.

rivaquiroga commented 3 years ago

@dxvidmr, qué bueno verte por acá :rocket:

dxvidmr commented 3 years ago

¡Buenas! Ando al fin revisando la lección. Estaba releyendo la parte 1 para hacerme una idea más completa de todo y me encontré con un error en el md. Os lo digo ahora para que no se me olvide y lo podáis revisar @jenniferisasi @rivaquiroga. Es una tontería:

En la sección "Validación sintáctica de documentos XML" (en la línea 116, si no me equivoco) hay dos enlaces en los que la sintaxis está al revés (paréntesis y corchetes) y por tanto no se visualizan bien.

Dice (https://codebeautify.org/xmlvalidator)[https://codebeautify.org/xmlvalidator] o (https://www.xmlvalidation.com)[https://www.xmlvalidation.com] y debiera ser [https://codebeautify.org/xmlvalidator](https://codebeautify.org/xmlvalidator) o [https://www.xmlvalidation.com](https://www.xmlvalidation.com).

Sigo, llegaré antes del 30 :)

rivaquiroga commented 3 years ago

¡Hola, @dxvidmr! Muchas gracias por avisarnos. Haremos la corrección para que los enlaces se vean bien.

dxvidmr commented 3 years ago

Terminé. Lo primero, felicitarte @nivaca por esta segunda entrega de la lección sobre TEI. Fue un placer poder revisarla.

Primero te dejaré unos comentarios e impresiones generales, y después comentarios específicos indicando la línea (siguiendo la previsualización). Espero no haberme extendido mucho con la revisión y, sobre todo, no sobrepasar mis funciones aquí. Intento explicar bien a lo que me refiero y sobre todo dar alternativas e ideas cuando veo algo que puede mejorarse, por eso parece una lista larga ;)

Comentarios generales

Comentarios específicos

(Utilizaré el formateado de código cuando te haga una propuesta de redacción, para que resalte y no se confunda.)

Y hasta aquí mi revisión. De nuevo felicitaciones y agradecer a PH por contactarme. Quedo pendiente por si alguno de mis comentarios suscita duda o hay feedback de retorno.

Un abrazo.

nivaca commented 3 years ago

@dxvidmr Muchas gracias por tu cuidadosa lectura y tus comentarios. Voy a ponerme a hacer las correcciones. Un abrazo también.

jenniferisasi commented 3 years ago

Muchas gracias @dxvidmr por tan atenta revisión. Todos los comentarios me parecen acertados y es recomendable añadirlos. Veo que @nivaca ya se puso a trabajar, pero quería reafirmar que es necesario realizar los cambios.

nivaca commented 3 years ago

@dxvidmr Mil gracias por tus comentarios. Viste muchas cosas que yo había pasado por alto. Corregí casi todo lo que me sugeriste. Lo que preferí evitar fue la recomendación de explicar los comentarios en XML (y TEI). En una versión anterior de la Parte 1 hablé de eso, pero decidimos quitarlo por ser tema propio de XML. Quedo pendiente de lo que se ofrezca.

dxvidmr commented 3 years ago

Entendido lo de los comentarios, @nivaca! Creo que ha quedado muy bien así con el "epílogo" tras las conclusiones. Enhorabuena.

jenniferisasi commented 3 years ago

¡Gracías @nivaca por realizar los cambios tan rápidamente! Creo que tengo a una segunda revisora preparada para la última fase. Doy inicio al proceso para incluirla por aquí. Rosa Muñoz @romamendo realizará la segunda ronda de revisión por pares.

nivaca commented 3 years ago

Con gusto, @jenniferisasi. Quedo pendiente.

romamendo commented 3 years ago

¡Hola! Muchas gracias @jenniferisasi por invitarme a participar en este proceso. Ya he podido ponerme con calma a revisar esta lección y me está gustando mucho. ¡Gracias! Voy enviando algunas sugerencias menores sobre lo que he leído hasta ahora. Por favor, si alguno de mis comentarios no se entiende bien, avísenme.

image

¿La etiqueta no debería ser <reg> ?

Hasta aquí he podido avanzar por ahora. Por favor, si me pueden dar un poquito de tiempo hasta mañana... ¡Gracias!

jenniferisasi commented 3 years ago

Gracias @romamendo por dar inicio a tu revisión. Por supuesto, tome el tiempo que necesite. Veo que está prestando atención a cuestiones de ortografía y puntuación que, aunque son importantes y lo agradecemos de veras, contamos con un servicio para corregirlo más adelante. Por favor, indique al final de su revisión si el contenido le parece adecuado o si ha notado que falta algo de información para entender la lección. De nuevo, ¡mil gracias!

romamendo commented 3 years ago

Acabo de terminar. Primero leí solo la segunda lección, pero después he revisado tanto la primera como la segunda para tener la secuencia completa. Para entender mi valoración es importante que consideren que mi formación en este tema es media y, a pesar de ello, he podido seguir la lección con facilidad. La explicación es fluida. Es cierto que hay partes más densas, pero creo que están suficientemente explicadas y las notas aclaratorias ayudan a comprender las diferentes posibilidades de codificación, como en el caso de corr y reg. No sé si corresponde a la revisión, pero solo resaltaría un detalle que comenté en mi primera intervención que quisiera mencionar de nuevo porque me confundió al hacer la lectura del texto codificado:

Sinceramente, gracias @jenniferisasi por esta posibilidad que me han brindado. Ha sido una oportunidad de aprendizaje.

nivaca commented 3 years ago

@romamendo Muchas gracias por tu cuidadosa lectura. Tienes toda la razón con respecto a los elementos y . Ya mismo los corrijo. Voy a tener en cuenta el resto de tus recomendaciones también.

romamendo commented 3 years ago

Al contrario, @nivaca ha sido muy grato leer ambas lecciones. Muchas gracias

nivaca commented 3 years ago

Muchas gracias, @romamendo, ya hice los cambios. (Solo no corregí los de la puntuación fuera del paréntesis.)

jenniferisasi commented 3 years ago

Muchísimas gracias @romamendo por tu atenta evaluación de la lección de Nicolás.

@nivaca apunto en mis tareas darle una última lectura a la lección en los próximos días para así proceder, en caso de estar todo bien ya, a enviar la lección a copyediting .

jenniferisasi commented 3 years ago

Bueno, pues creo que esta lección queda lista ya para hacer el copyediting, @rivaquiroga, que supongo hará @jamotilla? V

Va de entrada que veo que las notas no está funcionando en la preview. Lo miré y no le veo la razón.

Quedamos a la espera del penúltimo paso para publicar la lección

rivaquiroga commented 3 years ago

¡Hola! @jamotilla se encargará del copyediting 🚀

Respecto de las notas, sospecho que no se visualizan por la forma en que están escritas al final. Faltan los dos puntos después de la llamada. Es decir, esto:

[^1] Adoptaremos la convención...

habría que cambiarlo por

[^1]: Adoptaremos la convención...

nivaca commented 3 years ago

De acuerdo, @rivaquiroga, yo las arreglo mañana. Gracias.

jenniferisasi commented 3 years ago

@nivaca ya queda hecho el cambio, no te preocupes.

nivaca commented 3 years ago

Gracisa @jenniferisasi !

jamotilla commented 3 years ago

Hola a todos y todas, he realizado la primer revisión de la lección. @nivaca te pido por favor que revises los cambios sugeridos. Lo puedes consultar en el siguiente enlace: https://github.com/programminghistorian/ph-submissions/compare/Copyedit-introduccion-a-tei-2

Te felicito por la lección ¡Es buenísima!

nivaca commented 3 years ago

@jamotilla ¡Mil gracias por tu revisión! Estoy de acuerdo con todos tus comentarios.

rivaquiroga commented 3 years ago

¡Hola! Creo que solo faltaría el visto bueno final de @jenniferisasi para que podamos publicar esta lección :rocket: ¡Muchas gracias a todos!

jenniferisasi commented 3 years ago

@rivaquiroga estoy de acuerdo con los cambios. ¿Hago yo el pull request de la rama de José Antonio? Este proceso me ha pillado desprevenida.

rivaquiroga commented 3 years ago

Lo que te acomode más. Puedes hacerle merge directamente también si prefieres.

jenniferisasi commented 3 years ago

Dale, a ver si lo hago bien.

nivaca commented 3 years ago

@jenniferisasi y @rivaquiroga, mil gracias por todo el trabajo!!!

Pregunta técnica: cuando Riva dice "puedes hacerle merge" ¿se está refiriendo a hacerlo desde el repo de jekyll?

jenniferisasi commented 3 years ago

plas! Listo. Siguiendo con la idea de las "capas" se puede usar esta imagen o esta otra.

jenniferisasi commented 3 years ago

@jenniferisasi y @rivaquiroga, mil gracias por todo el trabajo!!!

Pregunta técnica: cuando Riva dice "puedes hacerle merge" ¿se está refiriendo a hacerlo desde el repo de jekyll?

No, el repo de ph-submissions donde @jamotilla había hecho su rama con las sugerencias de edición a tu tutorial. Ahora @rivaquiroga, como jefa, será quien mueva todo al repo ph-jekyll y nosotros podemos aprobar su PR cuando llegue el momento.

nivaca commented 3 years ago

No, el repo de ph-submissions donde @jamotilla había hecho su rama con las sugerencias de edición a tu tutorial. Ahora @rivaquiroga, como jefa, será quien mueva todo al repo ph-jekyll y nosotros podemos aprobar su PR cuando llegue el momento.

Ah, perfecto!

nivaca commented 3 years ago

plas! Listo. Siguiendo con la idea de las "capas" se puede usar esta imagen o esta otra.

La segunda es más bonita...? Lo que ustedes decidan está bien para mí.