Ajout du barème pour le chèque énergie #924

guillett · 2018-03-05T10:42:08Z

Évolution du système socio-fiscal.
Périodes concernées : à partir de 2017
Zones impactées : prestations/cheque_energie.py
Détails :
- Ajoute le calcul du montant du chèque énergie

benjello · 2018-03-07T15:48:03Z

@thomasdouenne : tu ne l'as pas implémenté ailleurs ?

guillett · 2018-03-14T14:15:59Z

@benjello aurais-tu un moment pour regarder cette PR ?

Note : Je n'ai pas trouvé dans https://github.com/openfisca/openfisca-france-indirect-taxation/tree/api_migration le barème du chèque énergie.

benjello · 2018-03-14T14:23:54Z

@gulliet: je n'y connais pas grand chose donc je ne regarderais que la forme

Sinon tout est sur la branche de thomas https://github.com/thomasdouenne/openfisca-france-indirect-taxation/tree/api_migration (oui on nest mal organisé ;-) mais on a dû faire face à beaucoup d'évolutions de core)

benjello · 2018-03-14T14:27:01Z

openfisca_france/model/prestations/cheque_energie.py

+
+    def formula_2017(menage, period, parameters):
+        uc = parameters(period).cheque_energie.unites_consommation
+        nb_persons = menage.nb_persons()


Le premier nb_personsa mettre en français

benjello · 2018-03-14T14:27:26Z

openfisca_france/model/prestations/cheque_energie.py

+        nb_persons = menage.nb_persons()
+        gardes_alternee = menage.sum(menage.members('garde_alternee', period.first_month))
+
+        adj_nb = nb_persons - 0.5 * gardes_alternee


Transformer adj_nb en quelque chose de plus explicite

benjello · 2018-03-14T14:30:02Z

@guillett : j'ai mis quelques suggestions mais j'ai approuvé sans faire exprès.

guillett · 2018-03-14T15:35:00Z

@benjello merci pour la revue sur la forme. Pour le fond, je vais regarder la branche chez @thomasdouenne. Nous avons mis à disposition ce développement à des personnes du MTES qui ont fait des retours positifs, donc je suis confiant sur le fond. Au pire, on corrigera le tir avec un patch (j'ai quand même ajouté pas mal de tests).

guillett · 2018-03-14T16:01:48Z

Pour référence :

unites_consommation_cheque_energie n'est pas implémenté dans api_migration@thomasdouenne/openfisca-france-indirect-taxation mais c'est une input variable
Mon barème me parait cohérent avec l'implémentation de @thomasdouenne https://github.com/thomasdouenne/openfisca-france-indirect-taxation/blob/api_migration/openfisca_france_indirect_taxation/reforms/officielle_2018_in_2016.py#L195-L206

Je considère que c'est bon 🙂

Anna-Livia · 2018-03-14T15:44:30Z

CHANGELOG.md

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

+


extra space

Anna-Livia · 2018-03-14T15:44:57Z

openfisca_france/model/mesures.py

@@ -11,6 +11,7 @@ class unites_consommation(Variable):
    value_type = float
    entity = Menage
    label = u"Unités de consommation du ménage, selon l'échelle de l'INSEE"
+    reference = u"https://insee.fr/fr/metadonnees/definition/c1802"


Anna-Livia · 2018-03-14T15:46:41Z

openfisca_france/model/prestations/cheque_energie.py

+
+class UCChequeEnergie(Enum):
+    __order__ = 'une_uc une_uc_a_deux_uc plus_de_deux_uc'  # Needed to preserve the enum order in Python 2
+    une_uc = u'Une UC'


Dans la description, Serait-il plus compréhensible de mettre 'une unité de consommation (UC)' ?

Anna-Livia · 2018-03-14T15:48:34Z

setup.py

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

 setup(
    name = 'OpenFisca-France',
-    version = '20.3.0',
+    version = '20.3.1',


20.4 --> Ajout de fonctionnalité

Anna-Livia · 2018-03-14T15:48:51Z

CHANGELOG.md

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

+
+### 20.3.1


20.4 --> ajout de fonctionnalité

Anna-Livia · 2018-03-14T15:55:46Z

openfisca_france/model/prestations/cheque_energie.py

+        th = baremes.thresholds
+        montants = baremes.montants
+
+        uc_nb = menage('cheque_energie_unites_consommation', period)


uc_menage ?

Anna-Livia · 2018-03-14T15:57:36Z

openfisca_france/model/prestations/cheque_energie.py

+    definition_period = MONTH
+
+    def formula_2017(menage, period, parameters):
+        return menage('cheque_energie_montant', period.this_year) * menage('cheque_energie_eligibilite_logement', period)


Ne faudrait-il pas une ligne vide à la fin du fichier ?

Anna-Livia · 2018-03-14T16:00:55Z

openfisca_france/model/prestations/cheque_energie.py

+            [UCChequeEnergie.une_uc, UCChequeEnergie.une_uc_a_deux_uc, UCChequeEnergie.plus_de_deux_uc]
+            )
+
+        return (base < th.un) * montants[uc].un + (th.un <= base < th.deux) * montants[uc].deux + (th.deux <= base < th.trois) * montants[uc].trois


proposition:

return ( + (base < th.un) * montants[uc].un + (th.un <= base < th.deux) * montants[uc].deux + (th.deux <= base < th.trois) * montants[uc].trois )

Je changerais aussi le nom des baremes ainsi :

return ( + (base_revenus < seuil.une_uc) * montants[uc].une_uc + (seuil.une_uc <= base_revenus < seuil.deux_uc) * montants[uc].deux_uc + (seuil.deux_uc <= base_revenus < seuil.trois_uc) * montants[uc].trois_uc )

Merci beaucoup @Anna-Livia, j'ai pris la première proposition (multiligne) mais la seconde n'est pas utilisable.

On ne peut pas utiliser une_uc, deux_uc car cela ne correspond pas au modèle (cf. https://github.com/openfisca/openfisca-france/pull/924/files#diff-ee2c2a6c119744fb4009faf34543d10eR15)

Anna-Livia · 2018-03-14T16:01:03Z

openfisca_france/model/prestations/cheque_energie.py

+        uc_nb = menage('cheque_energie_unites_consommation', period)
+        rfr = menage.personne_de_reference.foyer_fiscal('rfr', period.n_2)
+
+        base = rfr / uc_nb


base_revenus

Anna-Livia · 2018-03-14T16:01:48Z

openfisca_france/model/prestations/cheque_energie.py

+    entity = Menage
+    value_type = float
+    reference = "https://chequeenergie.gouv.fr"
+    label = u"Chèque énergie"


Qu'est ce que retourne cette variable ?
'Montant auquel le ménage peut prétendre au titre du chèque energie' ?

* Also rename variable to prefix with cheque_energie_

* From adj_nb to nb_personnes_ajuste

guillett · 2018-03-14T16:36:10Z

Merci à vous deux @Anna-Livia et @benjello !

guillett force-pushed the cheque_energie branch 2 times, most recently from b5bc7a1 to a98a2c3 Compare March 5, 2018 15:30

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

bonjourmauko assigned guillett and unassigned guillett Mar 8, 2018

guillett requested review from benjello and Anna-Livia March 12, 2018 14:52

guillett added 3 commits March 13, 2018 17:58

Implement and test UC for chèque énergie

b369f59

Add reference for INSEE UC definition

b50759c

Implement and validate cheque energie montant

d8c6008

guillett force-pushed the cheque_energie branch from 1b3072e to 0657a01 Compare March 13, 2018 16:59

benjello approved these changes Mar 14, 2018

View reviewed changes

Anna-Livia reviewed Mar 14, 2018

View reviewed changes

guillett added 5 commits March 14, 2018 17:25

Increment version

59c7bc4

Add parameters for cheque energie

43c0315

Add logement eligibilite

d05a12f

* Also rename variable to prefix with cheque_energie_

Having fun with Enum is OpenFisca

688a153

Improve variable name

877069e

* From adj_nb to nb_personnes_ajuste

guillett force-pushed the cheque_energie branch 2 times, most recently from 72ee621 to fadaf5b Compare March 14, 2018 16:27

Improve names

8b0a64c

guillett force-pushed the cheque_energie branch from fadaf5b to 8b0a64c Compare March 14, 2018 16:34

guillett merged commit 15b482b into master Mar 15, 2018

MattiSG removed the flow:contribution-backlog label Mar 15, 2018

guillett deleted the cheque_energie branch March 15, 2018 06:49

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Ajout du barème pour le chèque énergie #924

Ajout du barème pour le chèque énergie #924

guillett commented Mar 5, 2018

benjello commented Mar 7, 2018

guillett commented Mar 14, 2018 •

edited

Loading

benjello commented Mar 14, 2018

benjello Mar 14, 2018

benjello Mar 14, 2018

benjello commented Mar 14, 2018

guillett commented Mar 14, 2018

guillett commented Mar 14, 2018

Anna-Livia Mar 14, 2018

Anna-Livia Mar 14, 2018

Anna-Livia Mar 14, 2018

Anna-Livia Mar 14, 2018

Anna-Livia Mar 14, 2018

Anna-Livia Mar 14, 2018

Anna-Livia Mar 14, 2018

Anna-Livia Mar 14, 2018

guillett Mar 14, 2018

Anna-Livia Mar 14, 2018

Anna-Livia Mar 14, 2018

guillett commented Mar 14, 2018

Ajout du barème pour le chèque énergie #924

Ajout du barème pour le chèque énergie #924

Conversation

guillett commented Mar 5, 2018

benjello commented Mar 7, 2018

guillett commented Mar 14, 2018 • edited Loading

benjello commented Mar 14, 2018

Choose a reason for hiding this comment

Choose a reason for hiding this comment

benjello commented Mar 14, 2018

guillett commented Mar 14, 2018

guillett commented Mar 14, 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

guillett commented Mar 14, 2018

guillett commented Mar 14, 2018 •

edited

Loading