From 63784e341ad1d33f1bdb7060b88e25e0b3ad4d9e Mon Sep 17 00:00:00 2001
From: Emmanuel Viennet <emmanuel.viennet@gmail.com>
Date: Sat, 19 Feb 2022 01:00:40 +0100
Subject: [PATCH] Correction pour Amiens, Roanne

---
 app/comp/bonus_spo.py | 19 +++++++++++++++----
 1 file changed, 15 insertions(+), 4 deletions(-)

diff --git a/app/comp/bonus_spo.py b/app/comp/bonus_spo.py
index 65f6c03b9..0a95621e8 100644
--- a/app/comp/bonus_spo.py
+++ b/app/comp/bonus_spo.py
@@ -205,7 +205,8 @@ class BonusSportAdditif(BonusSport):
         """calcul du bonus
         sem_modimpl_moys_inscrits: les notes de sport
             En APC: ndarray (nb_etuds, nb_mod_sport, nb_ues_non_bonus)
-        modimpl_coefs_etuds_no_nan:
+            En classic: ndarray (nb_etuds, nb_mod_sport)
+        modimpl_coefs_etuds_no_nan: même shape, les coefs.
         """
         if 0 in sem_modimpl_moys_inscrits.shape:
             # pas d'étudiants ou pas d'UE ou pas de module...
@@ -228,12 +229,22 @@ class BonusSportAdditif(BonusSport):
             bonus_moy_arr = np.clip(bonus_moy_arr, 0.0, 20.0, out=bonus_moy_arr)
 
         # en APC, bonus_moy_arr est (nb_etuds, nb_ues_non_bonus)
-        if self.formsemestre.formation.is_apc() or self.classic_use_bonus_ues:
+        if self.formsemestre.formation.is_apc():
             # Bonus sur les UE et None sur moyenne générale
             ues_idx = [ue.id for ue in self.formsemestre.query_ues(with_sport=False)]
             self.bonus_ues = pd.DataFrame(
                 bonus_moy_arr, index=self.etuds_idx, columns=ues_idx, dtype=float
             )
+        elif self.classic_use_bonus_ues:
+            # Formations classiques apppliquant le bonus sur les UEs
+            # ici bonus_moy_arr = ndarray 1d nb_etuds
+            ues_idx = [ue.id for ue in self.formsemestre.query_ues(with_sport=False)]
+            self.bonus_ues = pd.DataFrame(
+                np.stack([bonus_moy_arr] * len(ues_idx)).T,
+                index=self.etuds_idx,
+                columns=ues_idx,
+                dtype=float,
+            )
         else:
             # Bonus sur la moyenne générale seulement
             self.bonus_moy_gen = pd.Series(
@@ -693,7 +704,7 @@ class BonusRoanne(BonusSportAdditif):
     displayed_name = "IUT de Roanne"
     seuil_moy_gen = 0.0
     bonus_max = 0.6  # plafonnement à 0.6 points
-    apply_bonus_mg_to_ues = True  # sur les UE, même en DUT et LP
+    classic_use_bonus_ues = True  # sur les UE, même en DUT et LP
 
 
 class BonusStDenis(BonusSportAdditif):
@@ -792,7 +803,7 @@ class BonusIUTV(BonusSportAdditif):
 
     name = "bonus_iutv"
     displayed_name = "IUT de Villetaneuse"
-    pass  # oui, c'ets le bonus par défaut
+    pass  # oui, c'est le bonus par défaut
 
 
 def get_bonus_class_dict(start=BonusSport, d=None):