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

Corrige la base de ressources FSL Eure-et-Loir #88

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

Conversation

sandcha
Copy link
Contributor

@sandcha sandcha commented Jul 1, 2021

Correction de eure_et_loir_fsl_base_ressources :

  • Le montant de rsa perçu est maintenant pris en compte
  • La paje_cmg déjà présente dans la page est supprimée de la base ressources
  • Les ressources familiales ne sont prises en compte qu'une seule fois
    • Avant ce correctif, les projections employées nous amenaient à compter les ressources familiales autant de fois qu'il y avait d'individus dans la population. En effet, les syntaxes similaires à menage.members.famille("ppa", period) renvoient des vecteurs de longueur égale au nombre d'individus dans la population et où la valeur de "ppa" (variable de l'entité Famille donc) est répétée pour chaque individu. Ainsi, les sommes appliquées dans le calcul comptaient plusieurs fois la même ressource.

@sandcha sandcha marked this pull request as ready for review July 1, 2021 16:56
@@ -0,0 +1,53 @@
- name: Les ressources de l'aide FSL Eure-et-Loir ne sont prises en compte qu'une seule fois
Copy link
Contributor Author

@sandcha sandcha Jul 1, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ce test ne me semble pas avoir vocation à rester, à moins de réduire le périmètre le la variable de base de ressources afin qu'elle renvoie le total des ressources prises en comptes à la place du booléen actuel qui résulte d'une comparaison avec le seuil de pauvreté.
Néanmoins, il est fourni pour le temps de la revue parce qu'il a servi au debug (avec un print de menage_resources).

@sandcha sandcha requested review from frtomas, guillett and mtifarine July 1, 2021 16:59
for resource in famille_resources_names_annuelles
])

# projette les ressources de la famille sur un individu quelconque pour éviter les doublons
selectionne_un_membre_par_groupe = menage.members.has_role(Menage.PERSONNE_DE_REFERENCE)
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ici, on sélectionne un rôle de l'entité Ménage pour pouvoir filtrer des vecteurs résultant d'une projection menage.members.famille. Il aurait été plus lisible de projeter sur un rôle de Famille mais la Famille n'a pas de rôle unique (parents peut avoir 2 membres) et je n'ai pas trouvé de syntaxe plus claire. 😕

@Allan-CodeWorks
Copy link
Contributor

Bonjour,

Cette PR est ouverte et inactive depuis longtemps, est-ce qu’elle est encore pertinente?
Merci pour votre retour.

@frtomas
Copy link
Contributor

frtomas commented May 9, 2023

Cette PR est à priori bien toujours d'actualité, nous sommes en train de dépiler progressivement les vieilles PR qui traînent depuis un certain temps

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