Améliore la partie WFS et permet l'utilisation en rasters des calques de vulnérabilités/LCZ#613
Améliore la partie WFS et permet l'utilisation en rasters des calques de vulnérabilités/LCZ#613QuentinMadura wants to merge 22 commits intodevfrom
Conversation
Preview Deployment ReadyYour feature branch has been deployed successfully! Preview URL: https://feature-optimize-wfs-carte.iarbre.fr The preview will be automatically deleted when the PR is closed or merged. |
2344947 to
f64ea70
Compare
f64ea70 to
33c7f24
Compare
e3f7993 to
ba7b76b
Compare
| "vulnerability": _entry("rasters/vulnerability.tif"), | ||
| "vulnerability_colors": _entry("rasters/vulnerability_colors.tif"), | ||
| "lcz": _entry("rasters/lcz.tif"), | ||
| "lcz_colors": _entry("rasters/lcz_colors.tif"), |
There was a problem hiding this comment.
Tu les as généré eux et tu les a mis en DB ?
There was a problem hiding this comment.
Si c'est possible sans faire une usine à gaz, je serais plus pour un truc plus général de rasterize_vector.py (on a déjà une commande vectorize_raster.py). Là tu choisis avec des args ce que tu veux rasterize mais en même temps si on rajoute un modèle il faut beaucoup rajouter de code
Ainsi que de mettre les codes couleurs dans un utils/constants.py parce qu'il y a la galère de les avoir aligné avec le front donc autant les avoir à un endroit plus facile à trouver que dans une commande ?
There was a problem hiding this comment.
Ca serait éventuellement aussi à tester comme commande
| "city__code", | ||
| "city__name", |
There was a problem hiding this comment.
Je les avais mis puis enlevé pour faire en sorte que cette requête supplémentaire (certes rapide parce le select_related et qu'il y a une clef étrangère) ne ralentisse pas le flux
There was a problem hiding this comment.
Tu as fait des tests en condition réel de ça ? Je veux dire de comparaison entre le deploy sur feature-... et le même flux sur carte.iarbre.fr ?
Parce que Claude m'avait proposé des trucs qui en réalité ralentissait quand je testais une fois deploy
| add_header Cache-Control "public, max-age=86400" always; | ||
| } | ||
| location = /api/rasters/plantability_colors.tif { | ||
| alias {{ backend_media_path }}/rasters/plantability_colors.tif; |
There was a problem hiding this comment.
Faudra avoir une attention que les noms sont ici en durs et donc qu'il faut les modifier si des choses bougent en DB
| - name: Regenerate calque rasters with platform colors | ||
| shell: | ||
| cmd: "{{ project_slug }}-ctl generate_calque_rasters" | ||
| ignore_errors: true | ||
| when: branch is not defined and (clonecode.changed or force_update is defined) |
There was a problem hiding this comment.
Les données sont amenées à changer une fois par an, donc plutôt pour faire le calcul une fois, mettre en DB et sur S3 plutôt que recalculer à chaque deploy
| } | ||
|
|
||
|
|
||
| # -- Command ----------------------------------------------------------------- |
No description provided.