osm-fr / bano

Différents outils pour la constitution de la Base Adresses Nationale Ouverte
Do What The F*ck You Want To Public License
30 stars 18 forks source link

Ne filter pas uniquement sur le nom quand des noms alternatifs peuvent être utilisés #396

Closed frodrigo closed 2 months ago

frodrigo commented 7 months ago

Ne filter pas uniquement sur le nom quand des noms alternatifs peuvent être utilisés.

J'ai supprimé les conditions, l'autre option est de mettre des conditions sur l’ensemble des champs name.

vdct commented 7 months ago

tu as des exemples de ce que ça permet ?

frodrigo commented 7 months ago

Non. Je n'en ai pas cherché, c'est théorique. C'est en fait une sous partie d'une modif plus large pour supporter les langues locales et les noms multiples (name:xx).

Mais je peux chercher des exemples.

El mar, 20 feb 2024, 18:57, vdct @.***> escribió:

tu as des exemples de ce que ça permet ?

— Reply to this email directly, view it on GitHub https://github.com/osm-fr/bano/pull/396#issuecomment-1954869969, or unsubscribe https://github.com/notifications/unsubscribe-auth/AANT5DXFZHHHQUW2HDJAZQ3YUTWZ7AVCNFSM6AAAAABDRPO7BCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNJUHA3DSOJWHE . You are receiving this because you authored the thread.Message ID: @.***>

frodrigo commented 7 months ago

On trouve bien des old_name ou alt_name sans name. Mais aussi des name:eu, name:oc...

[out:json][timeout:25];
{{geocodeArea:Aquitaine}}->.searchArea;
way[highway][!"name"]["old_name"](area.searchArea);
// print results
out geom;
vdct commented 7 months ago

c'est théorique

En l'état les modifs que tu proposes retournent des enregistrements avec parfois une valeur null pour le champ name en sortie alors qu'on cherche justement à récupérer des noms. Donc ce n'est pas raccord avec ce qu'on attend de ces requêtes. Mais je me demande s'il faut affiner car plus généralement je ne suis vraiment pas pour tolérer des cas de tagging où on aurait pas de tag name rempli et uniquement old_name ou alt_name. Je trouve que c'est une incitation à ne pas aller dans le bon sens côté contribution OSM. J'ai regardé les exemples que remonte ta requête Overpass, sur l'Aquitaine et ailleurs, et je trouve qu'il y avait à redire sur àpeu près tous les objets que ça ramène.

frodrigo commented 7 months ago

c'est théorique

En l'état les modifs que tu proposes retournent des enregistrements avec parfois une valeur null pour le champ name en sortie alors qu'on cherche justement à récupérer des noms.

J'avais vérifié qu'il y avait bien des noms, mais je peux revoir ça.

Donc ce n'est pas raccord avec ce qu'on attend de ces requêtes. Mais je me demande s'il faut affiner car plus généralement je ne suis vraiment pas pour tolérer des cas de tagging où on aurait pas de tag name rempli et uniquement old_name ou alt_name.

Je cherche à l'utilisé avec les noms en langue locales.

Je trouve que c'est une incitation à ne pas aller dans le bon sens côté contribution OSM. J'ai regardé les exemples que remonte ta requête Overpass, sur l'Aquitaine et ailleurs, et je trouve qu'il y avait à redire sur àpeu près tous les objets que ça ramène.

Oui, des autres échanges j'ai compris que l'on n'était pas raccord sur l'objectif. Je cherche à géocoder, et donc pouvoir retrouver le max de choses.

frodrigo commented 3 months ago

Pour info, la BAN va intégrer les noms en langue locale. Donc le besoin de supporter de multiples noms va arriver par ce coté également.

vdct commented 2 months ago

Je ferme cette PR sans merger. Comme dit plus haut si des objets n'ont pas de tag name ce n'est pas à BANO d'assumer ce schema de tagging, c'est dans OSM qu'il faut revoir la manière de tagguer.