Ipp 5 : Mise à jour de la réduction d'IR pour investissements forestiers #911

claireleroy · 2018-02-19T17:22:05Z

Les modifications de cette PR sont comparées à la branche ipp-update-tax-reductions_immobiliers (#910)

Évolution du système socio-fiscal.
Périodes concernées : à partir du 01/01/2002
Zones impactées :

openfisca_france/model/prelevements_obligatoires/impot_revenu/reductions_impots.py.
openfisca_france/model/prelevements_obligatoires/impot_revenu/variables_reductions_credits.py

Détails :
- Mise à jour des formules (2014-2016) de la réduction d'impôt 'invfor'
- Correction d'une erreur dans la formule précédente : les reports de dépenses entrent dans le plafonnement de la réduction
- TODO : une partie de la réduction d'impôt pour investissement forestier devient un crédit d'impôt en 2014 (Brochure Pratique IR revenus 2014, page 36)

Ces changements :

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

benjello

J'aurais dû le dire plus tôt mais tu peux faire un peu de nettoyage en enlevant les Pet en remplaçant par la feuille de l'arbre vers lequel ce P pointe.
Je vais faire un exemple en commentaire des changements.

Anna-Livia · 2018-02-28T15:24:12Z

openfisca_france/model/prelevements_obligatoires/impot_revenu/reductions_impot.py

@@ -1751,8 +1751,6 @@ class invfor(Variable):
    entity = FoyerFiscal
    label = u"invfor"
    definition_period = YEAR
-    end = '2013-12-31'


Est-ce qu'il est possible d'ajouter une référence pour cette variable ?

cf. PR #909

Anna-Livia · 2018-02-28T15:24:59Z

openfisca_france/model/prelevements_obligatoires/impot_revenu/reductions_impot.py

@@ -1820,7 +1818,7 @@ def formula_2011_01_01(foyer_fiscal, period, parameters):
        P = parameters(period).impot_revenu.reductions_impots.invfor

        max0 = max_(0, P.ifortra_seuil * (maries_ou_pacses + 1) - f7ul)
-        max1 = max_(0, max0 - f7uu + f7te + f7uv + f7tf)
+        max1 = max_(0, max0 - f7uu - f7te - f7uv - f7tf)
        return (P.taux * (


Il vaut mieux mettre les opérateurs (+ ) en début de ligne qu'en fin de ligne : https://www.python.org/dev/peps/pep-0008/#should-a-line-break-before-or-after-a-binary-operator

Anna-Livia · 2018-02-28T15:25:29Z

openfisca_france/model/prelevements_obligatoires/impot_revenu/reductions_impot.py

@@ -1820,7 +1818,7 @@ def formula_2011_01_01(foyer_fiscal, period, parameters):
        P = parameters(period).impot_revenu.reductions_impots.invfor

        max0 = max_(0, P.ifortra_seuil * (maries_ou_pacses + 1) - f7ul)
-        max1 = max_(0, max0 - f7uu + f7te + f7uv + f7tf)
+        max1 = max_(0, max0 - f7uu - f7te - f7uv - f7tf)


max0, max1 sont il assez explicites ?

C'est difficile de faire plus explicite, comme dans d'autres réductions, c'est simplement un moyen de respecter l'ordre d'imputation des dépenses éligibles à la réduction..

Anna-Livia · 2018-02-28T15:25:48Z

openfisca_france/model/prelevements_obligatoires/impot_revenu/reductions_impot.py

@@ -1846,7 +1844,7 @@ def formula_2012_01_01(foyer_fiscal, period, parameters):
        P = parameters(period).impot_revenu.reductions_impots.invfor

        max0 = max_(0, P.ifortra_seuil * (maries_ou_pacses + 1) - f7ul)
-        max1 = max_(0, max0 - f7uu + f7te + f7uv + f7tf)
+        max1 = max_(0, max0 - f7uu - f7te - f7uv - f7tf)


max0, max1 sont il assez explicites ?

cf. plus haut

Anna-Livia · 2018-02-28T15:26:03Z

openfisca_france/model/prelevements_obligatoires/impot_revenu/reductions_impot.py

@@ -1876,7 +1874,7 @@ def formula_2013_01_01(foyer_fiscal, period, parameters):
        P = parameters(period).impot_revenu.reductions_impots.invfor

        max0 = max_(0, P.ifortra_seuil * (maries_ou_pacses + 1) - f7ul)
-        max1 = max_(0, max0 - f7uu + f7te + f7uv + f7tf)
+        max1 = max_(0, max0 - f7uu - f7te - f7uv - f7tf)


max0, max1, ... sont il assez explicites ?

cf. plus haut

Anna-Livia · 2018-02-28T15:29:14Z

openfisca_france/model/prelevements_obligatoires/impot_revenu/variables_reductions_credits.py

+    value_type = int
+    unit = 'currency'
+    entity = FoyerFiscal
+    label = u"Investissements forestiers"


le label est le même que `f7ti, il y a t'il un moyen d'expliquer simplement la différence entre les deux ?

Anna-Livia · 2018-02-28T15:30:56Z

openfisca_france/model/prelevements_obligatoires/impot_revenu/variables_reductions_credits.py

@@ -2243,7 +2243,7 @@ class f7ui(Variable):
    value_type = int
    unit = 'currency'
    entity = FoyerFiscal
-    end = '2008-12-31'
+    # end = '2008-12-31' changes meaning in 2014


Est-ce que ça n'aurait pas plus de sens de mettre ce commentaire dans cerfa? Avec une mention de la variable qui est maintenant sur la même case.
Pourquoi avoir commenté le end?

Anna-Livia · 2018-02-28T15:31:39Z

openfisca_france/model/prelevements_obligatoires/impot_revenu/variables_reductions_credits.py

+    value_type = int
+    unit = 'currency'
+    entity = FoyerFiscal
+    label = u"Investissements forestiers"


le label est le même que `f7ti, il y a t'il un moyen d'expliquer simplement la différence entre les deux ?

Anna-Livia · 2018-02-28T15:31:44Z

openfisca_france/model/prelevements_obligatoires/impot_revenu/variables_reductions_credits.py

+    value_type = int
+    unit = 'currency'
+    entity = FoyerFiscal
+    label = u"Investissements forestiers"


le label est le même que `f7ti, il y a t'il un moyen d'expliquer simplement la différence entre les deux ?

Anna-Livia · 2018-02-28T15:31:50Z

openfisca_france/model/prelevements_obligatoires/impot_revenu/variables_reductions_credits.py

+    value_type = int
+    unit = 'currency'
+    entity = FoyerFiscal
+    label = u"Investissements forestiers"


le label est le même que `f7ti, il y a t'il un moyen d'expliquer simplement la différence entre les deux ?

claireleroy · 2018-03-21T16:19:17Z

@fpagnoux C'est normal que tu aies fermé ma PR ?

fpagnoux · 2018-03-21T16:28:25Z

Non c'est une erreur.
J'ai supprimé la branche ipp-update-tax-reductions-immobiliers en pensant qu'elle n'était pas utile parce que déjà mergée, et du coup ça a automatiquement fermé la PR.

Si tu as une version locale de ipp-update-tax-reductions-immobiliers, est-ce que tu peux la repousser sur le dépôt ?

claireleroy · 2018-03-21T17:02:34Z

C'est fait normalement

Anna-Livia · 2018-03-21T17:13:15Z

openfisca_france/model/prelevements_obligatoires/impot_revenu/variables_reductions_credits.py

+    value_type = int
+    unit = 'currency'
+    entity = FoyerFiscal
+    label = u"Investissements forestiers : report 2015, hors sinistre, avec adhésion à une association de producteurs


Les guillemets ne sont pas fermés :/

…or years 2014-2016

…not increase the ceiling of the tax reduction)

…rs_at)

…ion en 2014

benjello · 2018-03-27T10:19:41Z

Bravo @ClaireLeroyIPP !

claireleroy requested review from sandcha, Anna-Livia and benjello February 19, 2018 17:22

benjello approved these changes Feb 19, 2018

View reviewed changes

claireleroy mentioned this pull request Feb 19, 2018

Ipp 6 : Update réductions d'IR spécial DOM-TOM #912

Merged

claireleroy force-pushed the ipp-update-tax-credit-invfor branch from d02f38a to 39d7f98 Compare February 20, 2018 08:45

claireleroy force-pushed the ipp-update-tax-reductions-immobiliers branch from cadfa45 to 20a01ca Compare February 20, 2018 14:33

claireleroy force-pushed the ipp-update-tax-credit-invfor branch from 39d7f98 to e524eaf Compare February 20, 2018 15:27

Anna-Livia requested changes Feb 28, 2018

View reviewed changes

bonjourmauko added the flow:contribution-backlog label Mar 9, 2018

bonjourmauko assigned Anna-Livia Mar 9, 2018

claireleroy force-pushed the ipp-update-tax-reductions-immobiliers branch from 4fd1f41 to 61ccf9e Compare March 19, 2018 14:28

claireleroy mentioned this pull request Mar 19, 2018

Création de tests sur l'impôt sur le revenu #920

Closed

39 tasks

fpagnoux closed this Mar 21, 2018

fpagnoux removed the flow:contribution-backlog label Mar 21, 2018

Anna-Livia reopened this Mar 21, 2018

Anna-Livia reviewed Mar 21, 2018

View reviewed changes

claireleroy unassigned Anna-Livia Mar 23, 2018

Claire Leroy added 9 commits March 26, 2018 09:30

Update formula for tax reduction 'resimm' for year 2016

d69ca4c

Update formula for tax reduction 'scelli' in 2016 + Add new variables

38787de

Update formula, parameters and variables for tax reduction 'locmeu' f…

c13893b

…or years 2014-2016

Add variables of tax reduction 'invfor' 2014-2016

da1abeb

Add variables of tax reduction 'invfor' 2014-2016

f86f485

Fix error in formula of tax reduction 'invfor' (previous expenses can…

3fff174

…not increase the ceiling of the tax reduction)

Update formula for tax reduction 'invfor' for 2014-2016

07dd8ec

No 'end_date'

7de0447

Correct reminding errors after rebase (no more simulation and paramte…

595a4e1

…rs_at)

Claire Leroy added 6 commits March 26, 2018 09:38

Remove useless things

793eab5

Changements esthétiques - "+" en début de ligne

6c3d32d

Explicite les labels

9272c19

Séparation de la case 7UI en deux à cause du changement de significat…

9bece0d

…ion en 2014

Correction erreurs labels

c4293bf

Correction erreurs labels

32b2597

claireleroy force-pushed the ipp-update-tax-credit-invfor branch from ea6572a to 32b2597 Compare March 26, 2018 07:38

Update CHANGELOG et setup.py

e10dc21

bonjourmauko added the flow:contribution-backlog label Mar 26, 2018

claireleroy added 3 commits March 26, 2018 14:05

Correction erreur label

ae6e9fe

Correction erreur après rebase

e8123d8

Correction erreur

b393f91

claireleroy changed the base branch from ipp-update-tax-reductions-immobiliers to master March 26, 2018 15:53

bonjourmauko added contrib:ipp kind:evolution scope:tax labels Mar 26, 2018

Anna-Livia approved these changes Mar 27, 2018

View reviewed changes

Anna-Livia added flow:contrib:doing and removed flow:contrib:backlog labels Mar 27, 2018

Anna-Livia self-assigned this Mar 27, 2018

claireleroy merged commit 4b9d15e into master Mar 27, 2018

MattiSG removed the flow:contrib:doing label Mar 27, 2018

benjello deleted the ipp-update-tax-credit-invfor branch March 30, 2018 14:08

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Ipp 5 : Mise à jour de la réduction d'IR pour investissements forestiers #911

Ipp 5 : Mise à jour de la réduction d'IR pour investissements forestiers #911

claireleroy commented Feb 19, 2018

benjello left a comment

Anna-Livia Feb 28, 2018

claireleroy Mar 12, 2018

Anna-Livia Feb 28, 2018

Anna-Livia Feb 28, 2018

claireleroy Mar 20, 2018

Anna-Livia Feb 28, 2018

claireleroy Mar 20, 2018

Anna-Livia Feb 28, 2018

claireleroy Mar 20, 2018

Anna-Livia Feb 28, 2018

Anna-Livia Feb 28, 2018

Anna-Livia Feb 28, 2018

Anna-Livia Feb 28, 2018

Anna-Livia Feb 28, 2018

claireleroy commented Mar 21, 2018

fpagnoux commented Mar 21, 2018

claireleroy commented Mar 21, 2018

Anna-Livia Mar 21, 2018

benjello commented Mar 27, 2018

Ipp 5 : Mise à jour de la réduction d'IR pour investissements forestiers #911

Ipp 5 : Mise à jour de la réduction d'IR pour investissements forestiers #911

Conversation

claireleroy commented Feb 19, 2018

benjello left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

claireleroy commented Mar 21, 2018

fpagnoux commented Mar 21, 2018

claireleroy commented Mar 21, 2018

Choose a reason for hiding this comment

benjello commented Mar 27, 2018