Editeur partitions: Cache boite non affectés quand elle est vide

app/api/evaluations.py

@@ -153,8 +153,8 @@ def evaluation_set_notes(evaluation_id: int):
     The request content type should be "application/json",
     and contains:
     {
-      'notes' : [ (etudid, value), ... ],
-      'comment' : opetional string
+      'notes' : [ [etudid, value], ... ],
+      'comment' : optional string
     }
     Result:
      - nb_changed: nombre de notes changées

app/comp/bonus_spo.py

@@ -18,7 +18,7 @@ import pandas as pd
 
 from flask import g
 
-from app.scodoc.codes_cursus import UE_SPORT
+from app.scodoc.codes_cursus import UE_SPORT, UE_STANDARD
 from app.scodoc.codes_cursus import CursusDUT, CursusDUTMono
 from app.scodoc.sco_utils import ModuleType
 
@@ -740,6 +740,7 @@ class BonusGrenobleIUT1(BonusSportMultiplicatif):
 
     name = "bonus_iut1grenoble_2017"
     displayed_name = "IUT de Grenoble 1"
+
     # C'est un bonus "multiplicatif": on l'exprime en additif,
     # sur chaque moyenne d'UE m_0
     # Augmenter de 5% correspond à multiplier par a=1.05
@@ -782,6 +783,7 @@ class BonusIUTRennes1(BonusSportAdditif):
     seuil_moy_gen = 10.0
     proportion_point = 1 / 20.0
     classic_use_bonus_ues = False
+
     # S'applique aussi en classic, sur la moy. gen.
     def compute_bonus(self, sem_modimpl_moys_inscrits, modimpl_coefs_etuds_no_nan):
         """calcul du bonus"""
@@ -1336,6 +1338,7 @@ class BonusStNazaire(BonusSport):
     classic_use_bonus_ues = True  # s'applique aux UEs en DUT et LP
     amplitude = 0.01 / 4  # 4pt => 1%
     factor_max = 0.1  # 10% max
+
     # Modifié 2022-11-29: calculer chaque bonus
     # (de 1 à 3 modules) séparément.
     def compute_bonus(self, sem_modimpl_moys_inscrits, modimpl_coefs_etuds_no_nan):
@@ -1533,6 +1536,63 @@ class BonusIUTV(BonusSportAdditif):
     # c'est le bonus par défaut: aucune méthode à surcharger
 
 
+# Finalement inutile: un bonus direct est mieux adapté à leurs besoins.
+# # class BonusMastersUSPNIG(BonusSportAdditif):
+#     """Calcul bonus modules optionnels (sport, culture), règle Masters de l'Institut Galilée (USPN)
+
+#     Les étudiants  peuvent suivre des enseignements optionnels
+#     de l'USPN (sports, musique, deuxième langue, culture, etc) dans une
+#     UE libre. Les points au-dessus de 10 sur 20 obtenus dans cette UE
+#     libre sont ajoutés au total des points obtenus pour les UE obligatoires
+#     du semestre concerné.
+#     """
+
+#     name = "bonus_masters__uspn_ig"
+#     displayed_name = "Masters de l'Institut Galilée (USPN)"
+#     proportion_point = 1.0
+#     seuil_moy_gen = 10.0
+
+#     def __init__(
+#         self,
+#         formsemestre: "FormSemestre",
+#         sem_modimpl_moys: np.array,
+#         ues: list,
+#         modimpl_inscr_df: pd.DataFrame,
+#         modimpl_coefs: np.array,
+#         etud_moy_gen,
+#         etud_moy_ue,
+#     ):
+#         # Pour ce bonus, il nous faut la somme des coefs des modules non bonus
+#         # du formsemestre (et non auxquels les étudiants sont inscrits !)
+#         self.sum_coefs = sum(
+#             [
+#                 m.module.coefficient
+#                 for m in formsemestre.modimpls_sorted
+#                 if (m.module.module_type == ModuleType.STANDARD)
+#                 and (m.module.ue.type == UE_STANDARD)
+#             ]
+#         )
+#         super().__init__(
+#             formsemestre,
+#             sem_modimpl_moys,
+#             ues,
+#             modimpl_inscr_df,
+#             modimpl_coefs,
+#             etud_moy_gen,
+#             etud_moy_ue,
+#         )
+#         # Bonus sur la moyenne générale seulement
+#         # On a dans bonus_moy_arr le bonus additif classique
+#         # Sa valeur sera appliquée comme moy_gen += bonus_moy_gen
+#         # or ici on veut
+#         # moy_gen = (somme des notes + bonus_moy_arr) / somme des coefs
+#         # moy_gen +=  bonus_moy_arr / somme des coefs
+
+#         self.bonus_moy_gen = (
+#             None if self.bonus_moy_gen is None else self.bonus_moy_gen / self.sum_coefs
+#         )
+
+
 def get_bonus_class_dict(start=BonusSport, d=None):
     """Dictionnaire des classes de bonus
     (liste les sous-classes de BonusSport ayant un nom)

app/models/formsemestre.py

@@ -788,6 +788,8 @@ class FormSemestre(db.Model):
         Les groupes de parcours sont ceux de la partition scu.PARTITION_PARCOURS
         et leur nom est le code du parcours (eg "Cyber").
         """
+        if self.formation.referentiel_competence_id is None:
+            return  # safety net
         partition = Partition.query.filter_by(
             formsemestre_id=self.id, partition_name=scu.PARTITION_PARCOURS
         ).first()
@@ -811,7 +813,10 @@ class FormSemestre(db.Model):
             query = (
                 ApcParcours.query.filter_by(code=group.group_name)
                 .join(ApcReferentielCompetences)
-                .filter_by(dept_id=g.scodoc_dept_id)
+                .filter_by(
+                    dept_id=g.scodoc_dept_id,
+                    id=self.formation.referentiel_competence_id,
+                )
             )
             if query.count() != 1:
                 log(

app/scodoc/sco_moduleimpl_inscriptions.py

@@ -28,7 +28,7 @@
 """Opérations d'inscriptions aux modules (interface pour gérer options ou parcours)
 """
 import collections
-from operator import itemgetter
+from operator import attrgetter
 
 import flask
 from flask import url_for, g, request
@@ -671,7 +671,7 @@ def descr_inscrs_module(moduleimpl_id, set_all, partitions):
         gr.append((partition["partition_name"], grp))
     #
     d = []
-    for (partition_name, grp) in gr:
+    for partition_name, grp in gr:
         if grp:
             d.append("groupes de %s: %s" % (partition_name, ", ".join(grp)))
     r = []
@@ -683,25 +683,25 @@ def descr_inscrs_module(moduleimpl_id, set_all, partitions):
     return False, len(ins), " et ".join(r)
 
 
-def _fmt_etud_set(ins, max_list_size=7):
+def _fmt_etud_set(etudids, max_list_size=7) -> str:
     # max_list_size est le nombre max de noms d'etudiants listés
     # au delà, on indique juste le nombre, sans les noms.
-    if len(ins) > max_list_size:
-        return "%d étudiants" % len(ins)
+    if len(etudids) > max_list_size:
+        return f"{len(etudids)} étudiants"
     etuds = []
-    for etudid in ins:
-        etuds.append(sco_etud.get_etud_info(etudid=etudid, filled=True)[0])
-    etuds.sort(key=itemgetter("nom"))
+    for etudid in etudids:
+        etud = Identite.query.get(etudid)
+        if etud:
+            etuds.append(etud)
+
     return ", ".join(
         [
-            '<a class="discretelink" href="%s">%s</a>'
-            % (
+            f"""<a class="discretelink" href="{
                 url_for(
-                    "scolar.ficheEtud", scodoc_dept=g.scodoc_dept, etudid=etud["etudid"]
-                ),
-                etud["nomprenom"],
-            )
-            for etud in etuds
+                    "scolar.ficheEtud", scodoc_dept=g.scodoc_dept, etudid=etud.id
+                )
+            }">{etud.nomprenom}</a>"""
+            for etud in sorted(etuds, key=attrgetter("sort_key"))
         ]
     )
 

app/static/css/partition_editor.css

@@ -354,6 +354,10 @@ body.editionActivated .filtres .nonEditable .move {
 	display: initial;
 }
 
+.groupe:has(.etudiants:empty) {
+    display: none;
+}
+
 /* .filtres .unselect {
 	background: rgba(0, 153, 204, 0.5) !important;
 } */

app/tables/recap.py

@@ -479,17 +479,18 @@ class TableRecap(tb.Table):
         for row in self.rows:
             etud = row.etud
             admission = etud.admission.first()
-            first = True
-            for cid, title in fields.items():
-                cell = row.add_cell(
-                    cid,
-                    title,
-                    getattr(admission, cid) or "",
-                    "admission",
-                )
-                if first:
-                    cell.classes.append("admission_first")
-                    first = False
+            if admission:
+                first = True
+                for cid, title in fields.items():
+                    cell = row.add_cell(
+                        cid,
+                        title,
+                        getattr(admission, cid) or "",
+                        "admission",
+                    )
+                    if first:
+                        cell.classes.append("admission_first")
+                        first = False
 
     def add_cursus(self):
         """Ajoute colonne avec code cursus, eg 'S1 S2 S1'

sco_version.py

@@ -1,7 +1,7 @@
 # -*- mode: python -*-
 # -*- coding: utf-8 -*-
 
-SCOVERSION = "9.4.81"
+SCOVERSION = "9.4.82"
 
 SCONAME = "ScoDoc"