delobell commented

(Migrated from git.eleves.ens.fr)

K-Fêt

fix graph size of stats
for real, users can only see/get their own account stats
article stats can only be accessed by teamkfet

K-Fêt - Stats

clean py code
2 urls by kind of stats:
- kfet.<...>.stat.<type>.list : manifest of proposed stats to user
- kfet.<...>.stat.<type> : return stat datasets according to GET params (scale, begin, end, ...)

Scale

base class which implements logic for a scale
create a new scale, given params as begin, end, last...
iterate over it to get begin, end intervals (chunks)
give chunks labels

DayScale, WeekScale, MonthScale

use Scale
their chunks length is 1 day, 1 week, 1 month
take into account that a k-fet day begins at KFET_WAKES_UP_AT

K-Fêt - Misc

fix GET params in ajax call of statistic.js
delete unused template account_read_title.html
delete unused imports and defs in views.py
PkUrlMixin: use with SingleObject... class-based view, it gets object by filtering model objects on pk_url_kwarg

# K-Fêt - fix graph size of stats - for real, users can only see/get their own account stats - article stats can only be accessed by `teamkfet` ## K-Fêt - Stats - clean py code - 2 urls by kind of stats: - `kfet.<...>.stat.<type>.list` : manifest of proposed stats to user - `kfet.<...>.stat.<type>` : return stat datasets according to GET params (scale, begin, end, ...) ## Scale - base class which implements logic for a scale - create a new scale, given params as `begin`, `end`, `last`... - iterate over it to get `begin, end` intervals (chunks) - give chunks labels ### DayScale, WeekScale, MonthScale - use Scale - their chunks length is 1 day, 1 week, 1 month - take into account that a k-fet day begins at `KFET_WAKES_UP_AT` ## K-Fêt - Misc - fix GET params in ajax call of `statistic.js` - delete unused template `account_read_title.html` - delete unused imports and defs in `views.py` - `PkUrlMixin`: use with `SingleObject...` class-based view, it gets object by filtering model objects on `pk_url_kwarg`

delobell commented

2017-04-03 13:27:08 +02:00

(Migrated from git.eleves.ens.fr)

added 1 commit

b3a9ad8a - clean mixin

Compare with previous version

added 1 commit * b3a9ad8a - clean mixin [Compare with previous version](https://git.eleves.ens.fr/cof-geek/gestioCOF/merge_requests/208/diffs?diff_id=330&start_sha=d97a7be8196800562b3c4a4bbd7e3e7ff8eb01b5)

lstephan commented

2017-04-03 14:53:15 +02:00

(Migrated from git.eleves.ens.fr)

Est ce que c'est vraiment utile de le définir comme ça si tu override son fonctionnement dans la plupart des classes ? Pour moi, définir directement get_chunk_start est plus lisible et intuitif que définir offset_to_chunk_start.

Est ce que c'est vraiment utile de le définir comme ça si tu override son fonctionnement dans la plupart des classes ? Pour moi, définir directement `get_chunk_start` est plus lisible et intuitif que définir `offset_to_chunk_start`.

delobell commented

2017-04-03 15:11:24 +02:00

(Migrated from git.eleves.ens.fr)

added 1 commit

1bb83ccd - simplify StatScale

Compare with previous version

added 1 commit * 1bb83ccd - simplify StatScale [Compare with previous version](https://git.eleves.ens.fr/cof-geek/gestioCOF/merge_requests/208/diffs?diff_id=331&start_sha=b3a9ad8a964dc3128b404316eb32e92698bc526a)

lstephan commented

2017-04-03 15:36:01 +02:00

(Migrated from git.eleves.ens.fr)

Un petit oubli =)

lstephan commented

2017-04-03 15:36:36 +02:00

(Migrated from git.eleves.ens.fr)

Du coup avec ça on perd la responsivité de la taille ou non ?

lstephan commented

2017-04-03 15:38:14 +02:00

(Migrated from git.eleves.ens.fr)

Ça ne vient pas de toi mais les commentaires sont un peu déroutants ^^

delobell commented

2017-04-03 15:39:35 +02:00

(Migrated from git.eleves.ens.fr)

Je vais corriger ça

delobell commented

2017-04-03 15:42:42 +02:00

(Migrated from git.eleves.ens.fr)

Je viens de voir que ça pose problème d'enlever ça (enfin son équivalent avec begin et end) quand il n'y a aucune opération sur le compte

lstephan commented

2017-04-03 15:48:44 +02:00

(Migrated from git.eleves.ens.fr)

Actuellement, quand un compte est créé dans la dernière année, tout est tronqué à sa date de création ; est ce que c'est volontaire, et si oui pourquoi ne pas mettre des zéros avant ?

delobell commented

2017-04-03 16:09:52 +02:00

(Migrated from git.eleves.ens.fr)

added 2 commits

769c3763 - delete debug msg
87bc90ec - Begin/end balance stat graph

Compare with previous version

added 2 commits * 769c3763 - delete debug msg * 87bc90ec - Begin/end balance stat graph [Compare with previous version](https://git.eleves.ens.fr/cof-geek/gestioCOF/merge_requests/208/diffs?diff_id=332&start_sha=1bb83ccdd70338455da0eecb269cfcda14275786)

delobell commented

2017-04-03 16:11:47 +02:00

(Migrated from git.eleves.ens.fr)

@lstephan
Tout le temps prend en compte la date de création du compte mais pas les autres. Ça m'a l'air déjà plus legit comme ça

@lstephan `Tout le temps` prend en compte la date de création du compte mais pas les autres. Ça m'a l'air déjà plus legit comme ça

delobell commented

2017-04-03 16:13:27 +02:00

(Migrated from git.eleves.ens.fr)

Ça garde le ratio width/height pour le moment, donc sur une tablette (où tout n'est pas encore sur une colonne), ça fait effectivement petit.
Je pars retrouver l'option de chart.js pour contrôler ça...

Ça garde le ratio width/height pour le moment, donc sur une tablette (où tout n'est pas encore sur une colonne), ça fait effectivement petit. Je pars retrouver l'option de chart.js pour contrôler ça...

lstephan commented

2017-04-03 16:41:14 +02:00

(Migrated from git.eleves.ens.fr)

Ok c'est juste que j'avais des opérations qui dataient d'avant la création du compte, du coup ça faisait des trucs étranges.

delobell commented

2017-04-03 16:55:25 +02:00

(Migrated from git.eleves.ens.fr)

added 1 commit

903da795 - clean kfet statistic.js

Compare with previous version

added 1 commit * 903da795 - clean kfet statistic.js [Compare with previous version](https://git.eleves.ens.fr/cof-geek/gestioCOF/merge_requests/208/diffs?diff_id=334&start_sha=87bc90ec8bd25ce0ce4a4a0415b2dd44ebdf6181)

lstephan commented

2017-04-03 17:04:29 +02:00

(Migrated from git.eleves.ens.fr)

Ça sert dans les stats ou c'est juste par principe ?

lstephan commented

2017-04-03 17:07:18 +02:00

(Migrated from git.eleves.ens.fr)

Ok apparemment c'est censé être dans les labels, mais je les ai pas...

delobell commented

2017-04-03 17:07:33 +02:00

(Migrated from git.eleves.ens.fr)

added 2 commits

6d2e150a - clean align
10d2b58f - clean some comments

Compare with previous version

added 2 commits * 6d2e150a - clean align * 10d2b58f - clean some comments [Compare with previous version](https://git.eleves.ens.fr/cof-geek/gestioCOF/merge_requests/208/diffs?diff_id=335&start_sha=903da795ab3f7a5399a33cf221ea70d8b19880b9)

delobell commented

2017-04-03 17:11:14 +02:00

(Migrated from git.eleves.ens.fr)

Bonne question!
Oui c'était utilisé dans les labels d'opérations. Sauf que en même-temps ces labels ne sont pas utilisés pour l'affichage du graph...
Donc je l'ai mis là parce que ça peut toujours être utile (et que c'est utilisé dans une vue de stats).

Est-ce que @narmanli pourrait passer par là pour nous expliquer quel était le fond de sa pensée avec ces labels ?

Bonne question! Oui c'était utilisé dans les labels d'opérations. Sauf que en même-temps ces labels ne sont pas utilisés pour l'affichage du graph... Donc je l'ai mis là parce que ça peut toujours être utile (et que c'est utilisé dans une vue de stats). Est-ce que @narmanli pourrait passer par là pour nous expliquer quel était le fond de sa pensée avec ces labels ?

delobell commented

2017-04-03 17:12:09 +02:00

(Migrated from git.eleves.ens.fr)

C'est bon pour toi avec les derniers commits ?

delobell commented

2017-04-04 01:29:28 +02:00

(Migrated from git.eleves.ens.fr)

added 1 commit

df467767 - fix default GET param 'types' on operations stats

Compare with previous version

added 1 commit * df467767 - fix default GET param 'types' on operations stats [Compare with previous version](https://git.eleves.ens.fr/cof-geek/gestioCOF/merge_requests/208/diffs?diff_id=346&start_sha=10d2b58fa7e01fa1ddbecc969b1aa1c460f8a3af)

delobell commented

2017-04-04 01:36:32 +02:00

(Migrated from git.eleves.ens.fr)

added 1 commit

7989a07b - cleaner fix

Compare with previous version

added 1 commit * 7989a07b - cleaner fix [Compare with previous version](https://git.eleves.ens.fr/cof-geek/gestioCOF/merge_requests/208/diffs?diff_id=347&start_sha=df467767f441f55b193ac9a146b97660bf55db55)

mpepin commented

2017-04-04 10:39:41 +02:00

(Migrated from git.eleves.ens.fr)

added 2 commits

f13d1072 - Add simple tests for the stat views
b383923d - Merge branch 'Kerl/test_stats' into aureplop/fix_stats

Compare with previous version

added 2 commits * f13d1072 - Add simple tests for the stat views * b383923d - Merge branch 'Kerl/test_stats' into aureplop/fix_stats [Compare with previous version](https://git.eleves.ens.fr/cof-geek/gestioCOF/merge_requests/208/diffs?diff_id=350&start_sha=7989a07b5f61d7f4acb5165811b5c4765e43934f)

delobell commented

2017-04-04 11:06:00 +02:00

(Migrated from git.eleves.ens.fr)

added 1 commit

278459e8 - typo.......

Compare with previous version

added 1 commit * 278459e8 - typo....... [Compare with previous version](https://git.eleves.ens.fr/cof-geek/gestioCOF/merge_requests/208/diffs?diff_id=351&start_sha=b383923d49021395427b98697ebc8e7e428ce8fb)

delobell commented

2017-04-04 20:18:15 +02:00

(Migrated from git.eleves.ens.fr)

added 2 commits

885e40fd - cleaner scales
dc07b072 - Flatten scale args in GET params of stats urls

Compare with previous version

added 2 commits * 885e40fd - cleaner scales * dc07b072 - Flatten scale args in GET params of stats urls [Compare with previous version](https://git.eleves.ens.fr/cof-geek/gestioCOF/merge_requests/208/diffs?diff_id=355&start_sha=278459e80f2b478a958cdebe263936c466332649)

delobell commented

2017-04-04 20:35:18 +02:00

(Migrated from git.eleves.ens.fr)

changed title from {-Aureplop/fix stats-} to {+Stats K-Fêt - Python side+}

changed title from **{-Aureplop/fix stats-}** to **{+Stats K-Fêt - Python side+}**

mpepin commented

2017-04-05 00:31:55 +02:00

(Migrated from git.eleves.ens.fr)

added 1 commit

85ba44c2 - Tests the redirection using the appropriate method

Compare with previous version

added 1 commit * 85ba44c2 - Tests the redirection using the appropriate method [Compare with previous version](https://git.eleves.ens.fr/cof-geek/gestioCOF/merge_requests/208/diffs?diff_id=358&start_sha=dc07b072aba5808f9da497558d7c8a4224afd4ca)

mpepin commented

2017-04-05 00:41:49 +02:00

(Migrated from git.eleves.ens.fr)

merged

mpepin commented

2017-04-05 00:41:51 +02:00

(Migrated from git.eleves.ens.fr)

mentioned in commit fe66a6ef6b

mentioned in commit fe66a6ef6b27f9eb2003f0fe325a1b30a6a7dcb2

Stats K-Fêt - Python side #512

K-Fêt

K-Fêt - Stats

Scale

DayScale, WeekScale, MonthScale

K-Fêt - Misc