-
Notifications
You must be signed in to change notification settings - Fork 9
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
base: master
Are you sure you want to change the base?
Conversation
…fois que d'individus dans la famille
@@ -0,0 +1,53 @@ | |||
- name: Les ressources de l'aide FSL Eure-et-Loir ne sont prises en compte qu'une seule fois |
There was a problem hiding this comment.
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
).
…ant de fois que d'individus dans la famille
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) |
There was a problem hiding this comment.
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. 😕
Bonjour, Cette PR est ouverte et inactive depuis longtemps, est-ce qu’elle est encore pertinente? |
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 |
Correction de
eure_et_loir_fsl_base_ressources
:rsa
perçu est maintenant pris en comptepaje_cmg
déjà présente dans lapage
est supprimée de la base ressourcesmenage.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.