ta-data-bcn
commented
1 year ago Bueeenas Victor! 👋🏾
Aparte de lo que comentó Fer en tu presentación, te doy un poco de feedback del proyecto.
Respecto a la organización del repositorio 📚
- El gitignore funciona bien, pero podrías agregar igual el .DS_Store y el .ipynb_checkpoints para que no se suban al repo y haya más orden.
- Aparte de eso, podrías hacer un folder da data para meter todos los json's y otro "src" para meter los archivos .py (cuando los haya).
El README.md ✍🏽
- Está bien, bien explicado y visual. Muestras el proceso que seguiste y la justificación de tu decisión de crear la compañía en Alemania. Te falta desarrollar la última parte y agregar mapas para justificar la decisión final, pero me imagino que lo ibas a hacer y fue por falta de tiempo.
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.
- En tu notebook de limpieza trata de limitar la impresión de algunos resultados a sólo 10 variables, para que sea más fácil de leer una vez que esté en github.
- Podrías comentar un poco más el código, para que sea más fácil (para ti y para los que te leen) entender qué estás haciendo en cada parte.
- Acuérdate de poner tu código en funciones 👀
- En general el código está bien, está bien la limpieza y la extracción de datos. Así que buen trabajo.
- Muy bien por sacar la densidad de cada lugar y así tomar una decisión más acertada :)
Visualización (mapas) 🗺
- Buen uso de heatmaps y markers :) también muy bien por dibujar el radio alrededor de diferentes compañías y ver la diferencia en las distancias.
- Como tú dijiste, luego puedes hacer un mapa sólo con markers para que se vea más limpio. Pero de todas formas está bien el que tienes para ver la densidad de manera más clara.
En general, está bien el proyecto Victor! 👏🏾 hiciste lo que se pedía y justificaste bien tu decisión de crear la compañía en Alemania...puedes mejorar algunas cositas, pero en general, buen trabajo. Muchas felicidades y mucho ánimo para el siguiente.
https://github.com/victorjrp92/project-3.git