Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Supprime variable bourse_enseignement_sup #2010

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

bfabre01
Copy link

  • Supprime une variable redondante
  • Périodes concernées : toutes.
  • Zones impactées : chemin/vers/le/fichier/contenant/les/variables/impactées.
  • Détails :

@bfabre01 bfabre01 requested a review from benjello January 25, 2023 16:47
@benjello
Copy link
Member

@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

@guillett
Copy link
Member

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

@bfabre01
Copy link
Author

@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.

@guillett
Copy link
Member

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 ?

@benjello
Copy link
Member

benjello commented Aug 8, 2024

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants