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

[question] v3 - alertes osmose : faire/demander modif. FranceSpecificRules ? #303

Open
deuzeffe opened this issue Jul 31, 2023 · 17 comments

Comments

@deuzeffe
Copy link
Contributor

Hello,

Puisque le code TOPO contient un caractère de moins que le code FANTOIR, tous les codes rentrés à partir de la v3 provoquent des alertes osmose (qu'il faut marquer en faux positif une par une :/)

Est-ce qu'il est possible que https://josm.openstreetmap.de/wiki/Rules/FranceSpecificRules, qu'osmose intègre dans ses analyses, soit facilement modifié dès à présent pour accepter les deux formats (qui vont coexister "un certain temps") ou c'est prématuré ?

@vdct
Copy link
Member

vdct commented Aug 1, 2023

C'est même carrément souhaitable oui. Tu saurais t'en charger ?

@deuzeffe
Copy link
Contributor Author

deuzeffe commented Aug 1, 2023

Houlà, je ne peux me charger que d’ouvrir un ticket sur josm, vu que ce truc est bourré d'expressions rationnelles que je comprends encore moins que le langage overpass...

EDIT : done (https://josm.openstreetmap.de/ticket/23094#ticket)

@Olyon
Copy link
Contributor

Olyon commented Aug 4, 2023

Il faudrait modifier les pages du wiki qui parlent de ref:FR:FANTOIR pour dire que la lettre de controle est optionnelle.
Il y a au moins la page https://wiki.openstreetmap.org/wiki/FR:Key:ref:FR:FANTOIR à modifier.

@deuzeffe
Copy link
Contributor Author

deuzeffe commented Aug 5, 2023

@Olyon Just do it? ;)

@deuzeffe
Copy link
Contributor Author

deuzeffe commented Sep 2, 2023

Pas l'impression que @PanierAvide a vu le ticket https://josm.openstreetmap.de/ticket/23094#ticket sur JOSM :(

@PanierAvide
Copy link

Pas vu passer le ticket, mais j'ai pas l'impression d'être à l'origine de la règle sur le FANTOIR, c'était plutôt les 4 premières règles du fichier... Il faut juste mettre en optionnel le 10e caractère du code c'est bien ça ?

@deuzeffe
Copy link
Contributor Author

deuzeffe commented Sep 3, 2023

@PanierAvide oui, c'est ça. Me semble que @Olyon a répondu au ticket dans ce sens.

@deuzeffe
Copy link
Contributor Author

@Olyon tu te sens d'intégrer la modif. que tu as proposée sur le code écrit par @didier2020 (si je lis bien l'historique du fichier) ?

@didier2020
Copy link

didier2020 commented Sep 20, 2023

Bonjour,
si lettre de controle est optionnelle, il 'suffit" de modifier
([ABCDEFGHJKLMNPRSTUVWXYZ])
par
(|[ABCDEFGHJKLMNPRSTUVWXYZ])
dans le fichier des regles josm
https://josm.openstreetmap.de/wiki/Rules/FranceSpecificRules
(rien | [liste des lettres autorisées])

et mettre a jour la premiere ligne du wiki
"Le code FANTOIR a indiquer comme valeur doit faire 10 caractères..."

@deuzeffe
Copy link
Contributor Author

@didier2020 je n'ai aucune connaissance pour savoir si ta solution est meilleure que celle d'@Olyon. je laisse qui sait, faire la modif. du fichier.

@frodrigo
Copy link
Member

@didier2020 je n'ai aucune connaissance pour savoir si ta solution est meilleure que celle d'@Olyon. je laisse qui sait, faire la modif. du fichier.

Les deux, et c'est déjà fait coté JOSM.

@deuzeffe
Copy link
Contributor Author

@frodrigo vu, merci pour l'info et merci à @Olyon pour la modif.
@vdct on y est enfin arrivé.

@gosgtan
Copy link

gosgtan commented Feb 11, 2024

Bonjour,

Depuis aujourd'hui les FANTOIR à 10 caractères renvoient une alerte sur Osmose... Il aurait plutôt fallu pouvoir faire cohabiter ceux à 9 et 10 caractères, non? Tout modifier pour enlever le dernier caractère ça va être chronophage

@deuzeffe deuzeffe reopened this Feb 11, 2024
@deuzeffe
Copy link
Contributor Author

@gosgtan Modification d'aujourd'hui par @didier2020 (merci)
https://josm.openstreetmap.de/wiki/Rules/FranceSpecificRules?action=diff&version=92
Ça devrait encore s'améliorer.

@frodrigo
Copy link
Member

Le changement n'a été déployé que hier soir, mais avec un bug. Correction et redéploiement aujourd’hui. Ça devrait être bon demain.

@deuzeffe
Copy link
Contributor Author

@frodrigo merci pour les détails !

@gosgtan
Copy link

gosgtan commented Feb 11, 2024

Impeccable merci !

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

No branches or pull requests

7 participants