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

ajout du clustering #5

Merged
merged 3 commits into from May 26, 2016

Conversation

@haidaraM
Copy link
Contributor

haidaraM commented May 24, 2016

J'ai ajouté le clustering sur les markers et commenté le marker de l'INSA en attendant car il masque les clusters formés.

Dites-moi ce que vous en pensez!

@LoicTouzard

This comment has been minimized.

Copy link
Owner

LoicTouzard commented May 24, 2016

L'idée est intéressante et aurait été développée un jour ou l'autre.

Personnellement je suis pour, d'autant plus que le clustering ajoute des fonctionnalités supplémentaire sympa pour la navigation sur la carte. Le centrage de la vue au click, ainsi que le zoom au bon niveau.

@LoicTouzard

This comment has been minimized.

Copy link
Owner

LoicTouzard commented May 24, 2016

Le chiffre dans le cluster représente le nombre d’éléments sous-jacents. Est ce que ce chiffre est celui que l'ont veut voir apparaître ?
Il peut être considéré comme **le nombre d'endroit différent où se trouve des IF"
Est ce que avoir un chiffre représentant *
*le nombre d'IF présent dans la zone du cluster" est également intéressante ?
Qu'en pensez vous @pdautry @niosega ?

@haidaraM

This comment has been minimized.

Copy link
Contributor Author

haidaraM commented May 25, 2016

Dans l'etat actuel le chiffre représente le nombre de markers et non le
nombre d'IFs. Mais il peut être modifié assez facilement et customiser. Je
peux me pencher dessus et le modifier si besoin.
Le 25 mai 2016 01:37, "Loïc Touzard" notifications@github.com a écrit :

Le chiffre dans le cluster représente le nombre d’éléments sous-jacents.
Est ce que ce chiffre est celui que l'ont veut voir apparaître ?
Il peut être considéré comme
**le nombre d'endroit différent où se trouve des IF" Est ce que avoir un
chiffre représentant *
*le nombre d'IF présent dans la zone du cluster"
est également intéressante ?
Qu'en pensez vous @pdautry https://github.com/pdautry @niosega
https://github.com/niosega ?


You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
#5 (comment)

@koromodako

This comment has been minimized.

Copy link
Collaborator

koromodako commented May 25, 2016

Serait-il possible de faire la modification du chiffre affiché pour afficher le nombre de personnes dans le cluster plutôt que le nombre de villes ? Nous effectuerons le merge et le déploiement avec d'autres changements par la suite.

@haidaraM

This comment has been minimized.

Copy link
Contributor Author

haidaraM commented May 25, 2016

Je vais voir ça...

@LoicTouzard

This comment has been minimized.

Copy link
Owner

LoicTouzard commented May 25, 2016

Probablement en passant par là :
customising-the-clustered-markers puis en utilisant getAllChildMarkers() et en ayant extend les markers contenant le nombre de personne à cette localisation. Ou tout simplement une propriété custom sur les markers comme dans l'exemple de la doc.

@haidaraM

This comment has been minimized.

Copy link
Contributor Author

haidaraM commented May 25, 2016

vous pouvez check ?

@LoicTouzard

This comment has been minimized.

Copy link
Owner

LoicTouzard commented May 26, 2016

C'est vérifié, et c'est ok !
Il reste juste un détail quand au choix des paliers pour les couleurs :

var className = ' marker-cluster-';
if (nbIfs < 10) {
    className += 'small';
} else if (nbIfs < 100) {
    className += 'medium';
} else {
    className += 'large';
}

La valeur 10 me paraît cohérente contrairement au 100 qu'il faudrait lui changer.
Pour le palier 'large' je verrais plus une valeur comme 20 qui est plus cohérente pour notre nombre d'utilisateurs actuels.

Nous changerons ce palier plus tard soit en l'augmentant, soit en le calculant sur un pourcentage du nombre d'utilisateurs.

@haidaraM Pourrais-tu juste changer le 100 en 20 ?

@haidaraM

This comment has been minimized.

Copy link
Contributor Author

haidaraM commented May 26, 2016

Ok

@LoicTouzard LoicTouzard merged commit a7f3f8c into LoicTouzard:master May 26, 2016
@haidaraM haidaraM deleted the haidaraM:clustering branch May 26, 2016
LoicTouzard added a commit that referenced this pull request May 27, 2016
…opos" modal. Updated format for #11 as told in ee18a59.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.