Ajoute l'abattement d'IR de 30 ou 40% pour les résidents des DOM #1040

claireleroy · 2018-07-18T13:34:02Z

Évolution du système socio-fiscal.
Périodes concernées :à partir du 01/01/2013.
Zones impactées : model\prelevements_obligatoires\impot_revenu\ir.
Détails :
- Ajoute une variable "depcom_foyer" donnant le lieu de résidence fiscale du foyer.
- Utilise cette variable pour calculer l'abattement d'impôt spécial DOM

Ces changements :

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

Anna-Livia · 2018-07-19T09:34:57Z

CHANGELOG.md

+* Évolution du système socio-fiscal.
+* Périodes concernées : à partir du 01/01/2013.
+* Zones impactées :
+  - `model\prelevements_obligatoires\impot_revenu\ir`


Il me semble que les / ne sont pas dans le bon sens ...

Anna-Livia · 2018-07-19T09:39:54Z

openfisca_france/model/prelevements_obligatoires/impot_revenu/ir.py

@@ -121,6 +122,64 @@ def formula(individu, period, parameters):
        return (datetime64(period.start) - date_naissance).astype('timedelta64[M]')


+class depcom_foyer(Variable):


Est ce qu'on peut réutiliser les variables existantes comme https://fr.openfisca.org/legislation/residence_mayotte ?

Oui je me suis inspiré de cette variable mais pour moi ce sont deux variables différentes :

residence_mayotte c'est le fait d'être domicilié à Mayotte (variable ménage et mensuelle)

residence_fiscale_mayotte, c'est le fait d'avoir sa résidence fiscale à Mayotte (variable foyer fiscal et annuelle).

Je comprends mieux ^^. Je pense qu'il serait interessant d'indiquer l'existence de l'autre variable dans la description, pour aider les utilisateurs.

Pour le faire fonctionner, tu peux écrire :

class depcom_foyer(Variable): value_type = str max_length = 5 entity = FoyerFiscal default_value = "00000" label = u"Code INSEE (depcom) du lieu de résidence" definition_period = YEAR set_input = set_input_dispatch_by_period

Anna-Livia · 2018-07-19T09:44:22Z

openfisca_france/model/prelevements_obligatoires/impot_revenu/ir.py

-        conditionGuadMarReu = 0 # faire une condition avec département + code commune (Depcom qui est pour l'instant une variable ménage)
-        conditionGuyMay = 0 # provisoire
-        conditionDOM = conditionGuadMarReu | conditionGuyMay
+        conditionGuadMarReu = (residence_fiscale_guadeloupe | residence_fiscale_martinique | residence_fiscale_reunion)


En Python, il faut privilégier le "snake case"
proposition :
residence_guad_mart_reun
ou
residence_guadeloupe_martinique_reunion

Anna-Livia · 2018-07-19T09:45:56Z

openfisca_france/model/prelevements_obligatoires/impot_revenu/ir.py

-        abat_dom = (conditionGuadMarReu * min_(plafond_qf.abat_dom.plaf_GuadMarReu, plafond_qf.abat_dom.taux_GuadMarReu * IP1) +
-                   conditionGuyMay * min_(plafond_qf.abat_dom.plaf_GuyMay, plafond_qf.abat_dom.taux_GuyMay * IP1))
-        IP2 = IP1 - abat_dom
+        IP2 = max_(0, IP1 - abattement_dom)


Il y aurait-il un terme plus explicite pour IP2 ?

Oui c'est vrai je propose impot_apres_abattement_dom ? Et du coup je vais modifier I0 et I1 pour que ce soit cohérent :)

Anna-Livia · 2018-07-19T09:47:35Z

openfisca_france/model/prelevements_obligatoires/impot_revenu/ir.py

-        conditionDOM = conditionGuadMarReu | conditionGuyMay
+        conditionGuadMarReu = (residence_fiscale_guadeloupe | residence_fiscale_martinique | residence_fiscale_reunion)
+        conditionGuyMay = (residence_fiscale_guyane | residence_fiscale_mayotte)
+        conditionDOM = (conditionGuadMarReu | conditionGuyMay)


Que penses-tu de residence_dom?

Anna-Livia · 2018-07-19T09:49:31Z

openfisca_france/model/prelevements_obligatoires/impot_revenu/ir.py


-        return (not_(conditionDOM) * (condition62a * IP0 + condition62b * IP1) +
-                conditionDOM * IP2)
+        return (not_(conditionDOM) * (condition62a * IP0 + condition62b * IP1) + conditionDOM * IP2)


proposition :

return ( not_(conditionDOM) * (condition62a * IP0 + condition62b * IP1) + conditionDOM * IP2 )

Anna-Livia · 2018-07-19T09:50:22Z

setup.py

@@ -7,7 +7,7 @@

 setup(
    name = 'OpenFisca-France',
-    version = '22.2.1',
+    version = '22.2.2',


J'opterai plutot pour un bump mineur --> 22.3.0

claireleroy · 2018-07-19T12:08:15Z

@Anna-Livia Merci pour ta review :) !

Par ailleurs j'ai un petit problème avec les tests qui passent pas. J'ai l'impression qu'il y a un problème avec mes nouvelles variables de résidence fiscale qui sont des codes départements-communes en 5 lettres. J'utilise la fonction startswith dont j'ai vu qu'elle était utilisée ailleurs dans le code, mais ici cela ne semble pas fonctionner... Est-ce que tu aurais une idée de la source de cette erreur ?

Anna-Livia

Hey, je pense avoir fixé l'issue.

Je pense que ce serait chouette d'ajouter des tests type :

- name: Résidence fiscales
  period: "year:2018"
  individus:
    id: "parent"
  foyers_fiscaux:
    depcom_foyer: 97105
    declarants: "parent"
  output_variables:
    residence_fiscale_guadeloupe: true
    residence_fiscale_mayotte: false
    residence_fiscale_martinique: false
    residence_fiscale_guyane: false
    residence_fiscale_reunion: false

Et il faudrait un autre test pour le ir_plaf_qf en métropole, un pour la réunion et un autre pour la guyanne.

Anna-Livia · 2018-07-23T08:57:05Z

openfisca_france/model/prelevements_obligatoires/impot_revenu/ir.py

@@ -121,6 +122,64 @@ def formula(individu, period, parameters):
        return (datetime64(period.start) - date_naissance).astype('timedelta64[M]')


+class depcom_foyer(Variable):


Je comprends mieux ^^. Je pense qu'il serait interessant d'indiquer l'existence de l'autre variable dans la description, pour aider les utilisateurs.

Pour le faire fonctionner, tu peux écrire :

class depcom_foyer(Variable): value_type = str max_length = 5 entity = FoyerFiscal default_value = "00000" label = u"Code INSEE (depcom) du lieu de résidence" definition_period = YEAR set_input = set_input_dispatch_by_period

Anna-Livia · 2018-07-25T15:34:45Z

openfisca_france/model/prelevements_obligatoires/impot_revenu/ir.py

+        residence_guyane_mayotte = (residence_fiscale_guyane | residence_fiscale_mayotte)
+        residence_dom = (residence_guadeloupe_martinique_reunion | residence_guyane_mayotte)
+
+        abattement_dom = (


il faut remplacer IP1par impot_apres_reduction_complementaire dans cette formule

claireleroy · 2018-07-30T08:16:31Z

@Anna-Livia Merci pour avoir trouvé l'erreur ! Au niveau des tests, tu penses qu'il faudrait les ajouter dans quel sous-dossier de openfisca-france\tests ?

Anna-Livia · 2018-07-31T15:21:27Z

@ClaireLeroyIPP Tu peut créer un dossier impot_revenus par exemple.
Et tu peux nommer le fichier abattement_dom.yaml
Vérifie que tout passe bien en faisant openfisca-run-test tests/impot_revenus.abattement_dom.yaml

claireleroy · 2018-08-01T09:32:18Z

@Anna-Livia C'est fait ! Je rebase, change la version et c'est bon pour merger ?

Anna-Livia · 2018-08-02T10:08:59Z

Hello @ClaireLeroyIPP On y est presque ! Pour les tests, je pense qu'il faut que tu en écrives encore pour tester ir_plaf_qf, avec un foyer en métropole, un pour la Réunion et un autre pour la Guyane.

(pour l'instant, il me semble que les tests se portent exclusivement sur depcom_foyer.)

Proposition:

- name: ir_plaf_qf en métropole
  period: 2018
  absolute_error_margin: 1
  input_variables:
    f2dc: 70000
    depcom_foyer: 27681
  output_variables:
    ir_plaf_qf: 6949.72


- name: ir_plaf_qf en Réunion
  period: 2018
  absolute_error_margin: 1
  input_variables:
    f2dc: 70000
    depcom_foyer: 97411
  output_variables:
    ir_plaf_qf: 1849.72


- name: ir_plaf_qf en Guyanne
  period: 2018
  absolute_error_margin: 1
  input_variables:
    f2dc: 70000
    depcom_foyer: 97302
  output_variables:
    ir_plaf_qf: 249.72

Anna-Livia · 2018-08-02T09:46:00Z

CHANGELOG.md

@@ -1,5 +1,15 @@
 # Changelog

+### 22.2.2 [#1040](https://github.com/openfisca/openfisca-france/pull/1040)


Je pense que cela devrait être la version v22.3:) tu ajoutes des fonctionnalités :)

claireleroy · 2018-08-02T11:57:40Z

@Anna-Livia Pour les tests sur le montant de l'impôt, je propose que ça fasse partie de la PR à venir qui rajoute des tests sur l'impôt en utilisant comme comparaison les résultats du simulateur de la DGFiP. Ces tests sont listés dans l'issue #920 sur laquelle je suis entrain de travailler d'ailleurs :)

Anna-Livia · 2018-08-02T12:50:50Z

@ClaireLeroyIPP pour les tests, je te propose de commencer à mettre des petits tests tout simple dès maintenant. on peut même mettre dans la description des tests qu'ils sont temporaires.

L'issue #920 apportera beaucoup de valeur. Cependant, elle va être massive, et j'ai peur du temps de review.
Avoir des petits tests en place permettent de voir le comportement attendu lors des reviews et d'être sûrs qu'on ne modifie rien par inadvertance.
Tout en sachant qu'on peut les enlever s'il y a de la redondance avec #920.

Voici mon opinion. Maintenant, à toi de prendre la décision finale :)

OK pour rebase et merge

…dans les fichiers des déclarations)

…résidence fiscale "depcom_foyer"

…dence fiscale

claireleroy · 2018-08-02T13:33:31Z

@Anna-Livia J'ai rajouté les tests du coup ! Et ça a été très utile puisque j'ai pu repérer une grossière erreur : j'avais mis taux = 40 au lieu de taux = 0.40
Cette fois-ci tout tourne je pense que c'est prêt à merger :)

claireleroy requested review from Anna-Livia and sandcha July 18, 2018 13:40

claireleroy force-pushed the ajout-abattement-impot-dom-tom branch 8 times, most recently from 60eda1a to dd130fe Compare July 19, 2018 08:53

Anna-Livia requested changes Jul 19, 2018

View reviewed changes

Anna-Livia added the flow:contrib:doing label Jul 19, 2018

Anna-Livia self-assigned this Jul 19, 2018

Anna-Livia requested changes Jul 25, 2018

View reviewed changes

Anna-Livia requested changes Aug 2, 2018

View reviewed changes

Anna-Livia approved these changes Aug 2, 2018

View reviewed changes

Claire Leroy added 7 commits August 2, 2018 15:17

Ajoute une variable indiquant le lieu de résidence fiscale (présente …

d9ca029

…dans les fichiers des déclarations)

Calcule l'abattement DOM à partir de la nouvelle variable de lieu de …

6762a9e

…résidence fiscale "depcom_foyer"

Amméliore la mise en forme

4682367

L'abattement DOM ne peut donner lieu à un crédit

aa152b9

Explicite le nom de variable

8372bd7

Update le Changelog et le setup.py

0c784c0

Ajoute une default_value

3eba888

Claire Leroy added 10 commits August 2, 2018 15:18

Ajoute des indicatrices de résidence fiscale pour chaque DOM

bd10724

Update le Changelog et le setup.py

63e110b

Modifie les noms des variables

d6ec9c3

Mets les opérateurs en début de ligne

1cde1d2

Explicite les noms de variables intermédiaires

40b8096

Modifie la variable de résidence fiscale "depcom_foyer"

a157541

Corrige une erreur

79e977b

Ajoute un test sur l'indicatrice de résidence fiscale en DOM

6c6e2c2

Rajoute des tests sur le calcul de l'abattement selon le lieu de rési…

62e2eaa

…dence fiscale

Corrige une erreur (taux de l'abattement * 100)

4979802

claireleroy force-pushed the ajout-abattement-impot-dom-tom branch from ea41335 to 4979802 Compare August 2, 2018 13:31

claireleroy merged commit 638c74f into master Aug 2, 2018

claireleroy deleted the ajout-abattement-impot-dom-tom branch August 3, 2018 07:44

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Ajoute l'abattement d'IR de 30 ou 40% pour les résidents des DOM #1040

Ajoute l'abattement d'IR de 30 ou 40% pour les résidents des DOM #1040

claireleroy commented Jul 18, 2018

Anna-Livia Jul 19, 2018

Anna-Livia Jul 19, 2018

claireleroy Jul 19, 2018

Anna-Livia Jul 23, 2018 •

edited

Loading

Anna-Livia Jul 19, 2018

Anna-Livia Jul 19, 2018

claireleroy Jul 19, 2018

Anna-Livia Jul 19, 2018

Anna-Livia Jul 19, 2018

Anna-Livia Jul 19, 2018

claireleroy commented Jul 19, 2018

Anna-Livia left a comment

Anna-Livia Jul 23, 2018 •

edited

Loading

Anna-Livia Jul 25, 2018

claireleroy commented Jul 30, 2018

Anna-Livia commented Jul 31, 2018

claireleroy commented Aug 1, 2018

Anna-Livia commented Aug 2, 2018

Anna-Livia Aug 2, 2018

claireleroy commented Aug 2, 2018

Anna-Livia commented Aug 2, 2018

claireleroy commented Aug 2, 2018 •

edited

Loading

		@@ -121,6 +122,64 @@ def formula(individu, period, parameters):
		return (datetime64(period.start) - date_naissance).astype('timedelta64[M]')


		class depcom_foyer(Variable):

		@@ -1,5 +1,15 @@
		# Changelog

		### 22.2.2 [#1040](https://github.com/openfisca/openfisca-france/pull/1040)

Ajoute l'abattement d'IR de 30 ou 40% pour les résidents des DOM #1040

Ajoute l'abattement d'IR de 30 ou 40% pour les résidents des DOM #1040

Conversation

claireleroy commented Jul 18, 2018

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Anna-Livia Jul 23, 2018 • edited Loading

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 Jul 19, 2018

Anna-Livia left a comment

Choose a reason for hiding this comment

Anna-Livia Jul 23, 2018 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

claireleroy commented Jul 30, 2018

Anna-Livia commented Jul 31, 2018

claireleroy commented Aug 1, 2018

Anna-Livia commented Aug 2, 2018

Choose a reason for hiding this comment

claireleroy commented Aug 2, 2018

Anna-Livia commented Aug 2, 2018

claireleroy commented Aug 2, 2018 • edited Loading

Anna-Livia Jul 23, 2018 •

edited

Loading

Anna-Livia Jul 23, 2018 •

edited

Loading

claireleroy commented Aug 2, 2018 •

edited

Loading