ta-data-bcn
commented
1 year ago Hooola David! 👋🏾
Te doy un poco de feedback del proyecto.
La organización del repositorio 📚
- No subiste el .env y no se te coló ningún archivo extra al repo 🔝
- La organización del repo está bien.
- Podrías hacer un folder "src" para meter los archivos .py
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.
- Podrías poner las librerías y las herramientas que usaste.
El código 👨🏽💻
- Bien sacando toda la información de Mongo con queries. -Las llamadas a la API de Foursquare están mejorables, puedes resumirlo todo en un sola función pidiendo como variable el nombre de la query que buscas.
- Está muy bien que hayas puesto las funciones en un archivo .py, pero hubiera estado mejor que las hubieras usado jajaja me imagino que no te dio tiempo, pero se ve muuuuucho más limpio si lo haces.
- También te recomiendo ponerle docstrings a tus funciones para que sepas lo que hace cada una y para que la gente que ve tu código sepa.
- Buen análisis para encontrar el lugar perfecto para hacer la compañía :)
Visualización (mapas) 🗺
- Bien por poner markers en los diferentes venues, para mostrar que la ubicación que escogiste cumple con todos los requisitos.
- Los mapas están bien, se entienden y muestran lo que tienen que mostrar. Para el primero de EUA, podrías haber hecho un heatmap, creo que se vería más claro.
En general, está bien el proyecto David! 👏🏾 podrías reducir alguna parte del código y hacer la parte de modularización, pero está bieeen.
Muchas felicidades y disfruuuta el fin de semana de descanso 🚀
https://github.com/David231723/Project-3---Maps