ta-data-bcn
commented
10 months ago Hola Sara!!!! 👋🏾 Te dejo un poco de feedback sobre tu proyecto
Respecto a la organización del repositorio 📚
- Muy bien organizado el repo, te recomiendo dejar un notebook o archivo .py y modularizar tus funciones para poder ejecutarlas todas a la vez.
- No tienes .gitignore, que has hecho con el DS.Store, .env, etc? Los has borrado manualmente? 😱😱😱
El README.md ✍🏽
- Está muy bien explicado, mencionas todo el proceso que has seguido y acabas con una conclusión que makes sense.
- Igual puedes poner las librerías que usaste y los métodos: Mongo queries, Foursquare API, etc.
- En general muy bien organizado y presentado.
El código 👨🏽💻
-
Bien sacando toda la información de Mongo con queries.
-
Las llamadas a la API de Foursquare han fallado, si te fijas hay muchos aeropuertos que no lo son, para solucionar este problema, Foursquare tiene categorías especiales para diferentes establecimientos. Si usas estas referencias no te saldrán resultados indeseados en tus llamadas a Foursquare. https://location.foursquare.com/places/docs/categories
-
El código en general está bien organizado, podrías ir poniendo comentarios para seguirlo mejor.
-
He visto en el primer file que hay funciones que defines (companies_ubi, create_df) pero luego no las llamas.
-
También te recomiendo no printear listas ni diccionarios enteros cuando los quieras ver, si te va bien durante el proyecto adelante, pero antes de pushear yo le pondría un .sample(). Así no quedarán listas ni diccionarios largos en tu notebook
Visualización (mapas) 🗺
- Te han quedado muy bien los mapas. Muestran lo que tienen que mostrar para argumentar tu hipótesis✌🏾
En general, está muy bien el proyecto Sara! 👏🏾 hiciste lo que se pedía y justificaste bien tu decisión de crear la compañía en SF! Felicidades por el BONUS también, aunque no vea por ningún lado de dónde has sacado la data jeje.
Muchas felicidades!!! Sigue así que ya queda poco!!!!
https://github.com/sarabenitezinglott/project-3.git