Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Closed
wants to merge 1 commit into from

Conversation

frodrigo
Copy link
Member

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
Copy link
Member

vdct commented Feb 20, 2024

tu as des exemples de ce que ça permet ?

@frodrigo
Copy link
Member Author

frodrigo commented Feb 20, 2024 via email

@frodrigo
Copy link
Member Author

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;

@frodrigo frodrigo marked this pull request as draft February 20, 2024 21:58
@frodrigo frodrigo marked this pull request as ready for review February 20, 2024 22:16
@vdct vdct self-assigned this Feb 22, 2024
@vdct
Copy link
Member

vdct commented Feb 24, 2024

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
Copy link
Member Author

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
Copy link
Member Author

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
Copy link
Member

vdct commented Aug 6, 2024

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.

@vdct vdct closed this Aug 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants