bripollc
commented
8 months ago Marta,
En primer lugar, segundo proyecto superado! Se nota que le has metido mucho esfuerzo y horas! Y así lo demuestra este trabajazo!!!!!!! Felicidades ✨🌻 Te dejo un poco de feedback:
🧹 Repository organization
- Impecable!! El repositorio está muy bien organizado. Has creado las carpetas necesarias y los nombres de los archivos son claro.
👨🏽💻 Code
- Como siempre tu código es limpio, legible y muy muy estructurado. Nada a comentar. Sigue así Marta! Ps: Telita con la función cleaning_Web, hubo trabajo duro ahí eh!
- El next step será poner docstrings dentro de las funciones para documentar y que se entienda mejor qué hace cada una :)
- 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.
🎨 Visualizations
- La visualización en general está bien! Has ploteado varios gráficos y se entiende lo que quieres representar en cada uno de ellos. Lo único, es que en algunos los nombres de los labels podrían ser más representativos.
- También hay algunos plots en los que podrías escoger otro tipo de gráfico para visualizar varios datos a la vez, en lugar de hacer plots separados. Por ejemplo en aquellos que comparas las editoriales que más han vendido en España por años, etc.
- Ojo, en el gráfico amazon_genres_years.png los años están como floats? (2020.0, 2020.5...) No tiene sentido, xd.
- Por último, también he visto alguna leyenda superpuesta encima del gráfico y otros que se cortan (aunque no se si es por visualizarlo en GitHub).
✍🏽 Readme
- Top! Hay títulos, subtítulos, bulletpoints etc. Además está todo muy bien explicado, estructurado y referenciado! Es fluido de leer y se entiende que has usado, tus planteamientos, tu proceso para llegar a tus resultados y las conclusiones :)
- No se te han subido correctamente las imágenes del readme otra vez 🥲 Mhh, quizás es por las
\del path que indicas, que deberían ser/?
En general me parece un trabajo muuuuuy completo con pocas casi nada 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!!!!!! 🚀🚀
https://github.com/MartaGDC/Project-2
The cleaning of the other databases from kaggle and the building of a dataframe from web scrapping are available in py files, executable from main.py. But the modularization is not working completly because of an error ocurring while cleaning the dataframe made from web scrapping, but I didn't have time to find it. Visualization.py doesn't fully work because of the functions that require the faulty dataframe.