Remplace paje_clca par page_prepare dans le calcul de la paje. #884

Anna-Livia · 2018-01-26T10:40:49Z

Fixes #880
This PR is one solution to the issue bellow. #884 is another one
A partir d'avril 2017, la page_clca n'existe plus .

Évolution du système socio-fiscal.
Périodes concernées : A partir du 04/2017
Zones impactées : prestations_familiales/paje.py.
Détails :
- Description de la fonctionnalité ajoutée ou du nouveau comportement adopté.
- Cas dans lesquels une erreur était constatée.

Ces changements :

Corrigent ou améliorent un calcul déjà existant.

Anna-Livia · 2018-01-26T10:41:21Z

Cet appel à l'API web provoque une erreur 500 :

{
  "familles": {
    "famille_1": {
      "enfants": [
        "Janet"
      ],
      "parents": [
        "Bill",
        "Bob"
      ],
"paje":{
"2018-03": null
}
    }
  },
  "foyers_fiscaux": {
    "foyer_fiscal_1": {
      "declarants": [
        "Bill",
        "Bob"
      ],
      "personnes_a_charge": [
        "Janet"
      ]
    }
  },
  "individus": {
    "Bill": {
      "salaire_de_base": {
        "2017": 20000,
"2018":20000
      }
    },
    "Bob": {
      "salaire_de_base": {
        "2017": 30000,
"2018":20000
      }
    },
    "Janet": {}
  },
  "menages": {
    "menage_1": {
      "conjoint": [
        "Bob"
      ],
      "enfants": [
        "Janet"
      ],
      "impots_directs": {
        "2017": null
      },
      "personne_de_reference": [
        "Bill"
      ],
      "revenu_disponible": {
        "2017": null
      }
    }
  }
}

Anna-Livia · 2018-01-26T15:18:04Z

J'ai fait de nouvelles modifications. J'ai découvert que la paje_colca avait aussi été supprimée.

J'ai fait une sorte de "hack" pour que paje_clca_taux_plein et paje_clca_taux_partielcontinues d'être fonctionnel car je n'ai pas réussi à déterminer comment elles impactent paje_cmg

Anna-Livia · 2018-01-26T15:19:01Z

@monbocal : Je ne sais pas comment bien tester la paje, la page_clca et la PrEPare. Est ce que tu pourrais m'aider ?

benjello

Je ne suis pas spécialiste mais cela m'a l'air correct

fpagnoux · 2018-02-01T04:36:43Z

J'arrive bien après la bataille, mais est-ce que vous avez envisagé de mettre une date de fin aux formules qui appellent les paramètres qui ont disparu de la législation ?

Si on met un attribut end à paje_clca qui vaut 2017-12-31, alors cette variable vaudra toujours 0 à partir du 2018-01-01 et sa formule ne sera jamais appellée. On n'aura donc plus d’erreur de paramètre.

fpagnoux · 2018-02-01T04:43:43Z

openfisca_france/model/prestations/minima_sociaux/ppa.py

@@ -228,6 +228,27 @@ class ppa_base_ressources_prestations_familiales(Variable):
    label = u"Prestations familiales prises en compte dans le calcul de la PPA"
    definition_period = MONTH

+    def formula_2017_04(famille, period, parameters, mois_demande):


Si une date de fin est définie pour les variables paje_*, on peut probablement garder la formule précédente.

Pour éviter les erreurs 500, toute variable doit être calculable à toute période. Si un paramètre utilisé est légitimement supprimé de la législation à une certaine date, la variable qui l'appelle doit soit être déclarée supprimée de la législation (via end), soit adaptée avec une nouvelle formule qui n'utilise pas ces paramètres.

fpagnoux · 2018-02-01T04:46:33Z

Si on veut se prémunir d'un tel bug dans le futur, un test python qui vérifie qu'on peut bien calculer sans erreur un revenu disponible en 2025 serait à mon avis bienvenue 🙂

benjello · 2018-02-01T09:48:00Z

Bravo @Anna-Livia ! Et aussi @fpagnoux pour tes remarques que j'aurais dû/pu faire pour certaines ...
Je tenterai de faire mieux mon boulot la prochaine fois :-(

benjello · 2018-02-01T10:12:39Z

@Anna-Livia : suggestion renommer aussi les variables avec des noms barbares gar_dom etc comme cela a été fait ici pour améliorer la lisibilité.

Anna-Livia · 2018-02-01T10:58:44Z

@benjello le lien pointe vers cette même PR 😕 Est-ce qu'il y a des variables que tu voulais modifier en particulier ?

benjello · 2018-02-01T11:04:49Z

@Anna-Livia je pense que cela vaut le coup de modifier les variables suivantes:

empl_dir en emploi_direct
ass_mat en assistant_maternel
gar_dom en garde_a_domicile

Anna-Livia · 2018-02-01T11:06:31Z

@benjello C'est fait pour formula_2017_04_01, est-ce que ta proposition est de l'appliquer aussi aux formules plus anciennes ?

benjello · 2018-02-01T11:10:51Z

Pour toutes je veux changer le nom des variables au sens openfisca et pas seulement la variable apparaissant dans la formule.
Désolé de ne pas avori été assez clair @Anna-Livia.

benjello · 2018-02-01T11:11:31Z

Peut-être même faudrait-il ajouter un préfixe paje_.

Anna-Livia · 2018-02-01T13:44:12Z

@benjello cela entrainerait un breaking change, je pense que ce n'est pas forcément la bonne PR pour ça. Cependant, il y a un vrai sujet sur les variables (au sens openfisca) utilisées dans la paje.

guillett

Merci beaucoup !

Il faut changer la version.

Je n'ai pas d'expertise pour avoir un avis sur la logique du change.

guillett · 2018-02-01T13:55:42Z

CHANGELOG.md

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

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


## 20.1.0 ?

guillett · 2018-02-01T13:56:25Z

setup.py

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

 setup(
    name = 'OpenFisca-France',
-    version = '20.0.3',
+    version = '20.1.3',


benjello · 2018-02-01T14:29:44Z

@Anna-Livia : as you like, peut-être qu'il faut préparer une PR où l'on stocke des breaking changes "cosmétiques". C'est vous les pros, je pensais juste que c'était bien de faire les changements d'un bloc quand on les voit.

onmytab · 2018-02-01T16:13:58Z

Bonjour, est-ce que cette modification est liée au fait qu'en ce moment les montants des variable af, paje_base et cmg (et ars il me semble) ne varient plus selon salaire_de_base ?

fpagnoux

Je n'ai hélas pas eu le temps de regarder en détail cette proposition d'évolution de la législation.

Il me semble par contre que cette PR n'est pas adaptée à la correction urgente d'un bug critique sur OpenFisca.

La correction du bug, qui doit être livrée au plus tôt, semble être mêlée à une évolution législative plus large, le tout noyé dans des déplacements de variables qui rendent impossible une relecture rapide du diff.

fpagnoux · 2018-02-01T17:23:34Z

openfisca_france/model/prestations/prestations_familiales/paje.py

+        paje = (paje_base > 0)
+        paje_colca = opt_colca * condition * (nbenf >= 3) * P.af.bmaf * (
+            (paje) * P.paje.colca.avecab + not_(paje) * P.paje.colca.sansab)
+        return paje_colca


Manque un saut de ligne en fin de fichier

fpagnoux · 2018-02-01T17:29:04Z

openfisca_france/model/prestations/prestations_familiales/paje.py

@@ -76,7 +87,7 @@ def formula_2004_01_01(famille, period):
        paje_cmg = famille('paje_cmg', period)
        paje_colca = famille('paje_colca', period)

-        return paje_base + (paje_naissance + paje_clca + paje_cmg + paje_colca) / 12


Ce /12 qui disparaît ne me semble pas sans conséquence.
A-t-on changé le résultat des formules de paje_naissance , etc. ? Cette division était-elle illégitime ?

La paje c'est la somme des aides qui la composent (paje_base, paje_naissance ...)
--> source : http://www.caf.fr/allocataires/droits-et-prestations/s-informer-sur-les-aides/petite-enfance/la-prestation-d-accueil-du-jeune-enfant-paje

Toutes ces aides ont l'attribut definition_period = MONTH

la /12me semble donc illégitime, peut être un artéfact d'un calcul à l'année

Ok, je ne suis pas un expert, donc pas de blocage de mon côté. Cette prestation n'était pas calculée par mes-aides (qui ne calcule que la PAJE de base je crois) donc rien ne me porte à croire que le calcul précédent était particulièrement juste.

Si @benjello et @guillett n'ont pas de raison de garder ce /12, ok pour le supprimer.

fpagnoux · 2018-02-01T17:32:18Z

openfisca_france/model/prestations/prestations_familiales/paje.py

@@ -244,122 +255,134 @@ def formula_2004_01_01(famille, period, parameters):
        return nais_prime * elig * nb_enfants_7e_mois_grossese


-class paje_clca(Variable):


Ces déplacements de variables rendent très difficile la lecture du diff 😕

Sorry, didn't know :/

fpagnoux · 2018-02-01T18:07:39Z

Je suggère un correctif minimaliste pour remettre le package d'aplomb : #892.

Anna-Livia · 2018-02-02T09:27:50Z

@onmytab Les modifications de cette PR sont liées à la disparition de la paje_clcaet de la paje_colca ainsi que la fin de l'obligation de salaire minimal pour être éligible à la paje_cmg.
Par contre, la base de ressource des allocation familiale ou sa prise en compte dans la paje_cmgne sont pas impactés.

Je suis curieuse d'en savoir plus sur les changement dont vous parlez. Pourriez-vous ouvrir une nouvelle issue sur le sujet ?

Pas de blocage

…CLCA

Anna-Livia requested review from guillett and benjello January 26, 2018 10:41

benjello approved these changes Jan 26, 2018

View reviewed changes

Anna-Livia requested review from monbocal and benjello January 26, 2018 15:14

Anna-Livia requested review from sandcha and fpagnoux January 26, 2018 15:21

benjello approved these changes Jan 31, 2018

View reviewed changes

fpagnoux previously requested changes Feb 1, 2018

View reviewed changes

guillett approved these changes Feb 1, 2018

View reviewed changes

Anna-Livia requested a review from fpagnoux February 1, 2018 16:37

fpagnoux reviewed Feb 1, 2018

View reviewed changes

Anna-Livia requested a review from fpagnoux February 2, 2018 10:00

Anna-Livia and others added 25 commits February 28, 2018 17:49

Ajoute une référence à la prepare

8856807

Corrige l'url de référence de la PAJE

79bc7b1

Met à jour la formule de la PAJE pour prendre en compte la PREPARE

5acc1aa

Bump version

6857c08

Adapte la PPA à la fin du CLCA et du COLCA

34c1544

Adapte le RSA à la fin du CLCA et du COLCA

f30c848

Ajoute la référence à la PREPARE

961e1f1

[TO CHECK] Fait pointer d'anciennes formules vers PREPARE plutot que …

e62e309

…CLCA

Retire le COLCA du calcul de la paje

01bb17c

Modification de la CMG pour disparition de la clca

f253555

Retour sur la duplication des formules

f063f2b

fixup! Modification de la CMG pour disparition de la clca

d3b5ab2

fixup! Bump version

589fbe9

Améliore la lisibilité de CMG

f4bf4eb

Ajout test paje suite disparition paje_clca et paje_colca au 01-04-2017

db5b940

Ajout test paje avant disparition paje_clca et paje_colca au 01-04-2017

7eec807

Améliore la cmg

0d24451

Ajoute l'attribut "end" aux formules

b279dc2

Améliore la cmg

55b735d

Améliore les tests

f978511

Supprime la division par 12 des aides mensuelles

7c7a64f

Ajoute les tests temps partiel

40e913e

revert change of formula

61f8d40

Revert change to rsa

090878c

Retire le salaire minimum pour la cmg

2d0efa9

Anna-Livia force-pushed the remplace_paje_clca branch from 1bacad7 to 2d0efa9 Compare February 28, 2018 17:01

Anna-Livia merged commit 78babfe into master Mar 1, 2018

Anna-Livia deleted the remplace_paje_clca branch March 1, 2018 13:14

Morendil mentioned this pull request Oct 15, 2018

Bonnes pratiques en matière de fixation des valeurs des paramètres #1163

Closed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Remplace paje_clca par page_prepare dans le calcul de la paje. #884

Remplace paje_clca par page_prepare dans le calcul de la paje. #884

Anna-Livia commented Jan 26, 2018 •

edited

Loading

Anna-Livia commented Jan 26, 2018 •

edited

Loading

Anna-Livia commented Jan 26, 2018

Anna-Livia commented Jan 26, 2018

benjello left a comment

fpagnoux commented Feb 1, 2018 •

edited

Loading

fpagnoux Feb 1, 2018

fpagnoux commented Feb 1, 2018

benjello commented Feb 1, 2018

benjello commented Feb 1, 2018

Anna-Livia commented Feb 1, 2018

benjello commented Feb 1, 2018

Anna-Livia commented Feb 1, 2018

benjello commented Feb 1, 2018

benjello commented Feb 1, 2018

Anna-Livia commented Feb 1, 2018

guillett left a comment

guillett Feb 1, 2018

guillett Feb 1, 2018

benjello commented Feb 1, 2018

onmytab commented Feb 1, 2018

fpagnoux left a comment

fpagnoux Feb 1, 2018

fpagnoux Feb 1, 2018 •

edited

Loading

Anna-Livia Feb 2, 2018

fpagnoux Feb 12, 2018

fpagnoux Feb 1, 2018

Anna-Livia Feb 2, 2018

fpagnoux commented Feb 1, 2018

Anna-Livia commented Feb 2, 2018

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

		### 20.1.3 [#884](https://github.com/openfisca/openfisca-france/pull/884)

		@@ -244,122 +255,134 @@ def formula_2004_01_01(famille, period, parameters):
		return nais_prime * elig * nb_enfants_7e_mois_grossese


		class paje_clca(Variable):

Remplace paje_clca par page_prepare dans le calcul de la paje. #884

Remplace paje_clca par page_prepare dans le calcul de la paje. #884

Conversation

Anna-Livia commented Jan 26, 2018 • edited Loading

Anna-Livia commented Jan 26, 2018 • edited Loading

Anna-Livia commented Jan 26, 2018

Anna-Livia commented Jan 26, 2018

benjello left a comment

Choose a reason for hiding this comment

fpagnoux commented Feb 1, 2018 • edited Loading

Choose a reason for hiding this comment

fpagnoux commented Feb 1, 2018

benjello commented Feb 1, 2018

benjello commented Feb 1, 2018

Anna-Livia commented Feb 1, 2018

benjello commented Feb 1, 2018

Anna-Livia commented Feb 1, 2018

benjello commented Feb 1, 2018

benjello commented Feb 1, 2018

Anna-Livia commented Feb 1, 2018

guillett left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

benjello commented Feb 1, 2018

onmytab commented Feb 1, 2018

fpagnoux left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

fpagnoux Feb 1, 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

fpagnoux commented Feb 1, 2018

Anna-Livia commented Feb 2, 2018

Anna-Livia commented Jan 26, 2018 •

edited

Loading

Anna-Livia commented Jan 26, 2018 •

edited

Loading

fpagnoux commented Feb 1, 2018 •

edited

Loading

fpagnoux Feb 1, 2018 •

edited

Loading