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

Ipp 4 bis : Udpate réductions pour investissements immobiliers - merge sur master #950

Merged
merged 44 commits into from
Mar 20, 2018

Conversation

claireleroy
Copy link
Contributor

@claireleroy claireleroy commented Mar 20, 2018

Connected to #910

  • Évolution du système socio-fiscal.
  • Périodes concernées : principalement à partir du 01/01/2014
  • Zones impactées :
  • openfisca_france/model/prelevements_obligatoires/impot_revenu/reductions_impots.py.
  • openfisca_france/model/prelevements_obligatoires/impot_revenu/variables_reductions_credits.py
  • openfisca_france/parameters/impot_revenu/reductions_impots/ : rpinel, locmeu, scelli
  • Détails :
    • Mise à jour des formules (2014-2016) des réductions d'impôts : 'resimm' (réduction d'impot Malraux), 'locmeu' (réduction d'impôt Censi-Bouvard), 'scelli' (réduction d'impôt Scellier) et 'duflot (réduction d'impôt Duflot) qui sont toutes des réductions d'impôts portant sur les investissements immobiliers.
    • Correction de la formule de la réduction Duflot en 2013 (plafonnement)
    • Renommage de paramètres de taux de réduction de manière plus explicite
    • Ajout de la réduction d'impôt Pinel crée en 2014 (variabl 'rpinel') + Paramètres associés
    • Ajout d'inputs variables associées à ces réductions (i.e les nouvelles cases des déclarations fiscales, ex: f7ek)

Ces changements :

  • Ajoutent une fonctionnalité (par exemple ajout d'une variable).
  • Corrigent ou améliorent un calcul déjà existant.

Claire Leroy added 30 commits March 19, 2018 14:13
…ies to both métropole and outre-mer investments together
…Update formula for tax reduction 'duflot' (common tax base ceiling with 'rpinel')
…household tax deduction on financial profit and loss)
…e can assume they will go on in 2017 without further info
@claireleroy claireleroy merged commit 078d34a into master Mar 20, 2018
@fpagnoux
Copy link
Member

fpagnoux commented Mar 20, 2018

@Anna-Livia @ClaireLeroyIPP @benjello c'est normal que cette PR supprime tout le dossier de test tests/calculateur_impots/ sans pour autant en ajouter de nouveaux pour les remplacer ?

@claireleroy
Copy link
Contributor Author

claireleroy commented Mar 21, 2018

@fpagnoux Oui, ce sont de très vieux tests qui vont être redondants avec les nouveaux tests que j'ai créés et qui arriveront dans la dernière PR IPP. Là tout de suite ils posaient problème et ne passaient pas, à cause de la décision de dissocier les cases qui changent de signification d'une année à l'autre.
Exemple : la case 7PB se dissocie en f7pb_2012 (pour la période jusqu'en 2012) et f7pb (pour la nouvelle signification à partir de 2014), du coup le test dans tests/calculateur_impots/f7pb ne passe plus car il teste uniquement f7pb...
J'avais demandé à @Anna-Livia et @benjello si je pouvais les supprimer, ils étaient ok.

@fpagnoux
Copy link
Member

Ok pas de problème, j'étais juste surpris que les tests s'exécutent en 6 minutes et je voulais être sûr que c'était pas une erreur de rebase :)

@benjello
Copy link
Member

Faudrait quand même qu'on les garde et qu'on renomme les variables. J'ai peur que se cachent des tests qu'on ne peut plus regénérer via le calculateur des impôts en ligne (année antérieures)

@benjello
Copy link
Member

Mais bon go @ClaireLeroyIPP

@fpagnoux fpagnoux deleted the ipp-update-tax-reductions-immobiliers branch March 21, 2018 15:42
@claireleroy claireleroy restored the ipp-update-tax-reductions-immobiliers branch March 21, 2018 17:01
@claireleroy claireleroy deleted the ipp-update-tax-reductions-immobiliers branch March 27, 2018 11:37
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