Création d'opérations aléatoires #497

Merged
lstephan merged 0 commits from Aufinal/create_opes into master 2017-03-25 11:08:28 +01:00
lstephan commented 2017-03-19 20:08:28 +01:00 (Migrated from git.eleves.ens.fr)

Rajoute une commande createopes aux commandes de gestion.

Cette commande prend comme argument le nombre d'opérations à créer, un nombre de jours et (optionnellement) le nombre de transferts à créer, et génère des opérations aléatoires étalées sur le nombre de jours spécifié.

Rajoute une commande `createopes` aux commandes de gestion. Cette commande prend comme argument le nombre d'opérations à créer, un nombre de jours et (optionnellement) le nombre de transferts à créer, et génère des opérations aléatoires étalées sur le nombre de jours spécifié.
lstephan commented 2017-03-20 05:57:11 +01:00 (Migrated from git.eleves.ens.fr)

added 1 commit

  • 6d36d50e - Fix 'at' property for transfergroups

Compare with previous version

added 1 commit * 6d36d50e - Fix 'at' property for transfergroups [Compare with previous version](https://git.eleves.ens.fr/cof-geek/gestioCOF/merge_requests/193/diffs?diff_id=253&start_sha=36dc6439a80d5673e518715ff0bbe8a4cc8d2b16)
mpepin commented 2017-03-20 23:10:32 +01:00 (Migrated from git.eleves.ens.fr)

Pourquoi ne pas faire le choix aléatoire dès les début ? ^^ Tu tiens beaucoup à Assurancetourix ? :P

Pourquoi ne pas faire le choix aléatoire dès les début ? \^\^ Tu tiens beaucoup à Assurancetourix ? :P
mpepin commented 2017-03-20 23:11:32 +01:00 (Migrated from git.eleves.ens.fr)

Heu, pourquoi '0' ?

Heu, pourquoi `'0'` ?
lstephan commented 2017-03-20 23:14:39 +01:00 (Migrated from git.eleves.ens.fr)

Je l'initialise à '0' au départ, puis je l'update à chaque opération (ligne 130) pour que ce soit cohérent.

Je l'initialise à '0' au départ, puis je l'update à chaque opération (ligne 130) pour que ce soit cohérent.
mpepin commented 2017-03-20 23:15:59 +01:00 (Migrated from git.eleves.ens.fr)

Non mais 0 c'est pas '0', on ne va pas commencer à faire du JS ^^

Non mais `0` c'est pas `'0'`, on ne va pas commencer à faire du JS \^\^
lstephan commented 2017-03-20 23:16:14 +01:00 (Migrated from git.eleves.ens.fr)

Nonobstant l'easter egg (^^), avoir un compte concert constant (et ne pas avoir à réfléchir à qui a eu des majorations quand tu veux faire des tests) peut être utile je dirais.

Nonobstant l'easter egg (\^\^), avoir un compte concert constant (et ne pas avoir à réfléchir à qui a eu des majorations quand tu veux faire des tests) peut être utile je dirais.
mpepin commented 2017-03-20 23:18:12 +01:00 (Migrated from git.eleves.ens.fr)

Ok très bien. Et dans ce cas Assurancetourix me semble tout à fait approprié 👍

Ok très bien. Et dans ce cas Assurancetourix me semble tout à fait approprié :+1:
mpepin commented 2017-03-20 23:24:59 +01:00 (Migrated from git.eleves.ens.fr)

Après lecture de la doc et à contrecœur… Admettons

Après lecture de la doc et à contrecœur… Admettons
mpepin commented 2017-03-20 23:25:09 +01:00 (Migrated from git.eleves.ens.fr)

resolved all discussions

resolved all discussions
mpepin commented 2017-03-20 23:28:38 +01:00 (Migrated from git.eleves.ens.fr)

Plus court :

transfergroup = TransferGroup.objects.create(...)
Plus court : ~~~ transfergroup = TransferGroup.objects.create(...) ~~~
mpepin commented 2017-03-20 23:30:56 +01:00 (Migrated from git.eleves.ens.fr)

Plus rapide : opegroup = OperationGroup.objects.create(...)

Plus rapide : `opegroup = OperationGroup.objects.create(...)`
mpepin commented 2017-03-20 23:34:18 +01:00 (Migrated from git.eleves.ens.fr)

Pour gagner en performance si tu veux générer beaucoup d'opérations (peut être utile pour tester le module stats par exemple), tu peux utiliser bulk_create

Pour gagner en performance si tu veux générer beaucoup d'opérations (peut être utile pour tester le module stats par exemple), tu peux utiliser [`bulk_create`](https://docs.djangoproject.com/en/1.10/ref/models/querysets/#django.db.models.query.QuerySet.bulk_create)
mpepin commented 2017-03-20 23:38:48 +01:00 (Migrated from git.eleves.ens.fr)

À la réflexion ce n'est pas immédiat comme ajout, peut-être est-il mieux d'attendre qu'on en ait vraiment besoin. Ton avis ?

À la réflexion ce n'est pas immédiat comme ajout, peut-être est-il mieux d'attendre qu'on en ait vraiment besoin. Ton avis ?
mpepin commented 2017-03-20 23:40:11 +01:00 (Migrated from git.eleves.ens.fr)

My bad, c'est immédiat : tu as juste à maintenir une liste d'opérations tout le long et appeler bulk_create à la fin

My bad, c'est immédiat : tu as juste à maintenir une liste d'opérations tout le long et appeler bulk_create à la fin
lstephan commented 2017-03-21 00:19:17 +01:00 (Migrated from git.eleves.ens.fr)

added 1 commit

  • dd4d1f30 - Improves creation efficiency

Compare with previous version

added 1 commit * dd4d1f30 - Improves creation efficiency [Compare with previous version](https://git.eleves.ens.fr/cof-geek/gestioCOF/merge_requests/193/diffs?diff_id=260&start_sha=6d36d50e9ab5146bfeb3451aa9625156d8273744)
delobell commented 2017-03-25 10:59:26 +01:00 (Migrated from git.eleves.ens.fr)

resolved all discussions

resolved all discussions
delobell commented 2017-03-25 11:08:27 +01:00 (Migrated from git.eleves.ens.fr)

mentioned in commit 215d4a63ad

mentioned in commit 215d4a63ad59ec41b5f5bc6659cf97c203697e9e
delobell commented 2017-03-25 11:08:28 +01:00 (Migrated from git.eleves.ens.fr)

merged

merged
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: DGNum/gestioCOF#497
No description provided.