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

Mise à jour des barèmes IPP pour l'impot sur le revenu #906

Merged
merged 12 commits into from
Feb 27, 2018

Conversation

Anna-Livia
Copy link
Contributor

@Anna-Livia Anna-Livia commented Feb 19, 2018

Modifications de @ClaireLeroyIPP

  • Évolution du système socio-fiscal.
  • Périodes concernées : principalement, mise à jour avec les barèmes 2016, avec quelques corrections sur 2012-2013 et 2015.
  • Zones impactées : openfisca_france/parameters/impot_revenu/.
  • Détails :
    • Ajout des barèmes IPP 2016 pour l’impôt sur le revenu,
    • Correction de quelques paramêtres.

Ces changements corrigent ou améliorent un calcul déjà existant.

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.

Je n'ai aucune raison de ne pas considérer ces changements comme bons.
J'imagine que les séparer des tests est voulu ;-)

@Anna-Livia
Copy link
Contributor Author

Je propose d'ajouter des références :)

@benjello
Copy link
Member

benjello commented Feb 19, 2018

@Anna-Livia : pourquoi pas mais cela devrait arriver un jour ou l'autre avec les barèmes IPP ( #905).

@Anna-Livia
Copy link
Contributor Author

@benjello Je trouve cela dommage de pas mettre les références dès maintenant. Une mention u"barèmes IPP + URL" pourrait être chouette en première étape.

@claireleroy
Copy link
Contributor

@Anna-Livia Si ça peut faire avancer la PR plus vite, je les rajoutent! Mais c'est vrai que tout est déjà dans les barèmes IPP, qui devraient idéalement permettre un update automatique des paramètres YAML d'Openfisca comme suggéré par @benjello (au lieu de rajouter à la main les paramètres des nouvelles années et les références...)

@benjello
Copy link
Member

@Anna-Livia : juste la mention barème IPP avec ce lien t'irait ?

@claireleroy
Copy link
Contributor

@Anna-Livia @benjello Donc tout est bon, on peut merger ? (Question : est-ce que j'ai le droit d'appuyer sur "Merge Pull Request" moi même ?)

@Anna-Livia
Copy link
Contributor Author

@benjello
oui pour la mention et le lien --> c'est nickel
@ClaireLeroyIPP -->
Il vaut mieux attendre un consensus, et que les reviewer ait donnée tous leur GO.
Dernière remarque, il me semble que c'est un changement mineur, plutot que bugfix

@benjello
Copy link
Member

@ClaireLeroyIPP : dès que @sandcha a regardé on pourra merger

@Anna-Livia
Copy link
Contributor Author

@benjello @ClaireLeroyIPP C'est bon pour moi, vous mergez quand vous voulez.
@benjello quand je disais tous les reviewers, je parlais surtout de ceux qui avait commencé à faire des review. Ok pour que @sandcha passe un coup dessus aussi :)

@@ -1,5 +1,5 @@
description: Abattement minimum - Demandeur d'emploi
reference: ipp
reference: Barèmes IPP - https://www.ipp.eu/wp-content/uploads/2017/07/baremes-ipp-impot-revenu-income-tax.xlsx
Copy link
Contributor Author

Choose a reason for hiding this comment

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

🙌

@fpagnoux fpagnoux force-pushed the ipp-update-baremes branch 2 times, most recently from 10f924e to 879d6f0 Compare February 27, 2018 00:15
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.

4 participants