You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
El método buscar de la clase Municipio actualmente se basa en encontrar, dada una cadena nombre aquellos municipios de la lista del INE que contengan la misma cadena en el campo NOMBRE. Esto resulta en unos efectos secundarios, p.ej. dada la secuencia "Madrid", el método devolvería los objetos Municipio correspondientes a las siguientes localidades:
Humanes de Madrid
Madrid
Rozas de Madrid, Las
Madridejos
Madridanos
o por ejemplo si se busca una cadena con faltas de ortografía el método devolvería una lista vacía.
Propongo usar las mejores prácticas en estos casos que es un string matching basado en la distancia Levenshtein por ejemplo usando la librería fuzzywuzzy.
Entiendo que es un uso concreto de la librería y quizá no queremos empezar a añadir dependencias por que sí, pero lo dejo aquí por si recibe votos positivos.
Subiré la PR con el código que implementa esta mejora por si dado el caso que recibiera apoyos se pudiera hacer merge más fácilmente.
The text was updated successfully, but these errors were encountered:
jmrr
changed the title
Mejorar la busqueda de municipio
Mejorar la búsqueda de municipio
Apr 4, 2021
El método
buscar
de la claseMunicipio
actualmente se basa en encontrar, dada una cadenanombre
aquellos municipios de la lista del INE que contengan la misma cadena en el campoNOMBRE
. Esto resulta en unos efectos secundarios, p.ej. dada la secuencia "Madrid", el método devolvería los objetos Municipio correspondientes a las siguientes localidades:o por ejemplo si se busca una cadena con faltas de ortografía el método devolvería una lista vacía.
Propongo usar las mejores prácticas en estos casos que es un string matching basado en la distancia Levenshtein por ejemplo usando la librería
fuzzywuzzy
.Entiendo que es un uso concreto de la librería y quizá no queremos empezar a añadir dependencias por que sí, pero lo dejo aquí por si recibe votos positivos.
Subiré la PR con el código que implementa esta mejora por si dado el caso que recibiera apoyos se pudiera hacer merge más fácilmente.
The text was updated successfully, but these errors were encountered: