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

Fiabilisation générale du calcul des aides au logement #1269

Merged
merged 11 commits into from
Mar 11, 2019

Conversation

mtifarine
Copy link
Contributor

  • Évolution du système socio-fiscal.
  • Périodes concernées : toutes.
  • Zones impactées :
    • model/prestations/aides_logement.
  • Détails :
    • Ajoute la variable 'date_debut_chomage'.
    • Corrige une double application des coefficients 'coloc' et 'chambre' dans aides_logement.
    • Corrige l'application de la règle des 2/3 du montant du loyer pour l'AL et l'APL.

@mtifarine mtifarine added the contrib:msa Identification des sujets MSA label Feb 5, 2019
@Morendil
Copy link
Contributor

Morendil commented Feb 5, 2019

@mtifarine Merci ! Deux remarques avant de faire une relecture plus détaillée:

  • il faut rebaser cette PR sur la version la plus à jour de master
  • les tests ne passent pas 😿

@mtifarine mtifarine force-pushed the msa_fiabilisation_generele_al_apl branch from 40639c3 to d9fa22b Compare February 5, 2019 13:21
@Morendil Morendil removed their assignment Feb 10, 2019
@Morendil
Copy link
Contributor

@alexsegura @monbocal @guillett Je m'inquiète d'une possible régression dans Mes Aides lors du calcul des APL, avec l'introduction de la variable date_debut_chomage, qui n'est pas fournie pour l'instant. Cf les tests Mes Aides modifiés par la PR. Pouvez-vous vérifier ?

@mtifarine Je vois qu'une des corrections porte sur la condition 'coloc' mais je ne vois pas de mise à jour d'un test dans lequel cette condition est vraie; nous n'avons qu'un seul test où c'est le cas, tests/mes-aides.gouv.fr/test_mes_aides_54d8df76b93972db44593f80.yaml. Je ne comprends donc pas quel calcul est faux dans l'état actuel du code et se voit corrigé par la PR.

Même question pour logement_chambre, pour lequel les tests sont un peu plus nombreux, par exemple tests/mes-aides.gouv.fr/test_mes_aides_54ef4cdefa384b5575517bc2.yaml.

Egalement, dans les nouveaux tests apportés, seuls les cas n°1 et n°2 sont en échec lorsque je les déroule sur master, en n'ajoutant que la variable date_debut_chomage. Il me semble donc que les cas 3 à 6 ne sont pas utiles pour l'intégration continue et qu'il vaut mieux ne pas les ajouter.

Copy link
Contributor

@Morendil Morendil left a comment

Choose a reason for hiding this comment

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

Tests superflus à supprimer (cas 3 à 6 des tests ajoutés)
Test manquant ou à mettre à jour pour les conditions coloc et logement_chambre

@mtifarine
Copy link
Contributor Author

Merci @Morendil ,
Pour la correction de la condition 'coloc' et 'logement_chambre', j'ai supprimé une double application des coefficients "coloc" et "logement_chambre" qui sont déjà appliqués dans la formule "aide_logement_loyer_plafond".

@monbocal
Copy link
Contributor

monbocal commented Mar 5, 2019

Sur Mes Aides, nous avons date_arret_de_travail . Nous demandons cette date lorsque la personne se déclare en recherche d'emploi.

@Morendil
Copy link
Contributor

Morendil commented Mar 5, 2019

@monbocal Je suis allé voir comment se comporte Mes Aides.

Je réponds à la question "Quand s’est terminé votre dernier contrat de travail ?" avec par exemple 12/2018. Je suis ensuite allé regarder l'échange dans OpenFisca Tracer. Je fais l'hypothèse que ça correspond bien à ce qui est transmis à OpenFisca.

Pour date_arret_de_travail, je vois la valeur "2019-03-05" qui ne correspond pas à ma date de fin de contrat. Et je ne trouve pas de valeur passée à OpenFisca qui serait 2018-12 ou 2019-01.

Le fonctionnement actuel reste un peu flou pour moi. Et pour re-préciser ce qui me préoccupe: j'ai l'impression que si nous embarquons cette PR dans master en l'état, le risque d'une régression dans Mes Aides est important lorsque vous basculerez sur la version correspondante de France.

Si vous vous sentez à l'aise pour merger la PR et effectuer l'étude d'impact lorsque vous ferez la mise à jour de France, ça me va aussi et ça lèvera mon inquiétude sur ce sujet :)

@monbocal
Copy link
Contributor

monbocal commented Mar 5, 2019

Effectivement, j'ai répondu trop vite. date_arret_de_travail n'est pas en rapport avec la question posée et correspond à autre chose. J'ai dit une bêtise.

Je ne trouve pas non plus a priori la date concernée dans la simulation. Je vais regarder de plus prêt.

@Morendil Morendil force-pushed the msa_fiabilisation_generele_al_apl branch from 8d21f07 to f97a006 Compare March 5, 2019 11:31
@Morendil
Copy link
Contributor

Morendil commented Mar 5, 2019

@mtifarine J'ai rebasé la PR.

@Morendil
Copy link
Contributor

Morendil commented Mar 5, 2019

Vu IRL avec @monbocal : il semble que la réponse à "Quand s’est terminé votre dernier contrat de travail ?" n'est pas encore transmise à OpenFisca, il sera peut-être suffisant pour la mise à jour de la transmettre dans date_debut_chomage si la réponse à la question d'activité est "demandeur d'emploi".

Suite à cette analyse, @guillett @monbocal @alexsegura voyez-vous une objection à merger?

@Morendil Morendil merged commit 3505de5 into master Mar 11, 2019
@Morendil Morendil deleted the msa_fiabilisation_generele_al_apl branch March 11, 2019 10:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
contrib:msa Identification des sujets MSA
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants