Ipp 11 : Correction du calcul des revenus non salariaux pour l'IR #917

claireleroy · 2018-02-21T13:17:16Z

Les modifications de cette PR sont comparées à la branche ipp-update-tax-credit-aidper (#916)

Correction du système socio-fiscal
Périodes concernées : toutes

openfisca_france/model/prelevements_obligatoires/impot_revenu/ir.py.
openfisca_france/model/revenus/activite/non_salarie.py
openfisca_france/parameters/impot_revenu/rpns/micro/specialbnc/max.yaml
openfisca_france/parameters/impot_revenu/rpns/micro/specialbnc/min.yaml

Détails :
- Correction de la formule des 'rpns'
- Distinction des moins-values des revenus non salariaux "professionnels" (variable 'rpns_mvct_pro') de celles des revenus non salariaux "non professionnels" ('rpns_mvct_nonpro'), car seuls les premiers peuvent s'imputer sur le revenu global
- Création d'une variable intermédiaire 'rpns_frag' : revenus non salariaux relevant du forfait agricole. Celui-ci disparait en 2016 et est remplacé par le régime "micro bénéfice agricole" (A FAIRE)
- Correction du calcul de l'abattement appliqué aux revenus non commerciaux relevant du régime micro-bnc (34%) + Ajout de nouveaux paramètres associés à cet abattement
- Modification des labels et end_date de variables de revenus non salariaux dans non_salarie.py

Ces changements :

Ajoutent une fonctionnalité (par exemple ajout d'une variable).
Corrigent ou améliorent un calcul déjà existant.
Impactent l'API publique d'OpenFisca France (par exemple renommage ou suppression de variables).

benjello · 2018-02-21T13:48:54Z

openfisca_france/model/prelevements_obligatoires/impot_revenu/ir.py

+
+        return frag_impo
+
+    def formula_2016_01_01(individu, period, parameters):


Utiliser end plutôt que renvoyer 0

benjello · 2018-02-21T13:49:14Z

openfisca_france/model/prelevements_obligatoires/impot_revenu/ir.py

@@ -2499,7 +2555,7 @@ def abat_rpns(rev, P):

        # # E revenus non commerciaux non professionnels
        # regime déclaratif special ou micro-bnc
-        mncn_timp = abat_rpns(mncn_impo, micro.specialbnc.services)  # TODO check
+        mncn_timp = abat_rpns(mncn_impo,  micro.specialbnc)


espace en trop

benjello · 2018-02-21T13:49:29Z

openfisca_france/model/prelevements_obligatoires/impot_revenu/ir.py

-        RPNS = (rev_NS + rev_NS_mi + rpns_pvct + exon + abic_impm - abic_defm + alnp_imps + cncn_aimp - nbic_mvct)
+        exon_acc = max_(0, macc_timp + nacc_timp - macc_mvct) - macc_timp - nacc_timp  # ajout artificiel
+        exon_ncn = max_(0, mncn_timp - mncn_mvct) - mncn_timp  # ajout artificiel
+        RPNS = (rev_NS + rev_NS_mi + rpns_pvct + exon_acc + exon_ncn + abic_impm - abic_defm + alnp_imps + cncn_aimp - rpns_mvct_pro)


Ne pas utiliser de majuscules

Anna-Livia · 2018-03-06T14:57:59Z

openfisca_france/model/prelevements_obligatoires/impot_revenu/ir.py

@@ -2329,10 +2328,10 @@ def formula(individu, period, parameters):
        return frag_pvct + macc_pvct + mbic_pvct + mbnc_pvct + mncn_pvct


-class rpns_mvct(Variable):
+class rpns_mvct_nonpro(Variable):


Est-il possible de donner un nom plus explicite à cette variable ? moins_values_ct_non_pro ou moins_values_court_terme_non_pro

Anna-Livia · 2018-03-06T15:00:01Z

openfisca_france/model/prelevements_obligatoires/impot_revenu/ir.py

+    definition_period = YEAR
+
+    def formula(individu, period, parameters):
+        """Moins values de court terme


Aller à la ligne

J'ai viré ces descriptions finalement parce qu'elles me paraissent redondantes avec le label

Anna-Livia · 2018-03-06T15:01:44Z

openfisca_france/model/prelevements_obligatoires/impot_revenu/ir.py

@@ -2373,6 +2404,29 @@ def formula(individu, period, parameters):
        return mbic_mvlt + macc_mvlt + mbnc_mvlt + mncn_mvlt


+class rpns_frag(Variable):


Est-il possible de donner un nom plus explicite à cette variable ? rnps_revenus_forfait_agricoles ou revenus_forfait_agricoles

Anna-Livia · 2018-04-06T08:25:20Z

openfisca_france/model/prelevements_obligatoires/impot_revenu/credits_impot.py

-                P.taux_wj * min_(f7wj, max1)  + P.taux_wi * min_(f7wi, max2))
+        max1 = max_(0, max0 - f7wj) # f7wj s'impute avant f7wl et f7wi
+        max2 = max_(0, max1 - f7wi) # f7wi s'impute avant f7wl
+        return (


proposition :
return (
P.taux_wr * f7wr
+ P.taux_wj * min_(f7wj, max0)
+ P.taux_wi * min_(f7wi, max1)
+ P.taux_wl * min_(f7wl, max2)
+ P.taux_wl * min_(max_(0,f7wl - max2), max00)
)

Cf. PR #916

Anna-Livia · 2018-04-06T08:28:03Z

openfisca_france/model/prelevements_obligatoires/impot_revenu/credits_impot.py


-        return (P.taux_wr * f7wr + P.taux_wl * min_(f7wl, max00) + P.taux_wl * max_(f7wl - max00, 0) + P.taux_wj *
-                min_(f7wj, max1))
+        return (


Proposition :
return (
P.taux_wr * f7wr
+ P.taux_wj * min_(f7wj, max0)
+ P.taux_wl * min_(f7wl, max1)
+ P.taux_wl * min_(max_(0,f7wl - max1), max00)
)

Cf. PR #916

Anna-Livia · 2018-04-06T10:34:36Z

openfisca_france/model/prelevements_obligatoires/impot_revenu/variables_reductions_credits.py

@@ -707,7 +979,7 @@ class f7wv(Variable):
    entity = FoyerFiscal
    label = u"Dépenses en faveur de la qualité environnementale de l'habitation principale: achat de volets en 2012"
 #    start_date = date(2012, 1, 1)
-    end = '2012-12-31'
+#    end = '2012-12-31' changes meaning in 2015


cf discussions sur ces expressions dans les PR précédentes

Cf. PR #913

Anna-Livia · 2018-04-06T10:38:44Z

openfisca_france/model/prelevements_obligatoires/impot_revenu/variables_reductions_credits.py

 #    start_date = date(2013, 1, 1)
    definition_period = YEAR


-class fhsn(Variable):
-    cerfa_field = u"HSN"
+# Investissements OUTRE-MER dans le secteur du logement et autres secteurs d’activité


Les cases ci-dessous n'ont pas de label. Est-il possible d'en rajouter un, même si il est très large ?

Cf. PR #912

Anna-Livia · 2018-04-06T10:39:03Z

openfisca_france/model/prelevements_obligatoires/impot_revenu/variables_reductions_credits.py

    value_type = int
    unit = 'currency'
    entity = FoyerFiscal
-    label = u"Investissements outre-mer dans le cadre de l'entreprise : Autres investissements, Investissements dans votre entreprise avec exploitation directe, montant de la réduction d’impôt dont vous demandez l’imputation en 2013, en 2010"
-#    start_date = date(2013, 1, 1)
+    label = u"Cotisations pour la défense des forêts contre l'incendie "


Anna-Livia · 2018-04-06T10:40:00Z

openfisca_france/model/prelevements_obligatoires/impot_revenu/variables_reductions_credits.py

+    label = u"Investissements locatifs neufs dispositif Scellier : Reports concernant les investissements achevés ou acquis au cours des années antérieures, Investissements achevés en 2013 et réalisés du 1.1.2013 au 31.3.2013 avec promesse d'achat en 2012, en métropole, dans les DOM, à Saint-Barthélemy, Saint-Martin, Saint-Pierre-et-Miquelon"
+#    start_date = date(2014, 1, 1)
+    definition_period = YEAR
+


Il faut mettre 2 espaces entre les classes :)

Cf. PR #910

Anna-Livia · 2018-04-06T10:41:06Z

openfisca_france/parameters/impot_revenu/credits_impot/quaenv/max_rfr.yaml

+  2009-01-01:
+    value: 45000.0
+  2012-01-01:
+    value: 30000.0


ajouter une ligne vide à la fin :)

…ic_impm, nbic_impm)

…nnal income (rpns_mvct_nonpro) can be imputed on global income (NB : not the case for long-run capital loss (rpns_mvlt) or for short-run net losses of non-professionnal income (rpns_mvct) that can only be imputed on similar incomes

…'mncn_impo'

…n 2016 TODO : code the new system of "micro BA" that replaced it

… already taken into account in 'rpns_individu'

Everything is fixed :)

Anna-Livia · 2018-04-06T15:30:19Z

CHANGELOG.md

+* Correction du système socio-fiscal.
+* Périodes concernées : toutes
+* Zones impactées :
+  - `openfisca_france/model/prelevements_obligatoires/impot_revenu/ir`


Générallement, on omet openfisca_france/model/des chemins dans le contributing cf #956

Changements déjà pris en compte

claireleroy requested review from benjello, sandcha and Anna-Livia February 21, 2018 13:17

benjello previously requested changes Feb 21, 2018

View reviewed changes

claireleroy mentioned this pull request Feb 21, 2018

Ipp 12 : Update calcul du quotient familial, ajout nouvelle réduction sous conditions de revenus #918

Merged

bonjourmauko added the flow:contrib:backlog label Apr 5, 2018

claireleroy force-pushed the ipp-update-tax-credit-aidper branch from 0a2dc73 to f94ab8b Compare April 6, 2018 07:32

This was referenced Apr 6, 2018

Ajout du régime des micro-bénéfices agricoles (depuis 2016 ) #963

Closed

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

Closed

Anna-Livia previously requested changes Apr 6, 2018

View reviewed changes

Claire Leroy and others added 20 commits April 6, 2018 16:25

Correct reminding errors after rebase

a80fcc7

Fix indentation and parameters

4532bb7

Update stop date of variables of professional housing rent income (ab…

ab4d405

…ic_impm, nbic_impm)

Correct stop dates of some income variables (auto-entrepreneur)

2c22038

Correct mistake with 'mbnc_impo' (34% tax deduction not 50%)

f38e367

Correct mistake in 'rpns' formula : tax reduction is 34% not 50% for …

2e7d69d

…'mncn_impo'

Update income tax formula - the "forfait agricole" system disapears i…

3da0be5

…n 2016 TODO : code the new system of "micro BA" that replaced it

Correct formula for 'rev_cat_rpns' - short-run losses 'mbic_mvct' now…

e8aeda1

… already taken into account in 'rpns_individu'

Update 'rpns_individu' : clean + add stop date

f43485b

Update stop dates of income variables

109c0be

Fix minor issues of end_date

eeff7a7

Update la date de fin de la variable 'rpns_frag'

b83b6cb

Apporte des corrections sur la présentation du code

9fe4086

Explicite les noms de variables - revenus non salariaux

1e5c87f

Explicite les noms de variables - revenus non salariaux

d890a0a

Améliore l'indentation

30444dd

Simplifie le calcul des plus-values

15eb39a

Supprime tous les _P inutiles

1d8b3c7

Explicite le nom de 'rfr_pv'

13fe996

claireleroy added 4 commits April 6, 2018 16:35

Rends la formule du RFR plus lisible

0f57653

Economise une ligne

662686d

Ajoute une ligne à la fin du fichier de paramètres

107dc93

Explicite le nom de la variable 'rpns_mvlt'

bb2995a

claireleroy force-pushed the ipp-update-rpns branch from e2e9f8a to bb2995a Compare April 6, 2018 14:35

Update CHANGELOG et setup.py

2b9718f

Anna-Livia changed the base branch from ipp-update-tax-credit-aidper to master April 6, 2018 15:26

Anna-Livia approved these changes Apr 6, 2018

View reviewed changes

Update CHANGELOG

b41aa85

claireleroy merged commit 3f1bcf9 into master Apr 6, 2018

MattiSG removed the flow:contrib:backlog label Apr 6, 2018

claireleroy deleted the ipp-update-rpns branch April 9, 2018 13:45

bonjourmauko added contrib:ipp kind:evolution labels Jul 23, 2018

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Ipp 11 : Correction du calcul des revenus non salariaux pour l'IR #917

Ipp 11 : Correction du calcul des revenus non salariaux pour l'IR #917

claireleroy commented Feb 21, 2018

benjello Feb 21, 2018

benjello Feb 21, 2018

benjello Feb 21, 2018

Anna-Livia Mar 6, 2018

Anna-Livia Mar 6, 2018

claireleroy Apr 6, 2018

Anna-Livia Mar 6, 2018

Anna-Livia Apr 6, 2018

claireleroy Apr 6, 2018

Anna-Livia Apr 6, 2018

claireleroy Apr 6, 2018

Anna-Livia Apr 6, 2018

claireleroy Apr 6, 2018

Anna-Livia Apr 6, 2018

claireleroy Apr 6, 2018

Anna-Livia Apr 6, 2018

Anna-Livia Apr 6, 2018

claireleroy Apr 6, 2018

Anna-Livia Apr 6, 2018

Anna-Livia Apr 6, 2018


		return frag_impo

		def formula_2016_01_01(individu, period, parameters):

		@@ -2373,6 +2404,29 @@ def formula(individu, period, parameters):
		return mbic_mvlt + macc_mvlt + mbnc_mvlt + mncn_mvlt


		class rpns_frag(Variable):

Ipp 11 : Correction du calcul des revenus non salariaux pour l'IR #917

Ipp 11 : Correction du calcul des revenus non salariaux pour l'IR #917

Conversation

claireleroy commented Feb 21, 2018

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

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