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

GIT desde Rstudio #10

Open
marianpena opened this issue Jun 3, 2024 · 22 comments
Open

GIT desde Rstudio #10

marianpena opened this issue Jun 3, 2024 · 22 comments

Comments

@marianpena
Copy link
Member

marianpena commented Jun 3, 2024

Explicar cómo usar git spara los usuarios de R. Se puede usar parte de lo de Jorge: https://github.com/cornejotux/ClaseRMarkdownYShiny/blob/master/materials/RMarkdown/06_control_de_version-git.Rmd

Igual es mejor que lo exporte a pdf para que incluya las imágenes?

@cornejotux
Copy link
Collaborator

@marianpena veo que ya hay una seccion de git, la idea sería incluir en una carpeta mi material de intro a GitHub desde RStudio?
Si es eso, lo puedo trabajar a la brevedad.

@cornejotux
Copy link
Collaborator

ok.. Estoy tratando de subir un archivo MD a la carpta de RStudio, la idea es ahi dejar un documento sobre el uso de Github desde R.
Esta el material introductorio de Github que tal vez deberiamos eliminar, que opinas @marianpena?

Pero no me dejar subir el archivo:

/usr/bin/git push origin HEAD:refs/heads/main
remote: Permission to Intercoonecta/Aula-invertida.git denied to cornejotux.
fatal: unable to access 'https://github.com/Intercoonecta/Aula-invertida.git/': The requested URL returned error: 403

@marianpena
Copy link
Member Author

Hola Jorge, Muchas gracias por los avances. Tienes que mandárselo a @emiliom que es el que coordina los cambios del aula invertida. A mí me parecio mejor en principio añadir un pequeño apartado especídifco para R pero que lo vea Emilio como le parece mejor.

@emiliom
Copy link
Member

emiliom commented Jun 11, 2024

Hola. Veo que no estoy recibiendo notificaciones de este repositorio. No había visto el issue de Jorge hasta que Marian me etiquetó. Voy a cambiar mis settings para recibir notificaciones, y luego le doy seguimiento a la pregunta de Jorge.

@emiliom
Copy link
Member

emiliom commented Jun 12, 2024

@cornejotux estoy de acuerdo con @marianpena que lo mejor sería adaptar solamente tu material específico para git desde RStudio. Sugiero tratar de integrarlo un poco ya sea con el tutorial de Git/github o el de R, y mencionarlo y enlazarlo en el otro.

Veo que el formato de tu archivo RMarkdown/06_control_de_version-git.Rmd es RMarkdown, y no Markdown en sí. Entonces, Github no lo puedo convertir directamente a HTML, como hacemos con las demás páginas en https://github.com/Intercoonecta/Aula-invertida/ -- por ejemplo, https://github.com/Intercoonecta/Aula-invertida/blob/main/Intro-a-github/07-github-remote.md. ¿Es posible convertir Rmd a markdown regular? Creo que eso sería lo mejor (comparado a convertirlo a pdf), ya que así seguiría el mismo formato general de los demás tutoriales.

En cuanto al acceso al repositorio, te puedo añadir para que podás hacer commits directamente. Pero qué tal si comenzamos con un pull request? Así podría hacer comentarios antes de integrarlo.

También, si tenés más tiempo, sería fantástico si pudieras revisar el tutorial de Git/github que tenemos actualmente, que está basado en un tutorial de software carpentry. Si tenés sugerencias de correciones, actualizaciones o mejorías, ¡bienvenidas!

Gracias!

@cornejotux
Copy link
Collaborator

@emiliom ya transformé el documento a .md, asi que no debera tener problemas.

@emiliom
Copy link
Member

emiliom commented Jun 12, 2024

Fantástico! En cuanto a si ponerlo en el tutorial de git/github o el de R, creo que cualquiera de los dos está bien. Si te da igual, ponerlo en el de git/github me parece un poquito mejor.

@cornejotux
Copy link
Collaborator

Si, voy a ponerlo como una seccion dentro de RStudio, ahi podemos ver el material y pensarlo en detalle.

Aun tengo problemas para hacer el push @emiliom

/usr/bin/git push origin HEAD:refs/heads/main
remote: Permission to Intercoonecta/Aula-invertida.git denied to cornejotux.
fatal: unable to access 'https://github.com/Intercoonecta/Aula-invertida.git/': The requested URL returned error: 403

@emiliom
Copy link
Member

emiliom commented Jun 12, 2024

Te acabo de añadir al repo con el rol de "Write". Pero igual, prefiero que hagás un pull request, preferiblemente desde tu propio fork, pero si preferís hacerlo desde otro branch en este repo, eso también funciona.

@cornejotux
Copy link
Collaborator

cornejotux commented Jun 12, 2024

Ya @emiliom , hice un brach llamado "jorge" donde mande el archivo "09_control_de_version-git.md",
y mande todas las imagenes que uso en ese archivo, los puse todos juntos en la carpeta "GIT4RStudio" para que sea facil identificarlos.

@emiliom
Copy link
Member

emiliom commented Jun 12, 2024

Gracias. Le di un vistazo, pero lo voy a revisar más detenidamente mañana.

https://github.com/Intercoonecta/Aula-invertida/blob/Jorge/Intro-a-R/09_control_de_version-git.md

@cornejotux
Copy link
Collaborator

Hay bastante material repetido con la seccion de GIT, pero puede ser util volver a verlo rapidamente.
Pretento cambiar el paso a paso para crear un repositorio con nombre relacionados a la hackaton ya que el que esta fue para otro tema que tenia para enseñar.

