Closed rodriguezsamuel closed 6 years ago
Hay varias cosas que vale la pena comentar; en lo que sigue, trataré de explicar lo que debes hacer.
Primero que nada, este PR lo iniciaste desde la rama master, cosa que no está bien. Esto mismo lo hiciste en #16, por lo que es importante enfatizar que eso no hay que hacerlo.
Entonces, para hacer las cosas de manera separada sugiero lo siguiente:
Desde master, crea una rama para la Tarea0 y otra (distinta!) para la Tarea1; utiliza (desde master!) el comando git checkout -b <rama>
para hacerlo.
Cámbiate a master y borra todos los commits que creaste en este PR (y que no están en master del repositorio del curso). Checa lo que hiciste en "learn git-branching" para hacerlo.
Usaré ahora como ejemplo la rama de la Tarea0 (lo mismo lo harás para la rama de la Tarea1). Cámbiate a esa rama ("rama_tarea0"); la idea es que borres todos los commits que tienen que ver con la Tarea1, que son los últimos 4. Una vez hecho esto, subes los cambios a GitHub, y haces el PR de la Taarea0.
Cámbiate ahora a la rama relacionada con la Tarea1. Debes ahora borrar todos los commits relacionados con la Tarea0. Desde ahí subes los cambios a GitHub y haces un nuevo PR de la Tarea1.
Dime si si tienes algún problema con esto.
Aprovechando el espacio, aquí te mando un par de comentarios sobre las tareas.
En cuanto a la Tarea0, la imagen debe estar dentro del archivo Tarea0.ipynb, que debe encontrarse en el directorio "tareas/samuel" (si quieres, puedes cambiar lo de "samuel" por cualquier otro nombre que lo identifique como tuyo"). Checa el repositorio del curso para que quede claro.
En cuanto a la Tarea1, creo que está bastante bien, aunque vale la pena implementar varias cosas:
Sigo sin estar seguro de si estoy usando bien git. Pero estuvo muy divertido lo de las gráficas del ejercicio 3; no sé si me hace falta algo.