organicmaps/organicmaps

Handle implied access

Ouverte

#6 089 ouverte le 22 sept. 2023

Voir sur GitHubGuide contributeur
 (1 commentaire) (0 réaction) (0 personne assignée)C++ (761 forks)batch import
GeneratorGood first issue

Métriques du dépôt

Stars
 (7 887 étoiles)
Métriques de merge PR
 (Merge moyen 13j 6h) (91 PRs mergées en 30 j)

Description

In the case of service=emergency_access, it's implied that access=emergency, or at least vehicle=emergency.

I think the best way would be to add a new flag to replaced_tags.txt that only adds a tag if it doesn't already exist. Then we can do something like service=emergency_access : access=emergency | n

Should fix cases like https://github.com/organicmaps/organicmaps/issues/5983, where access is implied but not explicitly tagged

image

there are 26k ways worldwide tagged service=emergency_access, but without an access=* or vehicle=* so it'd be good to fix this. (to be fair, some of these may have gates with explicit access tagging)

Guide contributeur

Direction de recherche
Examinez la structure du fichier replaced tags.txt et implémentez un nouveau drapeau qui ajoute une balise seulement si elle n'existe pas déjà, puis testez avec service=emergency access.
Stack technique
cpp
Domaine
backend
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Git
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
65