search

LittoSim / LittoSim_model

Participatory simulation model of coastal flooding management
http://littosim.github.io/LittoSim_model/
7 stars 3 forks source link

Un jour ça serait rigolo d'ajouter un paramètre pour le nombre de commune (dans un fichier commenté de configuration) #20

Closed gnoubi closed 4 years ago

gnoubi commented 4 years ago

Changer le paramètre nombre de communes qui en dure dans le code afin que ca devienne dynamique

laatabix commented 4 years ago

Le nombre de communes jouées peut être 2, 3 ou 4, mais pas plus (pour le moment). Les grilles de l'interface Leader doivent être initialisées en avance, du coup elles sont toujours à 4 colonnes. Le nombre de communes actives est extrait directement de la table _MAP_DISTSNAMES de study_area.conf. Seules les communes qu'on veut jouer doivent figurer dans cette table.

https://github.com/LittoSim/LittoSim_model/commit/17dae83aed705e47992caa140fcb3276b9cb4944

nbecu commented 4 years ago

En effet, coté Leader et Player, le fait de modifier le paramètre MAP_DIST_SNAMES permet de réduire le nombre de joueur (par ex. MAP_DIST_SNAMES;["17385"::"com-A","17140"::"Tart"] ) image

Par contre , coté manager, ça continue d'afficher la carte du landuse avec les 4 communes, Du coup je suis pas tout à fait d'accord pour dire que cette issue est closed pour le cas "moins de 4 joueurs"

image

laatabix commented 4 years ago

Ouups, oui tu as raison !

On Wed, Jun 3, 2020 at 5:57 PM nbecu notifications@github.com wrote:

En effet, coté Leader et Player, le fait de modifier le paramètre MAP_DIST_SNAMES permet de réduire le nombre de joueur [image: image] https://user-images.githubusercontent.com/7630183/83659195-8235ef80-a5c3-11ea-92ed-7bf455b0a6ff.png

Par contre , coté manager, ça continue d'afficher la carte du landuse avec les 4 communes, Du coup je suis pas tout à fait d'accord que cette issue est closed pour le cas "moins de 4 joueurs"

Par contre

[image: image] https://user-images.githubusercontent.com/7630183/83658942-2f5c3800-a5c3-11ea-8ca7-1dc0a8fb264c.png

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub https://github.com/LittoSim/LittoSim_model/issues/20#issuecomment-638289386, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADVLFFZOFX76ETP3B54SXZDRUZXFPANCNFSM4NM4P7TA .

-- Ahmed Laatabi LIENSs - Université de La Rochelle +(33) 7 70 24 75 70

laatabix commented 4 years ago

Fixed #https://github.com/LittoSim/LittoSim_model/commit/4a8497531422362b3d1ad8e009abb079668d81ac