bripollc
commented
8 months ago Felix,
Antes de todo felicidades, segundo proyecto su-pe-ra-do!!! Se nota que le has metido horas y has usado la API de spotify, que no es facil 🫠 Así que felicidades! Te dejo un poco de feedback:
🧹 Repository organization
- Top!!!!! El repositorio está muy 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:) Felicidades, no tengo nada a comentar.
👨🏽💻 Code
- Tu código está muy limpio!!! Encapsulado en funciones, dentro de archivos .py, muy buen trabajo! Los nombres de tus funciones son claros y, junto con los comentarios, me ayudan a identificar que es cada parte. Lo que me hubiera ayudado ver en algun notebook el proceso que has seguido para desarrollar el proyecto, porque no veo nada de DATA! Solo funciones... Y es difícil descifrar que datos usas, que hace tu código, etc.
- También y SUPER importante❗️❗️❗️ Saca cuanto antes tus credenciales de SPOTIFY! Las has subido en auto_play.py. Acuérdate de usar .env la próxima vez.
🎨 Visualizations
- Las visualizaciones están bien! Hay títulos, labels y se entiende lo que quieres visualizar sin que lo tengas que explicar. Quizás me falta un poco de profundidad, creo que con los datos que tenías podrías haber hecho unos gráficos con más detalle e información más relevante y representativa.
✍🏽 Readme
- Bien! Hay readme, que ya es un gran paso! Pero me falta un poco de estructura: apartados, subapartados, bullet points, etc. De esta manera será más fluido de leer y facilitará entender tu trabajo y el proceso que has seguido para conseguirlo. Puedes jugar con el número de # en el readme para crear varios niveles de información.
- No haces referencia en ningún momento a los datos con los que has trabajado y me ha costado un poco entender tu proyecto 🥲
- Por otro lado, me cuesta entender tu apartado Key Features. Es un "mix" entre tu metodología de trabajo, tus key features y los plots que has usado... Yo dividiría la información en diferentes apartados para que quede más claro que es casa cosa.
- No acabo de entender del todo tus conclusiones, al no ver las gráficas también me falta aquí un poco de ayuda. La próxima vez, intenta añadir tus gráficos al readme porque al final será lo que explique mejor tus conclusiones. Puedes hacerlo con
En general me parece un trabajo con potencial pero con cosillas a pulir! Pero ojo, que se los problemillas técnicos que has tenido y se el esfuerzo que has metido:) Ya tendrás tiempo para retocarlo. Te lo estás currando y se nota tu motivación por el curso. SIGUE ASÍ!!!!!! 🚀🚀
https://github.com/imfap1/Project-II