-
Notifications
You must be signed in to change notification settings - Fork 18
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
Chargement en doublon des tags en base de données #397
Comments
Je ne comprends pas ce que tu veux dire là, on a une seule colonne pour le tag name ?
Ni l'une ni l'autre. Autant basculer les requêtes sur les colonnes dédiées alt_name et old_name, c'est plus lisible et c'était l'intention de départ. Mais pas question d'enlever la colonne tags, qui peut servir de fallback à tout moment. C'est de la fausse économie, ça ne gêne en rien (et je pense qu'il y a d'autres priorités)
Est-ce de nature à changer le paradigme actuel ? A savoir un seul nom par adresse et pas de code langue. Aiutant en discuter avant de se lancer dans des devs lourds |
Il y a donc un doublon sur les names, et les autres tags sont non utilisés.
Je ne comprends pas ce que tu veux dire là, on a une seule colonne pour le
tag name ?
Il faut soit supprimé le chargement explicite des noms, soit supprimé le
chargement des tags et utilise des colonnes explicites.
Je peux faire un PR là-dessus. Qu'elle est la solution à préférer ?
Ni l'une ni l'autre. Autant basculer les requêtes sur les colonnes dédiées
alt_name et old_name, c'est plus lisible et c'était l'intention de départ.
Mais pas question d'enlever la colonne tags, qui peut servir de fallback à
tout moment. C'est de la fausse économie, ça ne gêne en rien (et je pense
qu'il y a d'autres priorités)
Ok.
De mon côté en plus de alt_name et old_name j’avance à ajouter le support
des name:fr, name:eu, name:br, name:oc, name:de, name:ca, name:gsw.
Est-ce de nature à changer le paradigme actuel ? A savoir un seul nom par
adresse et pas de code langue. Aiutant en discuter avant de se lancer dans
des devs lourds
En fait déjà fait le dev car j'en ai besoin. J'essaie de te reporté ce qui
peut être utile.
Ça peut fonctionner à deux niveaux. D'abord pour améliorer la mise en
correspondance. Dans le pays basque deux langues sont réellement utilisés
pour les adresses et la mise en correspondance en trouve plus.
Secondement j'ai fait des modif pour sortir dans le json toutes les
variantes de noms d'une même rue pour améliorer le geocodage.
—
… Reply to this email directly, view it on GitHub
<#397 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/AANT5DQYG2S3HVGPK6IFZ33YUZXCRAVCNFSM6AAAAABDS2PJY6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNJYGEYTSNJXGY>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
Dans la config de imposm on a
Dans les requêtes SQL on fait des accès à
name
et àtags->'alt_name'
et tags->'old_name'.Il y a donc un doublon sur les names, et les autres tags sont non utilisés.
Il faut soit supprimé le chargement explicite des noms, soit supprimé le chargement des tags et utilise des colonnes explicites.
Je peux faire un PR là-dessus. Qu'elle est la solution à préférer ?
De mon côté en plus de
alt_name
etold_name
j’avance à ajouter le support desname:fr
,name:eu
,name:br
,name:oc
,name:de
,name:ca
,name:gsw
.The text was updated successfully, but these errors were encountered: