Patricia Sáez-Torres

[psaeztorres]

https://github.com/psaeztorres/PROJECT-III-Geolocalization.git

[ta-data-bcn]

Hola Patricia S!!!! 👋🏾 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
• Hay un csv suelto que yo metería dentro de la carpeta de data.
• Bien estructurado y todas las funciones dentro de la carpeta de src.

El README.md ✍🏽

• Está muy bien explicado, mencionas todo el proceso que has seguido, y la estructura es buena. Es importante que las decisiones que tomes las hagas apoyándote en los datos analizados y sus resultados.
• Puedes poner las librerías que has usado y los métodos: Mongo queries, Foursquare API, etc.
• No veo mapas en el readme, te recomiendo apoyarte en imágenes dentro del readme, para que las explicaciones se entiendan mejor.
• Te recomiendo añadir una parte de conclusión, dónde explicas tus resultados y los métodos.
• También recomiendo añadir una parte de Next Steps, en la que puedes explicar posibles mejoras dentro del proyecto y cómo las harías.

El código 👨🏽‍💻

• El código está super bien!! Perfectamente modularizado.

• 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.

• Para buscar establecimientos específicos en la API de Foursquare, no solo está la opción de buscar por el nombre, sino que hay una lista de categorías que se pueden buscar y te dan un resultado mucho más fiable. Para estadios de Basket o Aeropuertos se tendría que usar, ya que sino pasa lo que te ha pasado. Metiendo el código 10061, Foursquare sí te va a dar una respuesta fiable de estadios de Basket. https://location.foursquare.com/places/docs/categories

• También en la parte de pets, te han salido muchos establecimientos que empiezan por pet, pero no son de pet grooming services.

• Procura no dejar errores dentro del notebook final, puedes utilizar un try/except y que en vez de saltarte el error puedas poner un mensaje.

Visualización (mapas) 🗺

• Los mapas están bien, podrías haber utilizado diferentes iconos para describir los establecimientos. También puedes utilizar heatmaps para ver las zonas con más densidad de empresas.
• Solo se han creado los mapas de New York y San Francisco, miraría que ha pasado con los otros.
• Otra opción que puedes añadir a tus mapas son layers para ver una o más categorías de establecimientos a la vez.

Bravo por tu proyecto Patricia! El código está realmente limpio y organizado. Faltan algunos detalles dentro del readme y en la parte de los mapas, por lo demás te felicito!!! Está muy bien, sigue así que ya casi estamos!!