-
Notifications
You must be signed in to change notification settings - Fork 13
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
creation d'associatedStreet incomplète #113
Comments
Salut @vdct, la relation créée (Rue Cécile Ravallec) reste avec des points d'adresse manquants tels que https://www.openstreetmap.org/node/4604126013. Bon c'est un cas particulier (car créé avant la correction du bogue), je peux corriger à la main mais j'attends ton retour car il me semble intéressant de voir si des points adresses manquent à la relation. |
Il y a 2 raisons cumulées :
|
OK , c'est vrai que j'ai mis l'accent sur la rue, pas sur les points d'adresse, je comptais sur un super-outil pour recoller les morceaux. Devine lequel ;-). Du coup le point 1 est gênant car on ne détecte pas les relations incomplètes. La vérification doit être simple (comparer le nombre de points dans la BAN et le nombre de membres house dans la relation). |
Un intérêt de BANO est de relever les incohérences, typos et autres ;)
Soit tu enregistres le XML dans un fichier, par exemple avec une extension .osm, et JOSM le reconnaitra, soit tu places danston navigateur une URL comme
Rien d'impossible mais ça n'est pas prioritaire ici. Le propos est surtout d'ajouter des adresses, sans la prétention d'homogenéiser le modèle. Sinon à quoi bon proposer le schéma Points et le schéma Relation ? Si on trouve là dessus un consensus alors ok pour faire de l'outil un outil de conversion systématique. Mais on n'y est pas... |
Tu veux dire que le site pourrait prioriser les voies dont le nommage est incomplet ? ;-)
Avec un outil qui permet de créer facilement les relations (et de vérifier leur complétude) et les avantages offerts par les relations, le consensus sera plus simple à trouver. D'ailleurs tu pourrais ne présenter les points que pour les lieux-dits sans rue nommée^^. |
Bonjour, c'est vraiment très pratique. |
@zorglubu; comme tu as mis à jour https://www.openstreetmap.org/relation/7632318#map=18/48.77449/2.32664, @vdct ne pourra plus voir la raison, tu nous en dégotes une autre ou tu regardes de près ce qui clochait (tu es passé de 33 à 39 membres, quels doublons ?). Sinon je ne vois pas l'intérêt de garder |
Bonjour, Je partage totalement la totale inutilité des |
Je viens de réussir à reproduire le problème des doublons sur les 17 points du Sentier des Closeaux (940381470B), à l'Haÿ les Roses toujours. |
Vu, merci pour le nouvel exemple. Je le garde au chaud |
Dans ce cas le problème vient des données. La voie pour laquelle on propose 17 points à intégrer est celle du Fantoir 940381470B, qui est valide. Mais en même temps il existe une relation associatedStreet avec un tag ref:FR:FANTOIR=940381471C qui est un code annulé pour le même nom de voie.
Ca peut se corriger par du code, en vérifiant que soit il n'y a pas de code Fantoir sur la relation, soit c'est le bon. Mais ça ferait passer à côté du bug de données qui consiste en l'utilisation d'un code Fantoir annulé, en lieu et place d'un code valide. Il y a 2 actions à mon avis :
|
Merci beaucoup pour l'explication. J'ai corrigé ce code fantoir. |
Je vois bien l'idée mais il y a trop d'exceptions côté FANTOIR pour pouvoir appliquer un traitement uniforme sur les données France entière en comptant sur le fait qu'il corrige sans abîmer. Mais le sujet reste d'actualité comme je disais juste au dessus. De plus, une nouvelle page existe depuis ce soir, grâce au travail d' @Olyon et qui permet de traiter au cas par cas le sujet des FANTOIRs annulés |
@vdct et @Olyon : La page en question appelle http://bano.openstreetmap.fr/fantoir/fantoir_annule.py qui nous fait un 500 non remonté à l'interface. 2 améliorations : afficher le message d'erreur et éviter qu'il apparaisse^^. |
merci @Bibi56 d'avoir pointé le souci, c'est corrigé maintenant |
Merci d'avoir corrigé aussi vite ! Je pense que l'info est bonne, par contre elle pourrait être mieux valorisée : j'ai pris le premier gros paquet du 29, 29069, j'ai trouvé par l'adresse Fantoir qu'il s'agissait de Guilers et vu le nouvel onglet. Dans OSM et inconnu de FANTOIR20 voies et emprises OSM inconnues de FANTOIR C'est ce que j'allais demander ;-). Sauf que la liste brute donne 103. Est-ce uniquement des codes utilisés dans OSM ? Sinon ils nous intéressent peu. Vu de loin - et comme @zorglubu je peux me tromper - est-ce qu'il n'y a pas trois cas faciles :
Sur https://bano.openstreetmap.fr/fantoir/index.html#insee=29069&tab=6, Allée Jean Henry, ce serait sympa de pouvoir signaler un faux positif : la voirie existe bien (cf. ortho, https://www.openstreetmap.org/edit?editor=id#map=19/48.40696/-4.52796) mais est privée (hors cadastre) d'accès autorisé au public. |
103 : je ne comprends pas à quoi fait référence ce chiffre ?
Le listing recense des objets OSM donc par nature ce qu'on voit ici est utilisé dans OSM. Je n'ai pas compris ce point
Je ne comprends pas de quels pages ou listes tu parles ici.
Ce serait bien en effet mais cette liste de noms d'objets ne stocke pas d'identifiants stables, donc impossible de rappeler d'une fois sur l'autre qu'une ligne est en faux positif. Je pense qu'on a pas mal dérivé par rapport au sujet initial du ticket. Je ferme ici, ne pas hésiter à en ouvrir de nouveaux pour préciser les dernières questions non résolues |
La création d'associatedStreet ajoute bien les points d'adresse manquants mais oublie d'ajouter les PdA existants quand ils comprennent
addr:street
.Exemple :
https://bano.openstreetmap.fr/fantoir/index.html#insee=29150&tab=1
La relation a été créée (https://www.openstreetmap.org/changeset/94600597) par contre les points d'adresse préexistants tels que https://www.openstreetmap.org/node/4565574899 n'ont pas été intégrés.
Les points d'adresse devraient être intégrés et nettoyés (
addr:street
supprimés,addr:city
etaddr:postcode
s'ils sont portés par la relation de la commune).The text was updated successfully, but these errors were encountered: