Prise en compte du patrimoine dans les AL et le RSA #902

guillett · 2018-02-08T14:23:19Z

Évolution du système socio-fiscal. | Amélioration technique. | Correction d'un crash. | Changement mineur.
Périodes concernées :
- à partir du 2009 pour le RSA
- à partir d'octobre 2016 pour les AL
Détails :
- Mise à jour de la prise en compte du patrimoine dans la base ressource du RSA
  - Suppression de interets_epargne_sur_livrets et epargne_non_remuneree
  - Création de livret_a et epargne_revenus_non_imposables
- Prise en compte du patrimoine dans la base ressource des aides au logement
  - Création de valeur_locative_loue et epargne_revenus_imposables

Ces changements :

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

Cette PR contient des modifications qui nécessite une montée de version majeure.

Anna-Livia · 2018-02-15T10:46:16Z

openfisca_france/model/prestations/minima_sociaux/rsa.py

-        epargne_non_remuneree = individu('epargne_non_remuneree', period)
+        livret_a = individu('livret_a', period)
+        taux_livret_a = parameters(period).epargne.livret_a.taux
+        epargne_revenus_non_imposables = individu('epargne_revenus_non_imposables', period)


Pourquoi est ce que 'epargne_revenus_imposables' n'est pas pris en compte ? ou valeur_locative_loue?
Est ce que c'est compris dans revenus_capital ?

La contribution de l'épargne générant des revenus imposable se fait par l'intermédiaire de revenus_capital. Celle des biens loués par revenus_locatifs.

Anna-Livia · 2018-02-15T10:47:36Z

openfisca_france/model/revenus/capital/financier.py

    value_type = float
    entity = Individu
    base_function = requested_period_last_value
-    label = u"Intérêts versés pour l'épargne sur livret"


Livret A compris ou hors Livret A ?

Anna-Livia · 2018-02-15T10:49:16Z

openfisca_france/model/revenus/capital/foncier.py

+    entity = Individu
+    base_function = requested_period_last_value
+    label = u"Valeur locative des biens immobiliers et des terrains loués"
+    definition_period = MONTH


Peut on ajouter un set_input = set_input_divide_by_period, les loyers étant généralement stables sur l'année ?

Pour moi, set_input_divide_by_period s'utilise pour des flux et je voyais la valeur locative comme n'en n'étant pas un.

En effet, pour une année donnée, pour moi, la bonne valeur de cette variable n'est pas la somme sur les 12 mois mais la dernière valeur connue (j'ai regardé si il existe un tel set_input mais je n'ai pas trouvé).

Est-ce que cela te paraît sensé ?

@benjello as-tu des éléments complémentaires ? Que penses-tu de la perspective décrite ?

Je suis d'accord avec ton analyse @guillett

Attention, quand je lis "la bonne valeur de cette variable n'est pas la somme sur les 12 mois mais la dernière valeur connue " j'ai l'impression qu'on raisonne output, alors que les set_input définisse le comportement vis à vis des input.

La question à se poser c'est : quel est le meilleur comportement à avoir si un utilisateur déclare en entrée une valeur_locative_loue pour 2018 par exemple.

Dans le cas d'un flux, on utilise set_input_divide_by_period qui va diviser le montant entre les 12 mois de l'année

Dans le cas d'une variable "d'état", on peut utiliser set_input_dispatch_by_period, qui va copier coller le montant défini pour l'année sur tous les mois de l'année.

Dans le reste des cas, on préfère retourner une erreur.

à voir dans ce cas précis si un set_input_dispatch_by_period ne serait pas justifié

Anna-Livia · 2018-02-15T10:50:01Z

openfisca_france/model/revenus/capital/foncier.py

 class valeur_locative_immo_non_loue(Variable):
    value_type = float
    entity = Individu
    base_function = requested_period_last_value
    label = u"Valeur locative des biens immobiliers possédés et non loués"
    definition_period = MONTH
-    set_input = set_input_divide_by_period


pourquoi avoir retiré le set_input?

Cf. #902 (comment)

En tout cas c'est de fait un breaking change, peut-être que ça mérite d'expliquer dans le changelog qu'il n'est plus possible de définir une valeur annuelle pour valeur_locative_immo_non_loue

Le label Valeur locative des biens immobiliers possédés et non loués me déroute parce que, la variable étant définie sur le mois, j'ai envie d'inputer la valeur locative mensuelle.

Je propose la label suivant : '"Valeur locative, à l'année, des biens immobiliers possédés et non loués"'
Qu'en penses-tu ?

Anna-Livia · 2018-02-15T10:50:14Z

openfisca_france/model/revenus/capital/foncier.py

@@ -132,4 +139,3 @@ class valeur_locative_terrains_non_loue(Variable):
    base_function = requested_period_last_value
    label = u"Valeur locative des terrains possédés et non loués"
    definition_period = MONTH
-    set_input = set_input_divide_by_period


same question :)

cf. #902 (comment)

Anna-Livia · 2018-02-15T10:51:19Z

CHANGELOG.md

+# 21.0.0 [#902](https://github.com/openfisca/openfisca-france/pull/902)
+
+* Évolution du système socio-fiscal.
+* Périodes concernées : à partir du 1er octobre 2016.


rsa_base_ressources_patrimoine_individu est modifié à partir de juin 2009

guillett · 2018-02-26T11:01:19Z

@Anna-Livia y-a-t-il des modifications encore en attente ? Merci beaucoup.

guillett · 2018-02-27T16:15:59Z

@Anna-Livia vient de m'indiquer que cette PR ne fait plus partie de l'itération actuelle.
@openfisca/france-admin avez-vous une idée de quand cette PR pourra être mergée ?
Merci !

fpagnoux · 2018-02-27T16:33:27Z

openfisca_france/model/prestations/aides_logement.py

+        capitaux_non_productifs = livret_a + epargne_revenus_non_imposables
+        foncier = valeur_locative_loue + valeur_locative_immo_non_loue + valeur_locative_terrains_non_loue
+
+        patrimoine = epargne_revenus_imposables + capitaux_non_productifs + foncier


C'est normal qu'on ajouter une épargne (un stock) à une valeur locative, a priori plutôt un flux ?

fpagnoux · 2018-02-27T17:11:30Z

openfisca_france/model/revenus/capital/foncier.py

@@ -117,13 +117,20 @@ class revenus_locatifs(Variable):
    set_input = set_input_divide_by_period


+class valeur_locative_loue(Variable):


Quelle est la différence entre valeur_locative_loue et revenus_locatifs ?

guillett · 2018-02-28T16:26:26Z

@fpagnoux merci pour tes retours. Effectivement, j'ai eu une mauvaise compréhension de la législation. J'ai questionné la DHUP pour avoir de nouvels éléments.

Anna-Livia · 2018-03-06T15:25:21Z

openfisca_france/model/revenus/capital/foncier.py

 class valeur_locative_immo_non_loue(Variable):
    value_type = float
    entity = Individu
    base_function = requested_period_last_value
    label = u"Valeur locative des biens immobiliers possédés et non loués"
    definition_period = MONTH
-    set_input = set_input_divide_by_period


Le label Valeur locative des biens immobiliers possédés et non loués me déroute parce que, la variable étant définie sur le mois, j'ai envie d'inputer la valeur locative mensuelle.

Je propose la label suivant : '"Valeur locative, à l'année, des biens immobiliers possédés et non loués"'
Qu'en penses-tu ?

guillett · 2018-03-12T10:57:12Z

@Anna-Livia si c'est bon pour toi, est-ce que je peux te laisser valider, rebase et merge ? Merci beaucoup.

guillett · 2018-03-12T14:51:37Z

@openfisca/msa avez-vous des remarques sur cette PR vis à vis de #897 ?

everything is fixed

* Sourced from https://fr.wikipedia.org/wiki/Livret_A

* Update description

* to taux_interet_forfaitaire_epargne_non_imposable

* New name valeur_locative_terrains_non_loues

* Use to accelerate startup time * Hacky

* Différencie valeur patrimoniale et valeur locative

…cative_terrains_non_loue * New name prestations.minima_sociaux.rsa.patrimoine.abattement_valeur_locative_terrains_non_loues (s added)

- Patrimoine 80.000 € et - Valeur locative à 400 € / mois

…s AL

bonjourmauko added the flow:contribution-backlog label Feb 12, 2018

guillett requested review from benjello and Anna-Livia February 15, 2018 07:42

Anna-Livia requested changes Feb 15, 2018

View reviewed changes

Anna-Livia added flow:in-progress and removed flow:contribution-backlog labels Feb 15, 2018

guillett requested a review from Anna-Livia February 20, 2018 09:42

Anna-Livia added flow:contribution-backlog and removed flow:current-iteration labels Feb 26, 2018

guillett force-pushed the patrimoine branch from 8dc8437 to 47ed715 Compare February 26, 2018 13:33

guillett mentioned this pull request Feb 27, 2018

Améliorer la description du patrimoine betagouv/aides-jeunes#617

Closed

fpagnoux reviewed Feb 27, 2018

View reviewed changes

guillett force-pushed the patrimoine branch 2 times, most recently from 8de6df9 to 93854f4 Compare March 6, 2018 14:10

Anna-Livia previously requested changes Mar 6, 2018

View reviewed changes

bonjourmauko assigned guillett and Anna-Livia and unassigned guillett Mar 8, 2018

Anna-Livia approved these changes Mar 14, 2018

View reviewed changes

guillett force-pushed the patrimoine branch 2 times, most recently from 0d669f9 to 8769f0b Compare March 15, 2018 10:51

bonjourmauko added flow:contrib:doing and removed flow:contrib:backlog labels Mar 26, 2018

guillett and others added 23 commits April 3, 2018 11:03

Move RSA formula tests to specific folder

1d52c53

Add asset variables for accurate AL and RSA valuations

294e43b

Add livret A rates

67717cc

* Sourced from https://fr.wikipedia.org/wiki/Livret_A

Add livret A saving in RSA ressources

861a03f

Remove interets_epargne_sur_livrets

27f45ea

Remove interets_epargne_sur_livrets from tests

599ee00

Remove epargne_non_remuneree

3b2930e

Remove epargne_non_remuneree from tests

65e35bf

Add epargne_revenus_non_imposables in RSA ressources

7152e43

Update tests

4807191

Update (rsa|rmi).taux_interet_forfaitaire_epargne_non_remunere

6e1fb77

* Update description

Rename taux_interet_forfaitaire_epargne_non_remunere

e7838b6

* to taux_interet_forfaitaire_epargne_non_imposable

Prise en compte du patrimoine dans les ALs

e20ca5d

Ajout de tests pour la prise en compte du patrimoine dans les ALs

8056f73

Prevent inference on asset variables

2cc0d32

Rename valeur_locative_terrains_non_loue

3b1b950

* New name valeur_locative_terrains_non_loues

Prevent tracking pickle files

ba6add0

* Use to accelerate startup time * Hacky

Corrige la prise en compte du patrimoine

9a4c2cb

* Différencie valeur patrimoniale et valeur locative

Rename prestations.minima_sociaux.rsa.patrimoine.abattement_valeur_lo…

4443d15

…cative_terrains_non_loue * New name prestations.minima_sociaux.rsa.patrimoine.abattement_valeur_locative_terrains_non_loues (s added)

Update version and Changelog

b20bc6a

Clarify 'valeur locative' variable labels

f79e760

Prise en compte du patrimoine dans les AL

a6e520d

- Patrimoine 80.000 € et - Valeur locative à 400 € / mois

Ajoute les cas de tests pour la prise en compte du patrimoine dans le…

ab340f7

…s AL

guillett force-pushed the patrimoine branch from ae3d1b4 to ab340f7 Compare April 3, 2018 09:04

Anna-Livia merged commit 6cc823c into master Apr 3, 2018

Anna-Livia removed the flow:contrib:doing label Apr 3, 2018

bonjourmauko removed the kind:improvement label Jul 23, 2018

guillett deleted the patrimoine branch August 22, 2018 14:41

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Prise en compte du patrimoine dans les AL et le RSA #902

Prise en compte du patrimoine dans les AL et le RSA #902

guillett commented Feb 8, 2018

Anna-Livia Feb 15, 2018

guillett Feb 15, 2018

Anna-Livia Feb 15, 2018

Anna-Livia Feb 15, 2018

guillett Feb 15, 2018

guillett Feb 20, 2018

benjello Feb 20, 2018

fpagnoux Feb 27, 2018

Anna-Livia Feb 15, 2018

guillett Feb 26, 2018

fpagnoux Feb 27, 2018

Anna-Livia Mar 6, 2018

Anna-Livia Feb 15, 2018

guillett Feb 26, 2018

Anna-Livia Feb 15, 2018

guillett commented Feb 26, 2018

guillett commented Feb 27, 2018

fpagnoux Feb 27, 2018

fpagnoux Feb 27, 2018

guillett commented Feb 28, 2018

Anna-Livia Mar 6, 2018

guillett commented Mar 12, 2018

guillett commented Mar 12, 2018

		@@ -117,13 +117,20 @@ class revenus_locatifs(Variable):
		set_input = set_input_divide_by_period


		class valeur_locative_loue(Variable):

Prise en compte du patrimoine dans les AL et le RSA #902

Prise en compte du patrimoine dans les AL et le RSA #902

Conversation

guillett commented Feb 8, 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

guillett commented Feb 26, 2018

guillett commented Feb 27, 2018

Choose a reason for hiding this comment

Choose a reason for hiding this comment

guillett commented Feb 28, 2018

Choose a reason for hiding this comment

guillett commented Mar 12, 2018

guillett commented Mar 12, 2018