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 les periods dans calculs de ressources pour le RFR et le RNI #197

Merged
merged 4 commits into from
Dec 12, 2023

Conversation

Allan-CodeWorks
Copy link
Contributor

Descrition

Les calculs de ressources RFR et RNI sont basée sur les chiffre donnés sur les avis d'impositions.
La majorité du temps c'est en n-2.
Certaines formules ont été écrites avec d'autres période, cette PR corrige cela.

@Allan-CodeWorks Allan-CodeWorks requested a review from a team December 11, 2023 17:16
rni:
2022: 50000
2021: 50000
Copy link
Contributor

Choose a reason for hiding this comment

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

Du coup là on est sur du n-1 mais dans la modélisation de cette aide on récupère le rni de l'année n-2 ? Je veux bien des éclaircissements s'il te plait

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oula, oui effectivement, c'est une erreur de ma part qui je pense est couverte par un mécanisme openfisca qui fait un calcul de ressource des années suivantes approximatif.
C'est modifié, merci pour ta vigilance \o/

@Allan-CodeWorks Allan-CodeWorks self-assigned this Dec 12, 2023
@Allan-CodeWorks Allan-CodeWorks requested a review from a team December 12, 2023 14:00
@Allan-CodeWorks
Copy link
Contributor Author

Hier j'aurai dû mettre la PR en draft car je n'avais pas testé et ça m'aurait probablement permis de voir le problème signalé par @Shamzic.
J'ai maitenant testé les différents dispositifs en faisant un test complet avec le Simulateur AJ, les aides s'affichent avec les bons montant, les données du tracer sont cohérentes donc je vous soumets la PR en review.

Pour tester correctement le comportement de l'aide à la garde d'enfant des hauts de france il faut :

  • tester avec la modifs de la PR qui passe le RNI en période fiscale
  • réintégrer l'aide dans le simulateur! (on l'avait revert et on ne l'a jamais remise.... CF ce commit perdu pour ajouter le fichier dans le dossier openfisca, et il faut aussi supprimer le fichier équivalent qui est dans le dossier javascript)

Copy link
Contributor

@Shamzic Shamzic left a comment

Choose a reason for hiding this comment

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

Ok pour moi, testé sur le simulateur avec l'aide au permis de la MSA haute normandie

@Allan-CodeWorks Allan-CodeWorks merged commit d3d9a8d into master Dec 12, 2023
9 checks passed
@Allan-CodeWorks Allan-CodeWorks deleted the corrige_periods_ressources_n-2 branch December 12, 2023 16:43
@guillett guillett added this to the Passé milestone Mar 22, 2024
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