search

GeotrekCE / Geotrek-admin

Paths management for National Parks and Tourism organizations
https://geotrek.fr
BSD 2-Clause "Simplified" License
130 stars 76 forks source link

Echelle de la carte sur les exports fiches itinéraires #1182

Closed fmerzeau closed 9 years ago

fmerzeau commented 9 years ago

Je viens de faire la màj vers la v0.26.2. Je constate que l'échelle a été réduite, ce qui nuit à la lisibilité de l'itinéraire. Est-ce possible de gérer ce paramètre ? Exemple sur 0.25.1 trek-897 v 0 25 1

Exemple sur 0.26.2 trek-897 v 0 26 2

leplatrem commented 9 years ago

J'ai parcouru tous les changements entre la 0.25.1 et la 0.26.2 : https://github.com/makinacorpus/Geotrek/compare/v0.25.1...v0.26.2

Je vois qu'effectivement, je suis passé d'une taille de la carte en dur dans le code (13, 11) à un paramètre TREK_EXPORT_MAP_IMAGE_SIZE = (14.1, 11). J'avais mis 14.1 pour que la carte occupe bien toute la largeur de la demi-page...

Pouvez-vous essayer de mettre ce paramètre à (13, 11) dans vos settings pour voir ce que ça donne ?

Merci !

camillemonchicourt commented 9 years ago

J'ai fait différents test à 13 et 14.1 et parfois le zoom est plus serré avec le 13 parfois il est plus serré avec le 14.1. Je ne me l'explique pas.

leplatrem commented 9 years ago

Est-ce que le zoom attendu va au delà de 18 ?

camillemonchicourt commented 9 years ago

J'ai du mal à voir à quoi le ZOOM 18 correspond mais je ne pense vraiment pas. Je pense que ça se joue autour des ZOOM 13 et 14.

camillemonchicourt commented 9 years ago

Je ne sais pas vraiment comment cela fonctionne mais si c'est un zoom fixe, je crains qu'on arrive pas à bien coller tout le temps. Certains sont très petits (genre sentiers d'interprétations), d'autres sont très long (par exemple tour itinérants sur plusieurs jours).

Pourquoi ne pas calculer le zoom en fonction de l'itinéraire. En prenant l'étendue de l'itinéraire et en zoomant juste un peu plus large. Pas possible ?

leplatrem commented 9 years ago

Ce n'est évidemment pas un zoom fixe :) Par contre limité à 18 par défaut.

Voir:

https://github.com/makinacorpus/django-mapentity/blob/1.10.1/mapentity/static/mapentity/mapentity.map.js#L126-L135

https://github.com/makinacorpus/django-mapentity/blob/1.10.1/mapentity/templates/mapentity/mapgeometry_fragment.html#L5

camillemonchicourt commented 9 years ago

OK je me disais aussi, alors le problème vient peut être de la limite au zoom 18, je n'en sais rien. :-)

leplatrem commented 9 years ago

Pourrais-tu essayer en ajouter ceci dans custom.py stp ?:

MAPENTITY_CONFIG['MAP_CAPTURE_MAX_RATIO'] = 0

Cela va désactiver la contrainte sur la taille de l'image finale. Voyons si ça vient de ça... Même si cela ne semble pas avoir été modifié en 0.26 :(

leplatrem commented 9 years ago

Le plus représentatif de ce problème sont ces deux itinéraires:

Leur géométrie est très proche, et le ratio calculé est le même....

$ bin/django shell
>>> from geotrek.trekking.models import Trek
>>> t = Trek.objects.get(id=920733)
>>> t.get_geom_aspect_ratio()
1.2818181818181817
>>> t = Trek.objects.get(id=904199)
>>> t.get_geom_aspect_ratio()
1.2818181818181817

selection_503 selection_501

leplatrem commented 9 years ago

I could not reproduce the above situation with local data:

trek-10029 trek-10027

camillemonchicourt commented 9 years ago

I removed every maps exports from our Geotrek-admin in /home/sentiers/Geotrek/var/media/maps and relaunched a MAKE SYNC. Here it is updated after regenerating every Maps and PDF : http://rando.ecrins-parcnational.fr

SAUT DU LAIRE export is now OK : http://rando.ecrins-parcnational.fr/fr/le-saut-du-laire http://rando.ecrins-parcnational.fr/fr/files/api/trek/trek-904199.pdf And same for SAUT DU LAIRE PAR LE BASSET : http://rando.ecrins-parcnational.fr/fr/le-saut-du-laire-par-le-basset http://rando.ecrins-parcnational.fr/fr/files/api/trek/trek-920733.pdf !!!!

But this one is cut : http://rando.ecrins-parcnational.fr/fr/files/api/trek/trek-918678.pdf Same for http://rando.ecrins-parcnational.fr/fr/files/api/trek/trek-921164.pdf This one is too small : http://rando.ecrins-parcnational.fr/fr/files/api/trek/trek-903295.pdf This one is a bit small too : http://rando.ecrins-parcnational.fr/fr/files/api/trek/trek-903303.pdf Same for : http://rando.ecrins-parcnational.fr/fr/files/api/trek/trek-903311.pdf and http://rando.ecrins-parcnational.fr/fr/files/api/trek/trek-904063.pdf

Most of others are OK

gutard commented 9 years ago

Pour le saut du Laire sur la démo le résultat était différent car le départ et le parking n'étaient pas placés pareils. Après correction on a bien le même niveau de zoom. Sur la prod le départ/parking ne sont pas placés exactement comme sur la démo.

Donc les différence entre les deux randos et entre la démo et la prod s'expliquent. Par contre, ça n'explique pas les zooms trop petits ou trop grands.

camillemonchicourt commented 9 years ago

Ah OK ! Merci.

gutard commented 9 years ago

Fixed in 0.28.6

camillemonchicourt commented 9 years ago

Confirmed that MAPS are OK after installing 0.29.0 ! Thanks.

leplatrem commented 9 years ago

\o/ GG !