ta-data-bcn
commented
1 year ago Bueeeeenas Daani! 👋🏾
Te doy un poco de feedback del proyecto.
La organización del repositorio 📚
- El .gitignore está bien, no subiste el .env. Sólo se te coló el archivo .ipynb_checkpoints 👀
- Aparte de eso, la organización del repo está bien.
- Puedes sacar el archivo de cleaning.py y el de geoqueries.py que están vacíos.
El README.md ✍🏽
- Está bien. Muestras el proceso que seguiste, la justificación de tu decisión de crear la compañía en San Francisco y el resultado final. También muy bien por poner todas las herramientas y librerías.
El código
- Bien sacando toda la información de Mongo con queries.
- Las llamadas a la API de Foursquare están bien también.
- En tus notebooks trata de limitar la impresión de algunas cosas a sólo 10 variables, para que sea más fácil de leer una vez que esté en github.
- En general el código está bien, está bien la limpieza y la extracción de datos.
- Está muy bien organizado, muy bien por escribir docstrings en las funciones, la Dani del futuro seguuuro lo agradece.
- Bien por hacer archivos .py con todas las funciones. El siguiente paso sería usarlas jajaja porque vi que en vez de importarlas a los notebooks, las volvías a escribir, y esto lo hace un poco repetitivo.
- Podrías haber ido un paso más allá para encontrar el lugar exacto en donde hacerlo, y no sólo la ciudad.
Visualización (mapas) 🗺
- Gran idea hacer un heatmap para encontrar visualmente la ciudad con más compañías.
- Bien por poner markers en los diferentes venues, para mostrar que la ciudad que escogiste cumple con todos los requisitos.
- En general los mapas están muy bien :)
En resumen, está bien el proyecto Daani! 👏🏾 faltaría acabar de afinar la parte de modularización, y si quisieras ir un paso más allá, encontrar la ubicación exacta de la compañía.
Muchas felicidades y disfruuuta el fin de semana de descanso 🚀
https://github.com/danidimarti/project3-geospacial-data