Closed julistary closed 3 years ago
Hola Julieta! 🙋🏻♂️
Te voy contando:
Respecto a la organización del repositorio 📚
El README.md ✍🏻: está genial. Explicas el objetivo del proyecto, los medios, las librerías y el procedimiento. Nada que decir.
Bueno, sí, aprovecha que es un proyecto muy visual y añade una captura del mapa en el README si quieres.
Respecto al código:
En src:
Los docstrings están súuuper bien: explicas lo que hacen, los argumentos que reciben y devuelven y el tipo.
geoqueries.py
Aquí:
def grades(x):
if x == 0:
return 0
elif (x > 0) and (x <= 8):
return 1
elif (x > 8) and (x <= 20):
return 2
else:
return 3
igual puedes utilizar un diccionario, si te resulta más cómodo.
Y aqui:
for l in list_:
if l['place'] == 'party':
party += 1
elif l['place'] == 'starbucks':
starbucks += 1
elif l['place'] == 'school':
school += 1
elif l['place'] == 'vegan':
vegan += 1
elif l['place'] == 'basket':
basket += 1
puedes iterar sobre l in list_ y también sobre party, starbucks, etc. Así sólo te haces un bucle doble con una sola condición.
Para los jupyter:
No te puedo decir mucho más del código. El proyecto está genial Julieta. Tanto por forma como por contenido como planteamiento (incluso con los requirements y el yml). De pasos extra, si quisieras hacer, sería:
Pero vamos, que enhorabuena por el trabajo!! Vas en cohete!! 🚀
https://github.com/julistary/GeoSpatial_Data_Project.git