Validation page 1

[ColinFay]

Validate

Validation des modifications de la page 1.
(Vous pouvez cocher les cases et mettre un commentaire quand c'est bon)

https://connect.thinkr.fr/diades-main/

Choose a species

• La liste des espèces est correcte dans le dropdown #41

• Sélectionner une espèce change le "conservation status" et la carte

Conservation Status

• Le "Conservation Status" se met à jour lorsque je change d'espèce

• Le "Conservation Status" est correct en fonction de l'espèce

Source

• Le bouton a été retiré

Division / Rectangle

• Un sélecteur permet de naviguer entre Division et rectangle

• Lorsque je change ce sélecteur, la carte se met à jour

Carte

• La carte s'affiche

• La carte est correcte

• La carte se met à jour lorsque je change d'espèce ou de géométrie

• La carte est mise en cache : si je change entre division et rectangle une première fois, cela prend quelques secondes, si je refais le changement, c'est instantané.

TODO:

Aujourd'hui, à l'ouverture, nous simplifions grandement la géométrie pour que le temps d'affichage de la carte soit raisonnable.
Nous utilisons rmapshaper::ms_simplify() avec le réduction par défaut : On ne conserve que 5% des points.
Tu verras que les contours sont encore acceptables, mais peut-être souhaiteriez-vous une réduction moindre.

Si tu as accès à la base, ou à tes données directement, tu peux tester plusieurs valeurs de keep et observer avec {mapview} le résultat. Avec le script suivant si connecté à la base, sinon directement sur ta couche de polygones.

catchment_geom <- sf::st_read(
 	con, 
	query =   "SELECT * FROM diadesatlas.v_basin vb"
) %>%
	rmapshaper::ms_simplify(keep = 0.05)

mapview::mapview(catchment_geom)

Une autre manière de bien se rendre compte du temps d'affichage c'est de transformer ta carte tmap en leaflet

mon_tmap <- tm_shape(ma_couche) + tm_polygons()

# transformation en leaflet comme dans Shiny
tmap::tmap_leaflet(mon_tmap)

Sans rmapshaper::ms_simplify(), les cartes sont très (trop?) longue à afficher (jusqu'à 1min30 dans certains cas).
Blocage : rmapshaper::ms_simplify() prend plusieurs secondes et pourrait être fait en amont.

• Décider du pourcentage de réduction de la carte (Sur R, keep correspond au pourcentage de points conservées)
• Faire la mise à jour dans la base SQL avec des geométries simplifiées pour les tables avec des géométries.

Tout est ok ?

• Je ferme cette issue

[PMHLambert]

1. )la liste des espèces est correcte mais pourrait être réorganisée comme suit (les aloses, les lamproies, les salmonidés, l'esturgeon et l'éperlan pour les anadromes, l'anguille, le mulet et le flet pour les catadromes)
   c('Alosa alosa', 'Alosa fallax','Petromyzon marinus', 'Lampetra fluviatilis', 'Salmo salar', 'Salmo trutta', 'Acipenser sturio', 'Osmerus eperlanus', 'Anguilla anguilla', 'Chelon ramada', 'Platichthys flesus' )
2. déplacer le statut des espèces dans la partie inférieure du panel , prés de la carte : ce n'est pas une sélection à proprement parler.
3. Je ne suis pas sûr que la carte soit mise en cache, je ne vois pas de différence dans le délai d'affichage quad il s'agit d'un ré-affichage.
4. ok pour simplifier les cartes. Le pb c'est que l'exutoire à la mer ne se trouve plus sur le trait de cote, ce qui est déstabilisant quand (et uniquement quand ) on zoome fortement. Je n'ai pas de solution pour régler ce pb. Pour rester cohérent avec la simplification déjà effectuée, je préfère demander à Eric Quinton d'ajouter une nouvelle géométrie encore plus légère (il m'avait fait une proposition que j'avais écartée pour les pb d'exutoire)

[ColinFay]

TODO @statnmap

Correction après retours de Patrick, à valider

• 1, liste des espèces dans le bon ordre, réglé dans reorder species #45
• 2, déplacement status à côté de la carte, réglé dans Conservation Status is now shown next to the map #43
• 3, mettre en place le cache, réglé dans Correct issue with cache in renderTmap #46

[ColinFay]

@PMHLambert à valider sur https://connect.thinkr.fr/diades-main/ => "La carte est mise en cache"

[statnmap]

Page validée en réunion 2021-11-26.
Une issue ouverte : #67