Closed Juanpda closed 2 years ago
Hola Juan 🙋🏻♀️! Seguimos con más proyectos, y ahora es el turno del proyecto de las geoqueries!
En general el readme esta super bien Juan! Solo te djo por aquí un par de detalles:
En general esta super bien organizado el repo Juan, buen trabajo.
En archivo src
se te ha pasado un archivo temporal, el __pycache__
. Recuerda que estos archivos temporales deben ir en el .gitignore
.
Peero super bien que no te hayas dejado los tokens a la vista 👏🏽!
Tienes varios archivos json
, lo ideal en este caso es que nos creemos una carpeta que se llame por ejemplo data
donde vayamos guardando todos los ficheros, tanto json
como csv
Lo ideal es que tengamos modularizado el repo en carpetas. En este sentido, podemos crearnos una carpeta que se llame notebooks
donde guardemos todos los notebooks de nuestro proyecto. Si además numeramos los notebooks para saber el orden de trabajo,s sería TOP.
Tienes cosas en ingles y en español, lo ideal es que lo unificaras y lo pusieras todo en el mismo idioma.
Vamos con los docstrings de tus funciones. Por convenio, los docstrings van dentro de las funciones.
¿Qué es lo que tiene que tener un docstring?
Aquí algo de info sobre los docstrings .
def mifuncion(argumento1, argumento2):
'''
Lo que hace la función
args:
argumento1: tipo de dato
argumento2: tipo de dato
return:
que nos devuelve la función
'''
return "Hola"
Estoy viendo código muy interesante Juan, muy buen trabajo. Te dejo por aquí un detallin por si te aporta valor.
En esta función get_google_nearby_data
, tienes definidos estos dos diccionarios:
radius = { #Distancia medida en metross
"train_station":"10000",
"school":"5000",
"night_club":"2000",
"airport":"15000",
"cafe":"1000"
}
ponderations = {
"train_station":10,
"school":50,
"night_club":25,
"airport":10,
"cafe":5
}
Cuando nos creamos este tipo de diccionarios lo que normalmente se hace es crearnos otro .py
a parte donde vamos a ir guardando todos estos diccionarios. Este fichero funcionará como un archivo de configuración y lo llamaremos desde el .py
donde tenemos las funciones como lo hacemos normalmente.
No he visto por ninguna parte como haces las geoqueries. Era parte esencial del proyecto. Meter los datos en una base de datos de mongo y hacer geoqeuries...
Por el resto poco más Juan, tienes un proyecto impecable. Enhorabuena 🥳
Bueno Juan, te dejo por aquí un pequeño resumen de las cosinas que te dije:
Near
o GeoNear
. Venga que ya no queda nadaaaa 🔥
https://github.com/Juanpda/Proyecto_GeoSpatial.git