From 4e300c3dbbe9575732e75aee14f730514698653a Mon Sep 17 00:00:00 2001 From: lehmann Date: Sat, 22 Jul 2023 22:40:09 +0200 Subject: [PATCH 1/6] Groups auto assignment --- app/static/css/partition_editor.css | 22 +- app/templates/scolar/partition_editor.j2 | 119 ++- .../scolar/students_groups_auto_assignment.j2 | 859 +++++++++++++++++- 3 files changed, 977 insertions(+), 23 deletions(-) diff --git a/app/static/css/partition_editor.css b/app/static/css/partition_editor.css index c74e3f272..4697bc99e 100644 --- a/app/static/css/partition_editor.css +++ b/app/static/css/partition_editor.css @@ -106,7 +106,7 @@ body:not(.editionActivated) .editing { @keyframes boing { 100% { - transform: translateY(-20px) + transform: translateY(-20px); } } @@ -152,6 +152,7 @@ body.editionActivated .filtres>div>div>div>div { color: #000; border-radius: 4px; outline: 4px solid #FFF; + padding: 2px; } /* Suppression */ @@ -355,7 +356,7 @@ body.editionActivated .filtres .nonEditable .move { } .groupe:has(.etudiants:empty) { - display: none; + display: none; } /* .filtres .unselect { @@ -365,6 +366,23 @@ body.editionActivated .filtres .nonEditable .move { /*****************************/ /* Zone Etudiants */ /*****************************/ +#zoneChoix .autoAffectation>a { + text-decoration: underline; +} + +#zoneChoix .dropZone { + background: #FFF; + border-radius: 8px; + border: 2px dashed #09C; + margin-bottom: 4px; + padding: 4px; + transition: 0.2s; +} + +.fileOver { + transform: scale(0.9); +} + #zoneChoix>.autoAffectation { background: #c9c9c9; color: #141414; diff --git a/app/templates/scolar/partition_editor.j2 b/app/templates/scolar/partition_editor.j2 index d98d8dc19..82cca6586 100644 --- a/app/templates/scolar/partition_editor.j2 +++ b/app/templates/scolar/partition_editor.j2 @@ -22,6 +22,23 @@

Étudiants

+
+ + + Aide à l'affectation dans les parcours +
Importer les résultats : +
+
+ Déposez le fichier .xlsx ou
+ +
+
+
+
Affecter automatiquement les étudiants du groupe
@@ -38,6 +55,7 @@
+ + \ No newline at end of file From 07f5f6c332366d908389d6b6cb958066ee22b285 Mon Sep 17 00:00:00 2001 From: Emmanuel Viennet Date: Mon, 24 Jul 2023 16:48:41 +0200 Subject: [PATCH 2/6] Optimisation: ajout cache par requete a FormSemestre.get_ues() --- app/api/jury.py | 2 ++ app/models/formsemestre.py | 18 ++++++++++++--- app/scodoc/sco_pv_dict.py | 45 ++++++++++++++++++++++---------------- 3 files changed, 43 insertions(+), 22 deletions(-) diff --git a/app/api/jury.py b/app/api/jury.py index d3a1d1458..294eef2f6 100644 --- a/app/api/jury.py +++ b/app/api/jury.py @@ -282,6 +282,8 @@ def validation_rcue_record(etudid: int): ) operation = "record" db.session.add(validation) + # invalider bulletins (les autres résultats ne dépendent pas des RCUEs): + sco_cache.invalidate_formsemestre_etud(etud) db.session.commit() Scolog.logdb( method="validation_rcue_record", diff --git a/app/models/formsemestre.py b/app/models/formsemestre.py index a7b10195a..be395fea5 100644 --- a/app/models/formsemestre.py +++ b/app/models/formsemestre.py @@ -307,6 +307,17 @@ class FormSemestre(db.Model): - et sont associées à l'un des parcours de ce formsemestre (ou à aucun, donc tronc commun). """ + # per-request caching + key = (self.id, with_sport) + _cache = getattr(g, "_formsemestre_get_ues_cache", None) + if _cache: + result = _cache.get(key, False) + if result is not False: + return result + else: + g._formsemestre_get_ues_cache = {} + _cache = g._formsemestre_get_ues_cache + formation: Formation = self.formation if formation.is_apc(): # UEs de tronc commun (sans parcours indiqué) @@ -326,8 +337,7 @@ class FormSemestre(db.Model): ).filter(UniteEns.semestre_idx == self.semestre_id) } ) - ues = sem_ues.values() - return sorted(ues, key=attrgetter("numero", "acronyme")) + ues = sorted(sem_ues.values(), key=attrgetter("numero", "acronyme")) else: sem_ues = db.session.query(UniteEns).filter( ModuleImpl.formsemestre_id == self.id, @@ -336,7 +346,9 @@ class FormSemestre(db.Model): ) if not with_sport: sem_ues = sem_ues.filter(UniteEns.type != codes_cursus.UE_SPORT) - return sem_ues.order_by(UniteEns.numero).all() + ues = sem_ues.order_by(UniteEns.numero).all() + _cache[key] = ues + return ues @cached_property def modimpls_sorted(self) -> list[ModuleImpl]: diff --git a/app/scodoc/sco_pv_dict.py b/app/scodoc/sco_pv_dict.py index f9861f54f..e4e58ca12 100644 --- a/app/scodoc/sco_pv_dict.py +++ b/app/scodoc/sco_pv_dict.py @@ -63,25 +63,32 @@ def dict_pvjury( Si with_parcours_decisions: ajoute infos sur code decision jury de tous les semestre du parcours Résultat: { - 'date' : date de la decision la plus recente, - 'formsemestre' : sem, - 'is_apc' : bool, - 'formation' : { 'acronyme' :, 'titre': ... } - 'decisions' : { [ { 'identite' : {'nom' :, 'prenom':, ...,}, - 'etat' : I ou D ou DEF - 'decision_sem' : {'code':, 'code_prev': }, - 'decisions_ue' : { ue_id : { 'code' : ADM|CMP|AJ, 'event_date' :, - 'acronyme', 'numero': } }, - 'autorisations' : [ { 'semestre_id' : { ... } } ], - 'validation_parcours' : True si parcours validé (diplome obtenu) - 'prev_code' : code (calculé slt si with_prev), - 'mention' : mention (en fct moy gen), - 'sum_ects' : total ECTS acquis dans ce semestre (incluant les UE capitalisées) - 'sum_ects_capitalises' : somme des ECTS des UE capitalisees - } - ] - }, - 'decisions_dict' : { etudid : decision (comme ci-dessus) }, + 'date' : str = date de la decision la plus recente, format dd/mm/yyyy, + 'formsemestre' : dict = formsemestre, + 'is_apc' : bool, + 'formation' : { 'acronyme' :, 'titre': ... } + 'decisions' : [ + { + 'identite' : {'nom' :, 'prenom':, ...,}, + 'etat' : I ou D ou DEF + 'decision_sem' : {'code':, 'code_prev': }, + 'decisions_ue' : { + ue_id : { + 'code' : ADM|CMP|AJ, + 'ects' : float, + 'event_date' :str = "dd/mm/yyyy", + }, + }, + 'autorisations' : [ { 'semestre_id' : { ... } } ], + 'validation_parcours' : True si parcours validé (diplome obtenu) + 'prev_code' : code (calculé slt si with_prev), + 'mention' : mention (en fct moy gen), + 'sum_ects' : total ECTS acquis dans ce semestre (incluant les UE capitalisées) + 'sum_ects_capitalises' : somme des ECTS des UE capitalisees + }, + ... + ], + 'decisions_dict' : { etudid : decision (comme ci-dessus) }, } """ formsemestre: FormSemestre = FormSemestre.query.get_or_404(formsemestre_id) From c888eab6d7e47c30e042c0d4f918d0975fb3f612 Mon Sep 17 00:00:00 2001 From: lehmann Date: Mon, 24 Jul 2023 21:24:44 +0200 Subject: [PATCH 3/6] Correctif affectations groupes --- app/static/css/partition_editor.css | 11 ++-- app/templates/scolar/partition_editor.j2 | 50 ++++++++++--------- .../scolar/students_groups_auto_assignment.j2 | 16 +++--- 3 files changed, 44 insertions(+), 33 deletions(-) diff --git a/app/static/css/partition_editor.css b/app/static/css/partition_editor.css index 4697bc99e..777b37e4b 100644 --- a/app/static/css/partition_editor.css +++ b/app/static/css/partition_editor.css @@ -366,6 +366,11 @@ body.editionActivated .filtres .nonEditable .move { /*****************************/ /* Zone Etudiants */ /*****************************/ +#zoneChoix summary{ + margin: 0 0 16px; + cursor: pointer; +} + #zoneChoix .autoAffectation>a { text-decoration: underline; } @@ -383,7 +388,7 @@ body.editionActivated .filtres .nonEditable .move { transform: scale(0.9); } -#zoneChoix>.autoAffectation { +#zoneChoix .autoAffectation { background: #c9c9c9; color: #141414; padding: 4px 8px; @@ -391,13 +396,13 @@ body.editionActivated .filtres .nonEditable .move { border-radius: 4px; } -#zoneChoix>.autoAffectation>select { +#zoneChoix .autoAffectation>select { border: none; padding: 4px; border-radius: 4px; } -#zoneChoix>.autoAffectation>.affectationGo { +#zoneChoix .autoAffectation>.affectationGo { display: inline-block; background: #0c9; padding: 8px 16px; diff --git a/app/templates/scolar/partition_editor.j2 b/app/templates/scolar/partition_editor.j2 index 82cca6586..79ca2f620 100644 --- a/app/templates/scolar/partition_editor.j2 +++ b/app/templates/scolar/partition_editor.j2 @@ -22,30 +22,34 @@

Étudiants

-
- - - Aide à l'affectation dans les parcours -
Importer les résultats : -
-
- Déposez le fichier .xlsx ou
- -
-
+
+ Outils d'affections +
+ + + Aide à l'affectation dans les parcours +
Importer les résultats : +
+
+ Déposez le fichier .xlsx ou
+ +
+
+
-
-
- Affecter automatiquement les étudiants du groupe
- - vers le groupe - -
Valider
-
+
+ Affecter automatiquement les étudiants du groupe
+ + vers le groupe + +
Valider
+
+ +
diff --git a/app/templates/scolar/students_groups_auto_assignment.j2 b/app/templates/scolar/students_groups_auto_assignment.j2 index 3310e328e..1bd78cf66 100644 --- a/app/templates/scolar/students_groups_auto_assignment.j2 +++ b/app/templates/scolar/students_groups_auto_assignment.j2 @@ -375,7 +375,7 @@ Object.values(partition.groups).forEach(group => { parcours[group.id] = { nom: group.group_name, - places: savedData.parcours[group.id].places || 28, + places: savedData.parcours?.[group.id].places || 28, etudiants: [] // Résultat du tri } }) @@ -435,17 +435,19 @@ } },*/ etudiantsRaw.forEach(etudiant => { - saved = savedData.etudiants['id' + etudiant.id]; + saved = savedData.etudiants?.['id' + etudiant.id] || {}; etudiants['id' + etudiant.id] = { - actif: saved.actif ? true : false, + actif: saved.actif == false ? false : true, nom: etudiant.nom + " " + etudiant.prenom, criteres: saved.criteres || {}, voeux: saved.voeux || {} }; - /*Object.keys(parcours).forEach((idParcours, index) => { - etudiants['id' + etudiant.id].criteres[idParcours] = 10; - etudiants['id' + etudiant.id].voeux[idParcours] = index + 1; - })*/ + + if (Object.keys(saved).length == 0) + Object.keys(parcours).forEach((idParcours, index) => { + etudiants['id' + etudiant.id].criteres[idParcours] = 10; + etudiants['id' + etudiant.id].voeux[idParcours] = index + 1; + }) }) /* Zone étudiants */ From ce2b9df6c07a2d7fc0c82c9236e5930f11c8b2c1 Mon Sep 17 00:00:00 2001 From: Emmanuel Viennet Date: Mon, 24 Jul 2023 21:34:35 +0200 Subject: [PATCH 4/6] Correction affecation parcours --- app/templates/scolar/partition_editor.j2 | 2 +- app/templates/scolar/students_groups_auto_assignment.j2 | 4 ++-- sco_version.py | 2 +- 3 files changed, 4 insertions(+), 4 deletions(-) diff --git a/app/templates/scolar/partition_editor.j2 b/app/templates/scolar/partition_editor.j2 index 79ca2f620..b9a95eb1f 100644 --- a/app/templates/scolar/partition_editor.j2 +++ b/app/templates/scolar/partition_editor.j2 @@ -23,7 +23,7 @@

Étudiants

- Outils d'affections + Outils d'affectation