@emiliom
Copy link
Member

emiliom commented Jun 18, 2024

Hola @cornejotux y @marianpena. Disculpas por la tardanza con mi seguimiento.

@cornejotux: en general, creo que es mejor minimizar la presencia de materiales de tutoriales muy similares en diferentes tutoriales. Esa duplicación puede crear confusión (como nos pasó con las instrucciones de instalaciones de Python la vez anterior), y también diluye el esfuerzo común por crear buenos tutoriales sobre temas específicos.

Veo algunas alternativas:

  1. Seguir con la duplicación. Usar la lección nueva de Jorge tal y como está, y nada más añadir enlaces de esa página al tutorial de git-github, y vice versa.
  2. En la lección de Jorge, retener solamente el contenido que comienza en la sección Trabajando localmente con Git en RStudio. Añadir un enlace de esa lección al tutorial de git-github, y vice versa.
  3. En la lección de Jorge, retener solamente el contenido que comienza en la sección Trabajando localmente con Git en RStudio, y mover el contenido genérico sobre git y github al tutorial de git y github, como una lección (otra página) adicional. Ahí clarificaríamos que esa lección no está bien integrada con las demás de ese tutorial. Ese contenido de Jorge tiene más detalles específicos y prácticos sobre GitHub, incluyendo capturas de pantalla, comparado a la lección existente, Repositorios remotos en GitHub.

Mi preferencia es la 3, seguida por la 2. @cornejotux, con gusto te ayudo en lo que pueda, para hacer esos cambios. Lo que decís sobre volver a ver el tema de git y github durante el tutorial sincrónico de R me parece muy bien. Mi enfoque aquí es sobre los materiales y la división de contenido, independientemente del tutorial sincrónico.

¿Qué piensan?

En cuento a esto:

Pretento cambiar el paso a paso para crear un repositorio con nombre relacionados a la hackaton ya que el que esta fue para otro tema que tenia para enseñar.

Es posible que tengamos ideas diferentes. Yo asumo que nuestra meta común es que los tutoriales residan en este repositorio y aparezcan aquí de una manera consistente. Pensabas poner el tuyo en otro repositorio?

@marianpena
Copy link
Member Author

No termino de entender exactamente la opción 3 pero creo que me decanto por la 2. Creo que para el aula invertida es mejor algo simple introductorio: incluir el apartado Trabajando localmente con Git en RStudio. y luego referenciar al resto. 'si quieres más informacion'. Recordad que tenemos tutoriales intermedios donde creo que será mejor desarrollar estos temas

@cornejotux
Copy link
Collaborator

cornejotux commented Jun 20, 2024 via email

@emiliom
Copy link
Member

emiliom commented Jun 20, 2024

Gracias. Acabo de crear un PR (#13) que implementa la opción 2. Por favor revísenla, y si les parece hagan el merge directamente. O editen el PR si les parece necesario.

En cuanto a la opción 3, lo que quise decir es que en vez de descartar la parte del tutorial original de Jorge, podríamos añadirlo como una lección adicional en el tutorial de git y github que ya existe. A estas alturas creo que no vale la pena tratar de hacer eso esta semana, y por eso me tomé la libertad de crear el PR #13

@cornejotux
Copy link
Collaborator

@emiliom acabo de actulizar la seccion de Github desde RStudio para tener las imagenes con ejemplos asociados al aula invertida y con el nuevo formato ed Github.
Lo unico que quedaria pendiente seria ver si necesitamos recortar este documento.
cc: @marianpena

@emiliom
Copy link
Member

emiliom commented Jun 21, 2024

Gracias. Pero veo que hiciste los cambios en tu branch, no en el PR. Reintegrarlos con lo que ya tenía en el PR tomaría más trabajo.

Hagamos lo siguiente. Ya que aprobaste el PR (gracias!), acabo de hacer el merge. Entonces, podrías hacer tus modificaciones directamente sobre la branch main, en su nuevo estado actual? Y de ahí crear un PR que yo puedo revisar. Según @marianpena parece que tendremos unos días más para entregar esto.

Lo unico que quedaria pendiente seria ver si necesitamos recortar este documento.

La página https://github.com/Intercoonecta/Aula-invertida/blob/main/Intro-a-R/09_control_de_version-git.md que acabo de integrar de mi PR retiene solamente el contenido que comienza en la sección "Trabajando localmente con Git y GitHub en RStudio", hasta el final de tu documento original. Queda en tus manos si te parece mejor recortarlo aún más.

@cornejotux
Copy link
Collaborator

ok.. Parece que ahi quedarons las imagenes actualizadas.

@cornejotux
Copy link
Collaborator

@emiliom como se para el material actualizado que esta en el Branch "jorge" al main?
Elimine las secciones que no eran desde RStudio, asi que ahora deberia estar ok.

Me puedes ayudar con eso?

@emiliom
Copy link
Member

emiliom commented Jul 25, 2024

El material en tu branch jorge es una página completamente distinta y separada de las páginas en main. Yo lo que hice cuando integré tu material a la página 09_control_de_version-git.md, y lo que recomiendo, es modificar 09_control_de_version-git.md "manualmente", con cut & paste. Creo que es más cómodo hacerlo en tu local clone y después hacer un push o pull request.

@cornejotux
Copy link
Collaborator

cornejotux commented Jul 25, 2024 via email

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

No branches or pull requests

3 participants