bripollc
commented
1 year ago Amir,
En primer lugar, segundo proyecto superado! A pesar de los problemillas técnicos que has tenido, se nota que le has metido mucho esfuerzo (y horaaaas)! Felicidades ✨🥳 Te dejo un poco de feedback:
🧹 Repository organization
- Top!!!!! El repositorio está bien organizado. Has creado las carpetas necesarias y los nombres de los archivos son claros, cosa que aprecio mucho porque me ayuda a entender que es cada cosa:)
- Lo único ojo, que en images/ hay algunos nombres poco intuitivos y ¿creo que se te ha subido alguna imagen innecesaria? jeje (ej: 43508840.jpg)
👨🏽💻 Code
- En general muy muy buen trabajo limpiando y usando diferentes métodos! Tu código es limpio, ordenado y los nombres de las funciones son claras. Te será útil en un futuro para ti y para quien lo lea :) Pero, aunque está super bien explicado en el readme, me cuesta un poco seguir el hilo de lo que ocurre en tu código porqué hay bastantes archivos en src/ y en algunas partes no hay comentarios o docstrings para documentar las funciones.
- También... ¿puede que tu archivo main.ipynb esté sin acabar?
- Por último, buen trabajo!!!!!! Has conseguido scrapear bastante información y has sabido como juntarlo para enriquecer tu csv original, que era el objetivo principal del proyecto. Bien ejecutado Amir! Felicitats!
🎨 Visualizations
- La visualización está bien, no tengo mucho a comentar. Los gráficos están bien escogidos, tienen títulos, labels, leyendas y se entiende lo que quieres mostrar sin que lo tengas que explicar:)
✍🏽 Readme
- El readme en general bien y es divertido! Hay títulos, subtítulos, bulletpoints etc. Está todo muy bien explicado, estructurado y referenciado! Es fluido de leer y se entiende que has usado y tu métodología de trabajo.
- Lo único que echo de menos es el análisis/resultados extraídos de tu data. También incluir tus gráficos, al final es lo que mejor explicará tus resultados:)
En general me parece un trabajo bastante completo con solo algunas cosillas a pulir! Se te nota con energía durante el curso y no paras de currar!!!! SIGUE ASÍ!!!!!! Lo harás genial 🚀🚀
https://github.com/amirg33/Project-II-Data-Pipeline