Open santisoler opened 3 years ago
@santisoler Lo hago. Aún me confundo por falta de práctica. Disculpen, lo correcto es que yo haga una rama, haga estas modificaciones y después pull request?
Todo bien @MGomezN ! Sí efectivamente, deberías hacer lo siguiente:
git clone ...
).cd Intro-to-python
main
con git pull
git switch -c data
(por ejemplo)git add ...
, git commit
)git push --set-upstream origin data
(fijate que tenes que poner el nombre de la rama, y el --set-upstream
es para que tu rama local "trackee" cambios con respecto a la remota).Cualquier cosa no dudes en preguntar!
Gracias @santisoler ah ya, mismo procedimiento que con mi pull request para Boule. Este si será exitoso espero. No había visto que abriste un issue. Leí todo en los avisos de correo y estaba confundida. Gracias
Jaja, si puede ser confuso al ppio.
Y si, seria lo mismo q con Boule, pero acá no tenes q forkear pq ya tenes permiso para pushear a este repo 😉
Ah! ... Jaja no entendía por qué nadie había forkeado y luego el ReviewNB de dónde había salido :S y por qué me llegaban hartos correos... me sentía en un kayak, en unos rápidos, muy rápidos... (from meme import desesperacion)
Pero.. Ya los saludo desde la app Github en mi teléfono. Orden y progreso.
Estaría bueno mover todos los archivos de datos que usaremos en el curso a su propio directorio. La estructura del repositorio podría quedar:
De hacer eso, deberíamos modificar los notebooks que acceden a esos archivos. Una buena práctica para eso es no utilizar strings para definir los paths a los archivos, y sí utilizar librerías. Una solución que se usa bastante es usar el módulo
os.path
, aunque hay herramientas mucho mejores comopathlib
.Por ejemplo:
El
Path("..")
hace referencia al directorio padre del directorio en el que nos encontramos, es decir, el inmediato superior. Luego podemos armar paths con strings a través del operador "/", el cual es reemplazado por el correspondiente separador de paths del sistema operativo en el que lo corremos.@MGomezN tenés ganas de implementar esto?
(Idea original de @MGomezN en #2)