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

Taginfo : activer chronology (nécessite une mise à jour de l'instance) #251

Closed
vinber opened this issue Dec 6, 2020 · 9 comments
Closed
Assignees

Comments

@vinber
Copy link
Collaborator

vinber commented Dec 6, 2020

l'instance de https://taginfo.openstreetmap.org a une nouvelle fonctionnalité sur la Chronology for intéressante (https://taginfo.openstreetmap.org/keys/emergency#chronology).

Cette possibilité n'existe par dans la version de taginfo http://taginfo.openstreetmap.fr/tags/emergency=defibrillator

Issuer à considérer comme ce qu'elle est :) juste pour ne plus avoir à y penser mais pour le faire, moi je ne sais pas :)

@jocelynj
Copy link
Member

jocelynj commented Dec 6, 2020

Pour activer l'option de chronology, il faut un extract historique de la France, qu'on n'a pas actuellement.

@Marc-marc-marc avait commencé à regarder, mais je ne sais pas où il en est maintenant.

@vinber
Copy link
Collaborator Author

vinber commented Dec 6, 2020

ok merci !

donc :

    • extract historique de la france
    • activer l'option Chronology

@pyrog
Copy link

pyrog commented Dec 7, 2020

@jocelynj et @Marc-marc-marc Si je comprends bien, il faut :

  1. télécharger chez GeoFabrik le fichier France contenant toutes les métadonnées et l'historique complet.
  2. vider complètement la base sql de taginfo ?
  3. Mettre à jour le code de Taginfo
  4. recharger le fichier télécharger précédemment.
  5. modifier la procédure d'import des "minutes" ?

Pour le dernier point, est-il fait automatiquement avec la mise à jour du code de taginfo ?

@frodrigo
Copy link
Member

frodrigo commented Dec 7, 2020

télécharger chez GeoFabrik le fichier France contenant toutes les métadonnées et l'historique complet.

Non ce fichier n'est pas un fichier historique.

@pyrog
Copy link

pyrog commented Dec 7, 2020

Leur site indique que c'est le cas dans la partie privée 😃

Anyway : on trouve un fichier historique ne contenant que la France, c'est bien ça ?

PS: quel est le bon lien si GeoFabrik ne convient pas ?

@frodrigo
Copy link
Member

frodrigo commented Dec 7, 2020

Effectivement, il y a bien l'historique. sur cette version de geofabrik.

@Marc-marc-marc
Copy link

de ce que j'en sais, la maj du code peux se faire indépendamment du reste, le code est capable de traiter un pbf normal tel qu'il existe.

a faire selon moi

  • mettre à jour le code taginfo, attention il y a un PR non finit concernant le fichier de configuration et une modif pour le drapeau dont j'ignore si cela a été remonté sur git
  • test possible de l'option avec le fichier geofabrik, mais celui-ci n'tétait pas maj tous les jours, pour la mise en prod cela n'est pas top
  • tester la maj du fichier historique avec les diff (script 10_update_pbf et 20_filter.sh qui mériteraient d'être déplacé pour rentrer dans git)

@pyrog qu'est-ce qui te fait penser qu'il faut purger qlq chose lors de l'activation de cette option ?

@jocelynj
Copy link
Member

Le code de taginfo est normalement déjà à jour avec le support de pbf historique. Pour le PR dont tu parles, il n'a en fait jamais été mergé dans les scripts ansible, et le fichier de configuration sur taginfo.openstreetmap.fr a largement divergé depuis.

@Marc-marc-marc Marc-marc-marc changed the title Taginfo, faire la mise à jour de l'instance francaise Taginfo : activer chronology (nécessite une mise à jour de l'instance) Mar 7, 2023
@Marc-marc-marc
Copy link

PR osh publié, mergé, tourne depuis "un certain temps" sans soucis et de quasi a à z (reste le téléchargement initial du osh depuis geofabrik osm-fr/ansible-scripts#65)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants