-
Notifications
You must be signed in to change notification settings - Fork 101
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 déconjugalisation du aah_plafond_ressource + test #2218
Conversation
@@ -6,6 +6,8 @@ values: | |||
value: 0.89 | |||
2019-11-01: | |||
value: 0.81 | |||
2023-10-01: |
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.
Ne vaudrait-il pas mieux mettre ce paramètre à null
?
Quitte à corriger la formule.
Le point à avoir en tête est que ces séries doivent, dans la mesure du possible, avoir un sens indépendamment du code de la formule.
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.
Je me demandais aussi si il ne serait pas mieux de faire une formule différente dans aah_base_ressource après octobre 2023. Ce qui permettrait de garder plus de facilité à implémenter le choix (ce qui est demandé dans #2217)
...s/prestations_etat_de_sante/invalidite/aah/majoration_plafond/majoration_plafond_couple.yaml
Outdated
Show resolved
Hide resolved
@@ -32,5 +32,5 @@ metadata: | |||
2023-10-01: "2022-12-29" | |||
Notes: | |||
2023-10-01: | |||
title: "Déconjugalisation de l'AAH, sauf sur option. Nous ne la prenons pas en compte" | |||
title: "Déconjugalisation de l'AAH, sauf sur option. Nous ne prenons pas en compte cette option" |
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.
J'ai l'impression que cette phrase concerne l'implémentation du calcul de l'AAH mais je peux me tromper.
Dans ce cas, il vaudrait mieux la mettre dans le fichier python. des variables openfisca.
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.
Il me semble que cela concerne les deux, ce paramètre restant techniquement inchangé pour les personnes éligibles à la conjugalisation (alors qu'on le met à Null ici). Ce qui est potentiellement problématique pour les barèmes d'ailleurs ? Mais il manque effectivement une référence à ce manque dans le fichier python. Je compte de toute façon attendre que l'issue #2217 avec @kevinpolisano avance pour voir s'il n'y a pas de solution plus satisfaisante à l'option de déconjugalisation (et implémenter de meilleurs tests au passage).
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.
Tu peux laisser courrrir la série, enlever le null
et préciser que cela reste ainsi pour les personnes éligibles à la conjugalisation.
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.
Cela fait déplacer les commentaires quand même j'imagine, à mois de remettre l'ancienne valeur en 2023 pour insérer une note ?
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 n'est pas génial de répéter la valeur pour juste insérer une note.
Donc le mieux serait de rajouter une ligne dans le champ documentation.
Il n'y a pas de solution miracle pour ce genre de cas.
…tat_de_sante/invalidite/aah/majoration_plafond/majoration_plafond_couple.yaml Co-authored-by: Mahdi Ben Jelloul <[email protected]>
5621e48
to
62b7151
Compare
prestations_sociales/prestations_etat_de_sante/invalidite/aah/majoration_plafond
-
model/prestations/minima sociaux/aah
aah_base_ressource
etaah
selon leur statut de conjugalisation.Ces changements :
Quelques conseils à prendre en compte :
setup.py
.CHANGELOG.md
.Et surtout, n'hésitez pas à demander de l'aide ! :)