PROJECT III - MIQUEL PUIG

[Miquelpg6]

https://github.com/Miquelpg6/project_3

[ta-data-bcn]

Hola Miki!!!! 👋🏾
Te dejo un poco de feedback sobre tu proyecto

Respecto a la organización del repositorio 📚

• El repo está muy bien organizado, el .gitignore funciona bien e ignora el .env :)
• Bien estructurado y todo en sus respectivas carpetas, dejando solo el main.py para ejecutar el proyecto entero!! Felicidades!

El README.md ✍🏽

• Está muy bien explicado, mencionas todo el proceso que has seguido.
• Los diferentes approaches a los establecimientos están muy bien, pero yo lo dejaría como comentario dentro del notebook.
• En el readme haría una tabla con los establecimientos y los puntos que recibe cada ciudad, creo que quedaría más ordenado y te daría más tiempo para hablar de los otros procedimientos que has seguido durante el proyecto.
• Falta algo de conclusión, un pequeño apartado dónde hablas de tus distintos findings, tus decisiones en el proceso del proyecto, y tu resultado final. Hablando también de cómo has elegido el punto final.

El código 👨🏽‍💻

• El código está super bien!! Perfectamente modularizado. No estoy seguro si ejecutando el main.py se ejecuta el proyecto entero o solo una parte. Lo tendría que descargar y probar pero a primera vista parece que hay funciones que no se utilizan en el main.py.
• Te recomiendo ir poniendo algún docstring o comentario para que la comprensión de lo que hace cada función sea más fácil.
• Te felicito porque está en general súper bien el código!! Las llamadas a la api y las queries en Mongo también!
• Intenta no printear jsons o diccionarios largos, ya que hace la lectura complicada, o una vez vayas a pushear el proyecto dejalos en .head o .sample para que no se printee entero.
• También si hay algun error es mejor quitarlo o si te sirve para explicar algo usar un try except para que no salte el error grande.
• No acabo de encontrar la parte donde llegas a las coordenadas finales.

Visualización (mapas) 🗺

• Te han quedado muy bien los mapas. Muestran lo que tienen que mostrar para argumentar tu hipótesis✌🏾
• Bien puestos los iconos y los colores.
• También super top utilizar heatmaps!!!! El único que he visto hasta ahora!!!!!!
• Sería chulo una vez has elegido el punto de las nuevas oficinas poder verlo también en el mapa. También podrías dibujar una circunferencia alrededor de las oficinas para mostrar a la distancia que quedan los diferentes establecimientos.

En general está muy bien todo. Lo único que remarcaría la parte del readme, que acabes de detallar los pasos que has seguido y las razones detrás de cada paso.

Muchas felicidades!!! Se nota que te lo has currado y le has metido tiempo y esfuerzo!!! Sigue así que ya solo te queda uno más y luego ya el final!!!!!