From c890e21ed0b75d3cb8e85d739e1b6b317f8c466b Mon Sep 17 00:00:00 2001
From: Emmanuel Viennet <emmanuel.viennet@gmail.com>
Date: Wed, 8 Jun 2022 14:49:31 +0200
Subject: [PATCH] =?UTF-8?q?Modif=20bonus=20Tarbes,=20qui=20a=20la=20m?=
 =?UTF-8?q?=C3=AAme=20logique=20que=20celui=20de=20Rennes=201?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

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

diff --git a/app/comp/bonus_spo.py b/app/comp/bonus_spo.py
index 7cbac1c7..cd2db631 100644
--- a/app/comp/bonus_spo.py
+++ b/app/comp/bonus_spo.py
@@ -730,7 +730,7 @@ class BonusIUTRennes1(BonusSportAdditif):
     seuil_moy_gen = 10.0
     proportion_point = 1 / 20.0
     classic_use_bonus_ues = False
-    # Adapté de BonusTarbes, mais s'applique aussi en classic
+    # 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"""
         # Prend la note de chaque modimpl, sans considération d'UE
@@ -1067,14 +1067,15 @@ class BonusStNazaire(BonusSportMultiplicatif):
     factor_max = 0.1  # 10% max
 
 
-class BonusTarbes(BonusSportAdditif):
+class BonusTarbes(BonusIUTRennes1):
     """Calcul bonus optionnels (sport, culture), règle IUT de Tarbes.
 
     <ul>
     <li>Les étudiants opeuvent suivre un ou plusieurs activités optionnelles notées.
     La meilleure des notes obtenue est prise en compte, si elle est supérieure à 10/20.
     </li>
-    <li>Le trentième des points au dessus de 10 est ajouté à la moyenne des UE.
+    <li>Le trentième des points au dessus de 10 est ajouté à la moyenne des UE en BUT,
+    ou à la moyenne générale en DUT et LP.
     </li>
     <li> Exemple: un étudiant ayant 16/20 bénéficiera d'un bonus de (16-10)/30 = 0,2 points
     sur chaque UE.
@@ -1088,29 +1089,6 @@ class BonusTarbes(BonusSportAdditif):
     proportion_point = 1 / 30.0
     classic_use_bonus_ues = True
 
-    def compute_bonus(self, sem_modimpl_moys_inscrits, modimpl_coefs_etuds_no_nan):
-        """calcul du bonus"""
-        # Prend la note de chaque modimpl, sans considération d'UE
-        if len(sem_modimpl_moys_inscrits.shape) > 2:  # apc
-            sem_modimpl_moys_inscrits = sem_modimpl_moys_inscrits[:, :, 0]
-        # ici sem_modimpl_moys_inscrits est nb_etuds x nb_mods_bonus, en APC et en classic
-        note_bonus_max = np.max(sem_modimpl_moys_inscrits, axis=1)  # 1d, nb_etuds
-        ues = self.formsemestre.query_ues(with_sport=False).all()
-        ues_idx = [ue.id for ue in ues]
-
-        if self.formsemestre.formation.is_apc():  # --- BUT
-            bonus_moy_arr = np.where(
-                note_bonus_max > self.seuil_moy_gen,
-                (note_bonus_max - self.seuil_moy_gen) * self.proportion_point,
-                0.0,
-            )
-            self.bonus_ues = pd.DataFrame(
-                np.stack([bonus_moy_arr] * len(ues)).T,
-                index=self.etuds_idx,
-                columns=ues_idx,
-                dtype=float,
-            )
-
 
 class BonusTours(BonusDirect):
     """Calcul bonus sport & culture IUT Tours.