ta-data-bcn
commented
1 year ago Bueenas Santi! 🚀
Aquí te va un poco de feedback del proyecto.
Respecto a la organización del repositorio 📚
- El .gitignore: no subiste el .env 🔝 , pero se te coló el archivo de .DS_Store y el pycache, la próxima agrégalos al gitignore para que no se suban al repo, ya que no suman nada.
- Aparte de eso, la organización del repo está muy bien. Te recomiendo hacer otro folder llamado "src" para meter los archivos .py
El README.md ✍🏽
- Está súper bien, muy bien explicado. Muestras el proceso que seguiste, las librerías que usaste y la justificación de tu decisión de crear la compañía en Londres, así que 🔝
El código 👨🏽💻
- Bien sacando toda la información de Mongo con queries.
- Las llamadas a la API de Foursquare también están bien, peeeero podrías haber hecho una sola función para todas, en la que sólo le cambiaras el nombre de la query y las coordenadas.
- Intenta poner doctrings en tus funciones para que sepas qué hace cada una, una vez vuelvas a ver tu código.
- También muy bien la parte de modularización, aunque podrías haber pasado unas cuantas líneas de código más a funciones, me imagino que fue por falta de tiempo, pero intenta hacerlo, se ve más limpio.
- En general el código está bien, está bien la limpieza y la extracción de datos.
Visualización (mapas) 🗺
- Buen uso de heatmaps para encontrar la zona con mayor número de compañías tech y de diseño.
- También muy bien por poner markers en los diferentes venues, para mostrar que indeed la ubicación que escogiste cumple con todos los requisitos.
En resumen, está bien el proyecto Saanti! 👏🏾 podrías agregar más funciones y reducir algunas partes del código, peeero en general está bien :)
Muchas felicidades y ánimo en el siguiente 🚀
pd. acuérdate de hacer muuuuchos commits siempre 👇🏾
santiago20233
https://github.com/santiago20233/Project-III---Geospatial-Data