Documenta el caso de uso de la predicción de temperaturas por municipio y añade fuzzy search#28
Documenta el caso de uso de la predicción de temperaturas por municipio y añade fuzzy search#28jmrr wants to merge 8 commits intopablo-moreno:masterfrom
Conversation
| return municipios | ||
| except: | ||
| return None | ||
| nombres_municipios = list(map(lambda m: m.get("NOMBRE"), Municipio.MUNICIPIOS)) |
There was a problem hiding this comment.
Entiendo la funcionalidad y me parece bien mejorar la búsqueda de municipios en general, pero quiero evitar añadir más dependencias de las necesarias y creo que no es el objetivo principal de lo que nos ocupa.
There was a problem hiding this comment.
Sí tienes razón. Te parece que cree un issue para mejorar la búsqueda? Es que tal cual estaba en el init no funcionaba, devuelve una lista de todos los municipios con la palabra que metes en el argumento en la línea de comandos.
docs/README.md
Outdated
| pip install virtualenv | ||
| ``` | ||
|
|
||
| Clona el repo `git clone git@github.com:pablo-moreno/python-aemet.git && cd python-aemet` |
There was a problem hiding this comment.
Aquí sería mejor hablar de instalar la librería desde pypi, que es como está en el README.md
pip install python-aemetThere was a problem hiding this comment.
Cierto, ida de olla. Entonces quito todo lo del virtualenv que eso es gestión de entorno. Que cada usuario se apañe digamos.
Resumen
docs, potencialmente se puede mover a la wikiDescripción de la contribución
La motivación fue que al usar la interfaz CLI me di cuenta de que el método
buscarde la claseMunicipiodevolvia una lista con los municipios cuya busqueda simple por nombre arrojara los resultados que contuvieran la cadena de entrada.Pienso que quiza devolver el municipio mas cercano de una busqueda hecha con
fuzzywuzzyy la distancia Levenshtein que es lo que quiza el usuario anda buscando.Por ejemplo, una cadena incompleta como:
Arrojaria:
Y otra con faltas de ortografia como:
Resultaria:
Como lo veis?