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 incrément nombre de parts supplémentaires pour taux CSG revenus remplacement #2030

Merged
merged 5 commits into from
Feb 23, 2023

Conversation

pzuldp
Copy link
Contributor

@pzuldp pzuldp commented Feb 16, 2023

  • Amélioration technique.
  • Périodes concernées : toutes.
  • Zones impactées : model/prelevements_obligatoires/prelevements_sociaux/contributions_sociales/remplacement.py
  • Détails :
    • Correction d'une erreur dans la prise en compte de l'incrément par demi part supplémentaire pour la détermination des taux plein ou réduit ou intermédiaire de CSG sur revenus de remplacement

Ces changements (effacez les lignes ne correspondant pas à votre cas) :

  • Corrigent ou améliorent un calcul déjà existant.

Quelques conseils à prendre en compte :

Et surtout, n'hésitez pas à demander de l'aide ! :)

@pzuldp pzuldp marked this pull request as ready for review February 16, 2023 16:17
@pzuldp pzuldp changed the title corrige incrément nombre de parts supplémentaires corrige incrément nombre de parts supplémentaires pour taux CSG revenus remplacement Feb 16, 2023
@benjello benjello self-requested a review February 16, 2023 16:26
Copy link
Member

@benjello benjello left a comment

Choose a reason for hiding this comment

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

L'idéal serait d'ajouter des tests.
Il serait peut-être utile de rassembler les définitions des seuils dans une seule fonction pour réduire la redondance du code et donc les risques d'erreurs.

@pzuldp
Copy link
Contributor Author

pzuldp commented Feb 16, 2023

L'idéal serait d'ajouter des tests. Il serait peut-être utile de rassembler les définitions des seuils dans une seule fonction pour réduire la redondance du code et donc les risques d'erreurs.

Bonne idée. J'ai justement une petite série de tests tous chauds sortis d'openfisca-france-data qui sont le symétrique de ce dont on a besoin ici haha
Selon mes dispos ces prochains jours je rajouterai la fonction.

@pzuldp pzuldp marked this pull request as draft February 16, 2023 17:20
@pzuldp
Copy link
Contributor Author

pzuldp commented Feb 17, 2023

L'idéal serait d'ajouter des tests. Il serait peut-être utile de rassembler les définitions des seuils dans une seule fonction pour réduire la redondance du code et donc les risques d'erreurs.

Bonne idée. J'ai justement une petite série de tests tous chauds sortis d'openfisca-france-data qui sont le symétrique de ce dont on a besoin ici haha Selon mes dispos ces prochains jours je rajouterai la fonction.

Tu as eu raison de me demander des tests @benjello : j'ai identifié une autre erreur dans l'inversion (on n'inverse pas l'abattement pour frais pro sur les alloc chômage !)

Copy link
Contributor

@clallemand clallemand left a comment

Choose a reason for hiding this comment

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

Merci pour cette correction !

@pzuldp pzuldp marked this pull request as ready for review February 21, 2023 14:05
@pzuldp pzuldp merged commit 8482c3f into master Feb 23, 2023
@pzuldp pzuldp deleted the correction_csg_remplacement branch February 23, 2023 10:50
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