bripollc
commented
1 year ago Pati,
Primer proyecto su-pe-ra-do, y muy positivamente! 🫠 Deberías estar contenta. Buen trabajo. Te dejo un poco de feedback:
🧹 Repository organization
El repositorio está bien organizado, limpio y es fácil de navegar! No hay muchos files y están ordenados en carpetas, cosa que aprecio mucho:) Los nombres de los archivos que has subido son claros y me ayudan a identificar que es cada cosa. Yo quizás, y por decir algo, pondría los notebooks en una carpeta notebooks para dejar solo fuera el readme.md (documentación de tu proyecto) y el main.py (archivo principal de ejecución). También, en la carpeta src/ te recomiendo agrupar el código encapsulado en tres archivos: cleaning.py (con las funciones para la limpieza), transforming.py (con las funciones para la transformación) y visualizing.py (para las funciones de los plots). Pero aparte de eso está muy bien!!
👨🏽💻 Code
En general muy muy buen trabajo limpiando y usando diferentes métodos que hemos visto. Tu código está limpio y los títulos y comentarios son claros y me ayudan a entender el proceso que has seguido. Te será útil en un futuro para ti y para quien lo lea :)
Genial por otro lado, porque ya has puesto el código en funciones!!! Lo único que te añadiría es sobre la estructura. Has trabajado con funciones para cleaning y funciones para tranforming+visualizazing. Yo te recomendaría encapsular el código siguiendo la metodología de trabajo: funciones para cleaning, funciones para transforming y funciones para visualizing. De esta manera, será mucho más rápido y cómodo hacer modificaciones en un futuro.
🎨 Visualizations
La visualización está bien, no tengo mucho a comentar. Los gráficos tienen labels y se entiende lo que quieres mostrar. Lo único que te diría es que ojo, porque no hay título en algunos de los gráficos. También ordenaría descendente o ascendentemente los gráficos de los países.
✍🏽 Readme
En general muy bien explicado y bien estructurado. Se entiende que has usado y lo que has hecho en tu proceso de trabajo. Lo único que añadiría es el proceso que has seguido de inicio a fin (workflow) y referenciar el origen de tus datos (ya sea en un nuevo apartado "Dataset" o en los links del final).
Felicidades! Que no decaiga la energía y a por el segundo proyecto🌻✨
https://github.com/patriciazapatab/project-1