Marco project 2/pipelines

[marcoayora]

https://github.com/marcoayora/Project-2-pipelines-marco

[bripollc]

Marco,

En primer lugar, segundo proyecto superado! Se nota que le has metido horas y así lo demuestra este trabajazo!!!!!!! Felicidades 🙌🏼 🥳 Te dejo un poco de feedback:

🧹 Repository organization

• Buena mejora en la organización de tu repo desde el proyecto anterior. Esta todo en carpetas y los nombres de los archivos son claros, cosa que me ayuda a identificar que es cada cosa. Lo único, yo pondría todas las imágenes del repo principal en la carpeta images/ y el ola.csv (que no tiene un nombre muy intuitivo) lo metería en data/. Pero aparte de esto molt bé!!!!! También ojo, que el ola.csv vuelve a estar en la carpeta src/.

👨🏽‍💻 Code

• Suuuuuper! Buen trabajo encapsulando y modularizando tu código en archivos py. Que bonito.🥲 Sé que puede costar un poco este tema al principio y puede frustrar cuando se rompe el código, pero es cuestión de practicar y de paciencia. Te simplificará mucho el código, estará más estructurado y podrás reutilizarlo. Todo son ventajas!
• El next step será poner docstrings dentro de las funciones para documentar y que se entienda mejor qué hace cada una :)
• No hace falta que importes todas las librerías en cada .py, solo las necesarias para hacer funcionar ese archivo en concreto. Por ejemplo, seaborn no hace falta importarlo en extraction.py.
• No acabo de entender que es el código de tu cleaning.py y también hay un archivo test.py, que creo que lo has pusheado sin querer?
• Para acabar, buen trabajo, has conseguido scrapear varias páginas y has sabido como juntarlo para enriquecer tu csv preexistente, que es lo que requería el proyecto. Así que felicitats! Bien ejecutado Marco:)

🎨 Visualizations

• Las visualizaciones están bien! Hay detalles y has explorado varios tipos de gráficos:) Algunos por eso creo que no son los más representativos, aunque se entiende lo que quieres representar! Al final, no tener una conclusión también es una conclusión, pero ¿quizás sería más conveniente utilizar otro tipo de plot?

✍🏽 Readme

• Top! El readme está muy bien! Hay títulos, subtítulos, bulletpoints etc. La lectura es muy fluida y es fácil de seguir el hilo de tu proyecto. Además me gusta el enfoque personal que le has dado!
• Para los links puedes usar [Nombre_Link](https://TU-URL.com)

En general me parece un trabajo completo con pocas cosas a pulir! Te lo estás currando mucho y se ve tu motivación por el curso. Has conseguido de largo el objetivo principal del proyecto, FELICIDADES!!!!!! 🚀🚀