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

[Carte] Améliorer la robustesse de la recherche OpenStreetMap et restreindre sur certains pays #2011

Merged
merged 3 commits into from
Sep 15, 2022

Conversation

mvergez
Copy link
Contributor

@mvergez mvergez commented Sep 15, 2022

Objectif

  • Ajout d'une condition pour récupérer la boundingbox si le geojson n'est pas présent dans la réponse de nominatim OpenStreetMap lors de la recherche sur la carte.
  • Ajout d'un nouveau paramètre dans la configuration pour pouvoir restreindre cette recherche pour certains pays.

Comme le stipule la documentation de nominatim OpenStreeMap, il est possible de restreindre la recherche à une liste de pays.
Le paramètre proposé permet à l'utilisateur de les fournir sous forme d'une chaîne de caractères respectant le format suivant :

[MAPCONFIG]
OSM_RESTRICT_COUNTRY_CODES = "fr,ch,be,sp"

Les "pays" à fournir sont en réalité leur code ISO_3166-1_alpha-2. Un exemple est donné dans le fichier default_config.toml.example.

Pour désactiver cette restriction, il suffit de retirer ce paramètre ou de le définir comme null.

En espérant que cela vous convienne

Closes #2010

And fix the case when nominatim response has no geojson
@camillemonchicourt
Copy link
Member

Merci, ça mérite quelques précisions, car je ne suis pas certain que la plupart des utilisateurs savent que l'outil de localisation se base sur OSM, donc préciser où ce truc est utilisé, je pense.

@mvergez
Copy link
Contributor Author

mvergez commented Sep 15, 2022

Ok merci @camillemonchicourt, je rajoute ça dans les commentaires du code et du toml d'exemple

@codecov
Copy link

codecov bot commented Sep 15, 2022

Codecov Report

Base: 63.13% // Head: 63.14% // Increases project coverage by +0.00% 🎉

Coverage data is based on head (885b610) compared to base (2b28ac5).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@           Coverage Diff            @@
##           develop    #2011   +/-   ##
========================================
  Coverage    63.13%   63.14%           
========================================
  Files           79       79           
  Lines         7319     7320    +1     
========================================
+ Hits          4621     4622    +1     
  Misses        2698     2698           
Flag Coverage Δ
pytest 63.14% <100.00%> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
backend/geonature/utils/config_schema.py 91.91% <100.00%> (+0.03%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@mvergez
Copy link
Contributor Author

mvergez commented Sep 15, 2022

J'ai ajouté ce commentaire dans le code et dans le toml :

# Restreindre la recherche OpenStreetMap (sur la carte dans l'encart "Rechercher un lieu")
# à certains pays. Les pays doivent être au format ISO_3166-1 : 
# https://en.wikipedia.org/wiki/ISO_3166-1_alpha-2 et séparés par une virgule.
# Exemple : OSM_RESTRICT_COUNTRY_CODES = "fr,es,be,ch" (Restreint à France, Espagne, Belgique 
# et Suisse)
# Laisser à null pour n'avoir aucune restriction

@bouttier bouttier merged commit 2f8299a into PnX-SI:develop Sep 15, 2022
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

Successfully merging this pull request may close these issues.

4 participants