Ipp 8 : Update réduction et crédit IR pour emploi à domicile #914

claireleroy · 2018-02-20T15:25:27Z

Les modifications de cette PR sont comparées à la branche ipp-update-tax-reduction-quaenv (#913)

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

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

Détails :
- Mise à jour des formules (2014-2016) du crédit d'impôt 'saldom2' et de la réduction d'impôt 'saldom'
- Correction de la formule du crédit d'IR (sur la majoration pour nombre d'ascendants sd eplus de 65 bénéficiaires de l'APA et pour lesquels des dépenses d'emploi à domicile ont été engagées - case 7DL)
- Ajout de la case 7DD dans les inputs variables

Ces changements :

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

benjello · 2018-02-20T15:39:11Z

openfisca_france/model/prelevements_obligatoires/impot_revenu/reductions_impot.py

+        Sommes versées pour l'emploi d'un salariés à  domicile
+        2011 -
+        '''
+        nb_pac2 = simulation.calculate('nb_pac2', period)


Virer ces simulation.calculate

benjello · 2018-02-20T15:39:21Z

openfisca_france/model/prelevements_obligatoires/impot_revenu/reductions_impot.py

+        f7dl = simulation.calculate('f7dl', period)
+        f7dq = simulation.calculate('f7dq', period)
+        f7dg = simulation.calculate('f7dg', period)
+        _P = simulation.parameters_at(period.start)


Virer ce _P

benjello · 2018-02-20T15:39:39Z

openfisca_france/model/prelevements_obligatoires/impot_revenu/reductions_impot.py

+        f7dq = simulation.calculate('f7dq', period)
+        f7dg = simulation.calculate('f7dg', period)
+        _P = simulation.parameters_at(period.start)
+        P = simulation.parameters_at(period.start).impot_revenu.reductions_impots.salarie_domicile


Remplacer ce P par salarie_domicile

Anna-Livia · 2018-03-06T14:00:49Z

openfisca_france/model/prelevements_obligatoires/impot_revenu/reductions_impot.py

+        f7jy = foyer_fiscal('f7jy', period)
+        P = parameters(period).impot_revenu.reductions_impots.locmeu
+
+        m18 = (maxi(f7id, f7ie, f7if, f7ig) == max_(f7ie, f7if))


Il y a t'il un nom de variable plus explicite pour m18et m20?

Cf. PR #910

Anna-Livia · 2018-03-06T14:03:50Z

openfisca_france/model/prelevements_obligatoires/impot_revenu/reductions_impot.py

+
+        m18 = (maxi(f7id, f7ie, f7if, f7ig) == max_(f7ie, f7if))
+        m20 = (maxi(f7ij, f7il, f7in, f7iv) == max_(f7il, f7in))
+        return ((min_(P.max, maxi(f7ij, f7il, f7in, f7iv)) * (P.taux20 * m20 + P.taux18 * not_(m20)) +


concernant P.taux20et consorts, Si le taux n'est pas amené à changer, il me semble qu'il vaut mieux le mettre en dur dans la formule. S'il peut changer, un nom de paramètre plus générique me semble de mise.

Anna-Livia · 2018-03-06T14:06:43Z

openfisca_france/model/prelevements_obligatoires/impot_revenu/reductions_impot.py

+
+        m18 = (maxi(f7id, f7ie, f7if, f7ig) == max_(f7ie, f7if))
+        m20 = (maxi(f7ij, f7il, f7in, f7iv) == max_(f7il, f7in))
+        return ((min_(P.max, maxi(f7ij, f7il, f7in, f7iv)) * (P.taux20 * m20 + P.taux18 * not_(m20)) +


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

Cf. PR #910

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

openfisca_france/model/prelevements_obligatoires/impot_revenu/reductions_impot.py

        2002-2004
        '''
        f7df = foyer_fiscal('f7df', period)
-        f7dg = foyer_fiscal('f7dg', period)
+        isinvalid = foyer_fiscal('f7dg', period)


proposition: invalide

Anna-Livia · 2018-03-06T14:08:45Z

openfisca_france/model/prelevements_obligatoires/impot_revenu/reductions_impot.py

+        Sommes versées pour l'emploi d'un salariés à domicile
+        2011 -
+        '''
+        nb_pac2 = foyer_fiscal('nb_pac2', period)


Il y aurait-il un nom de variable plus explicite ?

Je change en "nb_pac_majoration_plafond" comme dans PR #909

Anna-Livia · 2018-03-06T14:12:56Z

openfisca_france/model/prelevements_obligatoires/impot_revenu/reductions_impot.py

+
+        nbpacmin = nb_pac2 + f7dl
+        maxBase = P.max1 * not_(annee1) + P.max1_1ereAnnee * annee1
+        maxDuMaxNonInv = P.max2 * not_(annee1) + P.max2_1ereAnnee * annee1


En python, il est plutôt recommandé d'utiliser du "camel case" c'est à dire de plutot écrire max_du_max_non_inv https://www.python.org/dev/peps/pep-0008/#id45

… related

…or years 2014-2016

…Update formula for tax reduction 'duflot' (common tax base ceiling with 'rpinel')

…n 2013 - f7.. variables were replaced by fh.. variables

…4-2016

…vious formula (ceiling applies to tax base not tax credit) + update parameters

…se of the ceiling due to f7dl only applies to tax reduction not tax credit

benjello · 2018-04-04T21:06:15Z

@ClaireLeroyIPP : il va te falloir éditer le CHANGELOG avant de merger.
Il faudra virer les .pydes fichiers impactés mentionnées dans le premier post de cette conversation.

Everything is fixed

claireleroy requested review from benjello, sandcha and Anna-Livia February 20, 2018 15:25

benjello requested changes Feb 20, 2018

View reviewed changes

claireleroy force-pushed the ipp-update-tax-reduction-quaenv branch from 68feb62 to 2f6f820 Compare February 20, 2018 17:58

claireleroy force-pushed the ipp-update-tax-reduc-and-credit-saldom branch from 0cbcb28 to 2e19b09 Compare February 20, 2018 18:00

claireleroy mentioned this pull request Feb 21, 2018

Ipp 9 : Mise à jour de la réduction d'impôt PME non cotées (2014-2016) #915

Merged

Anna-Livia previously requested changes Mar 6, 2018

View reviewed changes

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

claireleroy force-pushed the ipp-update-tax-reduction-quaenv branch 3 times, most recently from b4debc3 to 70934e2 Compare April 4, 2018 12:59

claireleroy force-pushed the ipp-update-tax-reduc-and-credit-saldom branch from f86edef to 8b9973b Compare April 4, 2018 16:41

claireleroy changed the base branch from ipp-update-tax-reduction-quaenv to master April 4, 2018 16:42

Claire Leroy and others added 16 commits April 4, 2018 18:43

Update formula for tax reduction 'scelli' in 2015 + add new variables…

0781df6

… related

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

3e13c19

…or years 2014-2016

Add new tax reduction 'rpinel' (dispositif Pinel) for 2014 onwards + …

7791bee

…Update formula for tax reduction 'duflot' (common tax base ceiling with 'rpinel')

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

4119a41

Correct mistake in formula and variables for tax reduction 'doment' i…

a5c5ab4

…n 2013 - f7.. variables were replaced by fh.. variables

Update formula and variables for tax reduction 'domlog' for years 201…

308c261

…4-2016

Rebase + Variable with change of meaning

2ad134a

Update formula for tax credit 'quaenv' 2014 + Correct mistakes in pre…

6434c8e

…vious formula (ceiling applies to tax base not tax credit) + update parameters

Correct reminding errors after rebase

be0c175

Add new variables for tax reductions 'saldom'

975ccf9

Update formula for tax reduction 'saldom' for years 2014-2016

a6b31ca

Update end date for tax credit 'saldom2' after checking validity

669c043

Correct mistake in 'saldom2' and 'saldom' formulas - The extra increa…

6c0f428

…se of the ceiling due to f7dl only applies to tax reduction not tax credit

Correction erreurs d'ancienne syntaxe ("simulation.calculate") restantes

ab61be1

Clean and simplify code

1aa35c2

Noms de variables plus explicites

16697ae

Correction erreur + Changement noms variables (suite)

2a1e7eb

claireleroy force-pushed the ipp-update-tax-reduc-and-credit-saldom branch from 8b9973b to 2a1e7eb Compare April 4, 2018 16:43

claireleroy added 3 commits April 4, 2018 19:20

Correction erreurs post-rebase

1c9a4f9

Correction erreurs post-rebase

03cacd9

Correction erreurs post-rebase

6d96876

benjello approved these changes Apr 4, 2018

View reviewed changes

claireleroy added 3 commits April 5, 2018 09:53

Rename parameters

53df44b

Correction erreur post-rebase

f1b9c1f

Correction noms paramètres

d446a83

Anna-Livia added flow:contrib:doing and removed flow:contrib:backlog labels Apr 5, 2018

claireleroy added 2 commits April 5, 2018 11:57

Correction noms paramètres

403ea76

Update CHANGELOG et setup.py

b453bd0

claireleroy merged commit bc3cf8f into master Apr 5, 2018

MattiSG removed the flow:contrib:doing label Apr 5, 2018

This was referenced Apr 6, 2018

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

Closed

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

Merged

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

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Ipp 8 : Update réduction et crédit IR pour emploi à domicile #914

Ipp 8 : Update réduction et crédit IR pour emploi à domicile #914

claireleroy commented Feb 20, 2018

benjello Feb 20, 2018

benjello Feb 20, 2018

benjello Feb 20, 2018

Anna-Livia Mar 6, 2018

claireleroy Mar 27, 2018

Anna-Livia Mar 6, 2018

Anna-Livia Mar 6, 2018

claireleroy Mar 27, 2018

Anna-Livia Mar 6, 2018

Anna-Livia Mar 6, 2018

claireleroy Mar 27, 2018

Anna-Livia Mar 6, 2018

benjello commented Apr 4, 2018

Ipp 8 : Update réduction et crédit IR pour emploi à domicile #914

Ipp 8 : Update réduction et crédit IR pour emploi à domicile #914

Conversation

claireleroy commented Feb 20, 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

benjello commented Apr 4, 2018