-
Notifications
You must be signed in to change notification settings - Fork 4
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
Guardar archivos de datos en su propio directorio #5
Comments
@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:
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. |
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)
The text was updated successfully, but these errors were encountered: