Merge Variables and DatedVariables #522 #770

fpagnoux · 2017-06-01T08:37:27Z

Connected to openfisca/openfisca-core#371

MattiSG · 2017-06-02T17:12:45Z

Documentons-nous aujourd'hui quelque part les regexp et traitements appliqués lors des mises à jour ? Il va falloir prendre l'habitude de le faire pour faciliter la migration des différents systèmes socio-fiscaux.

fpagnoux · 2017-06-07T17:04:40Z

Non, pas aujourd'hui, mais oui, il faudrait le faire.

sandcha · 2017-06-12T08:20:13Z

CHANGELOG.md

+
+* Amélioration technique
+* Détails :
+  - Adapte OpenFisca-France à OpenFisca-Core 13.0.0


Mettre à jour la version de Core. Sinon, est-ce qu'un lien vers le descriptif avant/après du changelog Core aurait sa place ici ?

sandcha · 2017-06-12T08:24:51Z

CHANGELOG.md

+* Amélioration technique
+* Détails :
+  - Adapte OpenFisca-France à OpenFisca-Core 13.0.0
+    - Modifie la manière de définir les dates de début des formules, et les dates de fin des variables.


Devrions-nous évoquer la modification de nommage des formules (même sans date) ?

sandcha · 2017-06-12T08:53:40Z

openfisca_france/model/prelevements_obligatoires/impot_revenu/charges_deductibles.py

@@ -41,7 +41,7 @@ class f6el(Variable):
    column = IntCol(val_type = "monetary")
    entity = FoyerFiscal
    label = u"Autres pensions alimentaires versées à des enfants majeurs: 1er enfant"
-    start_date = date(2006, 1, 1)
+    # start_date = date(2006, 1, 1)


Est-ce que le simple fait de commenter le start_date comme nous l'avions prévu est finalement suffisant pour l'utilisateur ? # Déprécié: start_date = date(2006, 1, 1) ?

Est-ce que le mot "Déprécié" rend vraiment l'intention plus claire ?
Je pense que la question est pertinente, mais vu le retard qu'on a pris, je préférerais rester sur ce qui a été défini dans l'issue plutôt que de passer du temps à réfléchir à une bonne formulation.

sandcha · 2017-06-12T09:07:44Z

openfisca_france/model/prelevements_obligatoires/impot_revenu/charges_deductibles.py

    column = FloatCol
    entity = FoyerFiscal
    label = u"Charges déductibles non plafonnées"
    url = "http://impotsurlerevenu.org/definitions/215-charge-deductible.php"
    definition_period = YEAR
+    end = '2014-12-31'


sandcha · 2017-06-12T09:14:10Z

openfisca_france/model/prelevements_obligatoires/impot_revenu/charges_deductibles.py

@@ -554,11 +546,10 @@ class cd_doment(Variable):
    column = FloatCol
    entity = FoyerFiscal
    label = u"Investissements DOM-TOM dans le cadre d’une entreprise"
-    start_date = date(2002, 1, 1)


Conserver le start_date commenté ou le transcrire explicitement dans formula_2002_01_01 afin de ne pas perdre d'information explicite (jour, mois) ?
Si l'on en profite pour simplifier la lecture, devrait-on appliquer ça à tout formule éligible ? C'est à dire même pour les groupes de formules (variables à plusieurs formules).

Sachant que la variable est définie à l'échelle de l'année (definition_period = YEAR), je trouve formula_2002 suffisamment explicite.

Je ne pense pas que l'uniformisation soit de notre responsabilité. On adapte juste france à core en essayant d'introduire le moins de désordre possible 🙂

sandcha · 2017-06-12T10:18:49Z

openfisca_france/model/prelevements_obligatoires/impot_revenu/ir.py

@@ -679,8 +675,7 @@ def function_20050101_20121231(self, simulation, period):
        DEF = deficit_rcm
        return max_(TOT1 + TOT2 + TOT3 - DEF, 0)

-    @dated_function(start = date(2013, 1, 1))
-    def function_20130101_20151231(self, simulation, period):


De même que précédemment : que faire d'une information de fin de formule qui ne figurait que dans le nom ? Remonter un warning à openfisca-france pour clarification ?

Commentaire ajouté.

sandcha · 2017-06-12T10:28:18Z

openfisca_france/model/prelevements_obligatoires/impot_revenu/ir.py

        out = (plus_values.taux2 * (f3vd + f3sd) + plus_values.taux3 * (f3vi + f3si) +
            plus_values.taux4 * (f3vf + f3sf) + plus_values.taux1 * max_(0, f3vg - f3vh) + plus_values.pvce * rpns_pvce)
                # TODO: chek this rpns missing ?
        return round_(out)

-    @dated_function(start = date(2013, 1, 1))
-    def function_20130101_20151231(self, simulation, period):  # f3sd is in f3vd holder


De même : perte d'information sans impact sur l'exécution.

Commentaire ajouté

sandcha · 2017-06-12T10:31:46Z

openfisca_france/model/prelevements_obligatoires/impot_revenu/ir.py

@@ -1756,8 +1735,7 @@ def function_20020101_20071231(self, simulation, period):
        out = f2dh + f2ee
        return out * not_(finpfl) / 12

-    @dated_function(start = date(2008, 1, 1))
-    def function_20080101_20151231(self, simulation, period):


De même : que faire de cette perte d'information sans impact sur l'exécution ?

Commentaire ajouté 👌

sandcha · 2017-06-12T11:33:05Z

openfisca_france/model/prelevements_obligatoires/impot_revenu/reductions_impot.py


-    @dated_function(start = date(2009, 1, 1), stop = date(2009, 12, 31))
-    def function_20090101_20091231(self, simulation, period):
+


Typo : ligne en trop.

sandcha · 2017-06-12T11:36:08Z

openfisca_france/model/prelevements_obligatoires/impot_revenu/reductions_impot.py

    column = FloatCol
    entity = FoyerFiscal
    label = u"patnat"
    definition_period = YEAR
+    end = '2013-12-31'
+


Typo : ligne en trop.

sandcha · 2017-06-12T12:59:49Z

openfisca_france/model/prelevements_obligatoires/impot_revenu/variables_reductions_credits.py

@@ -3930,7 +3930,7 @@ class f1ar(Variable):
    column = BoolCol
    entity = FoyerFiscal
    label = u"Crédit d'impôt aide à la mobilité : le déclarant déménage à plus de 200 km pour son emploi"
-    stop_date = date(2080, 12, 31)
+    end = '2080-12-31'


En marge de la PR : 2080 typo pour 2008 ?

sandcha · 2017-06-13T06:40:58Z

...ca_france/model/prelevements_obligatoires/prelevements_sociaux/cotisations_sociales/stage.py

@@ -25,10 +25,9 @@ class stage_gratification(Variable):
    column = FloatCol
    entity = Individu
    label = u"Gratification de stage"
-    start_date = date(2014, 11, 1)  # TODO: remove when updating legislation backwards


Conserver le commentaire vers le nom de la formule ?

sandcha · 2017-06-13T06:41:38Z

...ca_france/model/prelevements_obligatoires/prelevements_sociaux/cotisations_sociales/stage.py

@@ -44,10 +43,9 @@ class stage_gratification_reintegration(Variable):
    column = FloatCol
    entity = Individu
    label = u"Part de la gratification de stage réintégrée à l'assiette des cotisations et contributions sociales"
-    start_date = date(2014, 11, 1)  # TODO: remove when updating legislation backwards


De même : conserver le commentaire ?

sandcha · 2017-06-13T07:02:18Z

openfisca_france/model/prestations/minima_sociaux/aah.py

-    @dated_function(start = date(2002, 1, 1), stop = date(2005, 6, 30))  # TODO FIXME start date
-    def function_2005_06_30(self, simulation, period):
+    # TODO FIXME start date
+    def formula_2002_01_31(self, simulation, period):


Typo start date : formula_2002_01_01

Good catch 👍 👍

sandcha · 2017-06-13T08:09:22Z

openfisca_france/reforms/plf2015.py

    label = u"Décote IR 2015 appliquée sur IR 2014 (revenus 2013)"
    definition_period = YEAR

-    @dated_function(start = date(2013, 1, 1), stop = date(2013, 12, 31))


Ajouter end = '2013-12-31'

We are in a reform, and in that case, the end didn't mean that the formula stops in '2013-12-31'.

It meant that we were replacing the formula of decote from '2013-01-01' to '2013-12-31' (it took me some time to understand, as I had errors when putting an end date).

The behaviour we now have is slightly different than it was before, as we are now changing the formula "from 2013" instead of "for 2013 only".

It's probably alright in this context (I doubt we'll ever study the impact of the PLF 2015 in 2017), and we don't have a better helper :(.

sandcha · 2017-06-13T08:39:13Z

openfisca_france/reforms/plfr2014.py

        label = u"Somme des réductions d'impôt à intégrer pour l'année 2013"
        reference = reductions_impot.reductions
        definition_period = YEAR

-        @dated_function(start = date(2013, 1, 1), stop = date(2013, 12, 31))


Ajouter end = '2013-12-31' ?

Même réponse que pour les cas précédents

sandcha · 2017-06-13T08:41:40Z

setup.py

@@ -51,7 +51,7 @@
    install_requires = [
        'Biryani[datetimeconv] >= 0.10.4',
        'numpy >= 1.11',
-        'OpenFisca-Core >= 12.2.0, < 13.0',
+        'OpenFisca-Core >= 13.0, < 14.0',


Mettre à jour la version à : 'OpenFisca-Core >= 14.0, < 15.0', ?

sandcha · 2017-06-13T09:05:13Z

openfisca_france/model/prestations/minima_sociaux/rsa.py

        ressortissant_eee = individu('ressortissant_eee', period)
        duree_possession_titre_sejour = individu('duree_possession_titre_sejour', period)
        duree_min_titre_sejour = legislation(period).prestations.minima_sociaux.rsa.duree_min_titre_sejour
        return or_(ressortissant_eee, duree_possession_titre_sejour >= duree_min_titre_sejour)

-    @dated_function(stop = date(2009, 5, 31))
-    def function_rmi(individu, period, legislation):


Conserver le suffixe rmi en commentaire ?

sandcha · 2017-06-13T09:09:23Z

openfisca_france/model/prestations/minima_sociaux/rsa.py

@@ -970,8 +946,7 @@ def function_rsa(famille, period, legislation):

        return eligib * socle * taux

-    @dated_function(stop = date(2009, 5, 31))
-    def function_rmi(famille, period, legislation):


Conserver "rmi" (du nom de fonction) en commentaire ?

sandcha · 2017-06-13T09:19:43Z

openfisca_france/model/prestations/aides_logement.py

    column = FloatCol
    entity = Famille
    label = u"Revenu de référence, basé sur la situation familiale, pris en compte dans le calcul des AL."
    definition_period = MONTH

-    @dated_function(stop = date(2014, 12, 31))
-    def function_2014(famille, period, legislation):
+    def formula_2014(famille, period, legislation):


Renommer en formula ? Cette formule commence à l'âge de pierre si on fait fi de son nom. 🙂

Good catch 👍 👍

Rename function -> formula Rename stop_date -> end

fpagnoux changed the title ~~rAdapt France to Core v13~~ Adapt France to Core v13 Jun 1, 2017

fpagnoux changed the title ~~Adapt France to Core v13~~ Merge Variables and DatedVariables #522 Jun 7, 2017

fpagnoux requested a review from sandcha June 7, 2017 17:10

sandcha requested changes Jun 12, 2017

View reviewed changes

fpagnoux requested a review from sandcha June 12, 2017 16:25

sandcha requested changes Jun 13, 2017

View reviewed changes

fpagnoux force-pushed the dated-variable-#371 branch from c7f1aac to be45484 Compare June 13, 2017 14:02

fpagnoux requested a review from sandcha June 13, 2017 14:09

fpagnoux added 2 commits June 13, 2017 16:44

Upgrade core

ce5c4a1

Start migrating variables

09a11ba

Rename function -> formula Rename stop_date -> end

sandcha approved these changes Jun 13, 2017

View reviewed changes

fpagnoux force-pushed the dated-variable-#371 branch from be45484 to e7dbf0f Compare June 13, 2017 15:05

fpagnoux added 4 commits June 13, 2017 17:10

Don't use deprecated DatedVariable

05e557a

Don't use deprecated decorators

ab4b563

Remove start_date

89cb032

Make de_net_a_brut more robust

027a730

fpagnoux force-pushed the dated-variable-#371 branch 3 times, most recently from 867073b to ca4d836 Compare June 13, 2017 15:29

fpagnoux added 2 commits June 13, 2017 17:38

Fix plf 2016 reform

876661c

Bump version number

7581217

fpagnoux force-pushed the dated-variable-#371 branch from ca4d836 to 7581217 Compare June 13, 2017 15:39

fpagnoux merged commit 9d9fabb into master Jun 13, 2017

fpagnoux deleted the dated-variable-#371 branch June 13, 2017 16:11

guillett mentioned this pull request Apr 12, 2018

Semantic versioning usability in OpenFisca extension openfisca/extension-template#10

Closed

Morendil mentioned this pull request Oct 15, 2018

Factorise la duplication dans le calcul de la 'Prime de Noël' #1149

Merged

7 tasks


		@dated_function(start = date(2009, 1, 1), stop = date(2009, 12, 31))
		def function_20090101_20091231(self, simulation, period):

Merge Variables and DatedVariables #522 #770

Merge Variables and DatedVariables #522 #770

Conversation

fpagnoux commented Jun 1, 2017 • edited Loading

MattiSG commented Jun 2, 2017

fpagnoux commented Jun 7, 2017 • 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

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

Choose a reason for hiding this comment

fpagnoux Jun 13, 2017 • 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

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

fpagnoux commented Jun 1, 2017 •

edited

Loading

fpagnoux commented Jun 7, 2017 •

edited

Loading

fpagnoux Jun 13, 2017 •

edited

Loading