openfisca-france icon indicating copy to clipboard operation
openfisca-france copied to clipboard

Supprime variable bourse_enseignement_sup

Open bfabre01 opened this issue 2 years ago • 5 comments

  • Supprime une variable redondante
  • Périodes concernées : toutes.
  • Zones impactées : chemin/vers/le/fichier/contenant/les/variables/impactées.
  • Détails :
    • Supprime bourse_enseignement_sup (pour n'avoir que bourse_criteres_sociaux)
    • Résout #1980

bfabre01 avatar Jan 25 '23 15:01 bfabre01

@bfabre01 : Je ne sais pas si la différence reflête le fait qu'il y a plusieurs types de bourses dans l'enseignement supérieur:

  • au mérite
  • sur critères sociaux
    donc je vais laisser @guillett répondre

benjello avatar Jan 25 '23 19:01 benjello

Il y a au moins la distinction soulignée par @benjello qui est en particulier nécessaire pour le calcul de la Complémentaire Santé Solidaire. Et en l'occurrence, toutes les bourses de l'enseignement supérieur sont incluses dans la base ressource https://www.legifrance.gouv.fr/codes/article_lc/LEGIARTI000030055485/2018-08-29

L'aide au mérite est incluses dans les bourses de l'enseignement supérieur en complément des bourses sur critères sociaux

https://www.complementaire-sante-solidaire.gouv.fr/ressources-cmu-complementaire.php

guillett avatar Jan 26 '23 08:01 guillett

@guillett @benjello : ok. Le problème est que la variable bourse_enseignement_sup est selon le label, toute bourse enseignement supérieur. Dans l'état actuel du code, la distinction critère socioaux/mérite n'existe pas, et la variable bourse_enseignement_sup , qui est une variable d'input, contient donc aussi dans sa définition bourse_criteres_sociaux, sans qu'elle soit liée avec elle. Donc:

  • soit on fait cette distinction de manière claire, pour éviter toute redondance
  • soit on ne la fait pas, et mieux vaut merger la PR, car la version actuelle prête à confusion et erreur. @guillett : je te laisse me dire si tu veux coder cette distinction. Si non, je propose de merger. Car ça sera une amélioration par rapport à l'existant dans tous les cas.

bfabre01 avatar Jan 27 '23 09:01 bfabre01

Est ce que les renommages suivants seraient suffisants pour toi

  • bourse_criteres_sociaux en bourse_criteres_sociaux_enseignement_superieur
  • bourse_enseignement_sup en bourse_enseignement_superieur

?

Est ce qu'il faudrait ajouter une première formule de bourse_enseignement_sup/bourse_enseignement_superieur pour renvoyer la même chose que bourse_criteres_sociaux/bourse_criteres_sociaux_enseignement_superieur ?

Est ce qu'il faudrait aussi intégrer un 9ème de aide_merite_montant ?

guillett avatar Jan 27 '23 17:01 guillett

@bfabre01 et @guillett : où en est-on de cette PR ?

benjello avatar Aug 08 '24 07:08 benjello