Al primo accession #801

dgombert · 2017-08-11T12:39:49Z

Implémentation du calcul des aides au logement primo accédant.

Évolution du système socio-fiscal.
Périodes concernées : toutes.
Zones impactées : /prestations/aides_logement.py.
Détails :
- Calcul des aides au logement pour les primo-accédants.
- Marge d'erreur (~5% sur les tests) voir avec un expert métier pour rectifier le calcul.

Ces changements (effacez les lignes ne correspondant pas à votre cas) :

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

- Ajout du calcul de ressources plafonné - Mise à jour des tests avec fiabilité à 5% (voir avec un expert métier pour améliorer la fiabilité)

benjello

Il serait utile de rajouter les variables intermédiaires ici créées dans cette blacklist qui a pour but de limiter le nombre de variables conservées en cache.

fpagnoux

Great 🎉!

Il manque juste:

les références législatives
si possible quelques tests (seulement 3 pour le moment)
quelques corrections mineures

fpagnoux · 2017-08-11T12:45:30Z

openfisca_france/model/prestations/aides_logement.py

+class aides_logement_primo_accedant(Variable):
+    column = FloatCol
+    entity = Famille
+    label = u"Allocation logement primo accedant"


Il manque un accent : primo-accédant :)

fpagnoux · 2017-08-11T12:49:32Z

openfisca_france/model/prestations/aides_logement.py

+        majoration_loyer = prestations.al_param.majoration_du_loyer_minimum_lo
+        N = famille('aides_logement_primo_accedant_nb_part', period)
+
+        return ((bareme.calc(baseRessource / N) * N) + majoration_loyer) / 12  


Certaines parenthèses ne sont pas nécessaires:

(bareme.calc(baseRessource / N) * N + majoration_loyer) / 12

devrait revenir au même

De même: si possible, indiquer une source législative.

fpagnoux · 2017-08-11T12:52:52Z

openfisca_france/model/prestations/aides_logement.py

+           z1.personne_isolee_sans_enfant * not_(couple) * (al_nb_pac == 0) +
+           z1.menage_seul * couple * (al_nb_pac == 0) +
+           z1.menage_ou_isole_avec_1_enfant * (al_nb_pac == 1) +
+           z1.menage_ou_isole_avec_2_enfants* (al_nb_pac == 2) +


Il manque un espace avant le *

z1.menage_ou_isole_avec_2_enfants * (al_nb_pac == 2) +

fpagnoux · 2017-08-11T12:53:17Z

openfisca_france/model/prestations/aides_logement.py

+        loyer = famille.demandeur.menage('loyer', period)
+        coef_plancher_ressources = legislation(period).prestations.aides_logement.ressources.dar_3
+        return max_(baseRessource, loyer * coef_plancher_ressources)
+


Il manque une ligne vide en fin de fichier.

fpagnoux · 2017-08-11T12:53:59Z

openfisca_france/parameters/prestations.xml

+            <VALUE deb="2007-07-01" valeur="0.328"/>
+        </TAUX>
+      </TRANCHE>
+    </BAREME>


fpagnoux · 2017-08-11T12:59:49Z

tests/formulas/aides_logement_primo_accedant.yaml

+    loyer: 500
+    depcom: 82121
+  output_variables:
+    aides_logement_primo_accedant: 95


Il manque un saut de ligne à la fin du fichier.

fpagnoux · 2017-08-11T13:00:16Z

tests/formulas/aides_logement_primo_accedant_dhup.yaml

+    2017-02: 1
+    2017-03: 1
+output_variables:
+    aide_logement: 154


Il manque un saut de ligne à la fin du fichier.

fpagnoux · 2017-08-11T13:04:11Z

openfisca_france/model/prestations/aides_logement.py

+        K = famille('aides_logement_primo_accedant_k', period)
+        Lo = famille('aides_logement_primo_accedant_loyer_minimal', period)
+
+        return K * ( L + C - Lo)


Existe-il un article de loi qui définit cette formule ?
Si oui, on peut le mettre dans l'attribut référence de la variable.

label = u"Allocation logement primo accedant" reference = "http://legifrance.gouv.fr/affichCodeArticle.doidArticle=LEGIARTI000032852785&cidTexte=LEGITEXT000006073189&dateTexte=20170803"

fpagnoux · 2017-08-11T13:04:47Z

openfisca_france/model/prestations/aides_logement.py

+        R = famille('aides_logement_primo_accedant_ressources', period)
+        N = famille('aides_logement_primo_accedant_nb_part', period)
+
+        return coef_k - ( R / (multi_n * N))


De même: si possible, indiquer une source législative.

fpagnoux · 2017-08-11T13:06:01Z

openfisca_france/model/prestations/aides_logement.py

+        baseRessource = famille('aide_logement_base_ressources', period)
+        loyer = famille.demandeur.menage('loyer', period)
+        coef_plancher_ressources = legislation(period).prestations.aides_logement.ressources.dar_3
+        return max_(baseRessource, loyer * coef_plancher_ressources)


De même: si possible, indiquer une source législative.

…as d'un primo accédant

fpagnoux · 2017-08-11T13:24:37Z

Il serait utile de rajouter les variables intermédiaires ici créées dans cette blacklist qui a pour but de limiter le nombre de variables conservées en cache.

Je m'en occupe :)

fpagnoux

Presque

fpagnoux · 2017-08-11T14:38:13Z

openfisca_france/model/prestations/aides_logement.py

+    column = FloatCol
+    entity = Famille
+    label = u"Allocation logement pour les primo-accédants plafond mensualité"
+    reference = u"https://www.legifrance.gouv.fr/affichCodeArticle.do;jsessionid=41AE9B6BDDC571E9CB28A605FDA0F639.tpdila22v_1?idArticle=LEGIARTI000006737216&cidTexte=LEGITEXT000006073189&dateTexte=20170811&categorieLien=id&oldAction=&nbResultRech="


Je crois que pour les accédants c'est plutôt celui-là

… aides au logement pour les primo-accédants

fpagnoux · 2017-08-11T14:49:52Z

Prêt à merger 🚀 !

GOMBERT Damien added 5 commits August 10, 2017 16:56

Calcul des AL pour les primo accédant

06905e1

Création fonction pour K

754c0c7

Ajout des plafonds de mensualité pour les primo accédant

e010752

Fiabiliser le calcul des al pour les primo accédant

ddd1146

- Ajout du calcul de ressources plafonné - Mise à jour des tests avec fiabilité à 5% (voir avec un expert métier pour améliorer la fiabilité)

Linting

3e99030

MattiSG added the contribution label Aug 11, 2017

benjello reviewed Aug 11, 2017

View reviewed changes

fpagnoux requested changes Aug 11, 2017

View reviewed changes

GOMBERT Damien added 2 commits August 11, 2017 15:10

Ajout de tests sur l'aide au logement pour les primo accédant

5eb4bf9

Ajout de date de début pour le calcul des aides au logement dans le c…

537c419

…as d'un primo accédant

Prise en compte des commentaires de la revue du code

bfb9b4f

fpagnoux requested changes Aug 11, 2017

View reviewed changes

GOMBERT Damien and others added 3 commits August 11, 2017 16:49

Mise à jour de changelog et de la version

2f3b29f

Mis d'une référence législative pour le calcul du plafond mensuel des…

d07d7c3

… aides au logement pour les primo-accédants

Add new variable to cache blacklist

7f1f001

fpagnoux force-pushed the al_primo_accession branch from 37ac5e5 to 7f1f001 Compare August 11, 2017 14:49

fpagnoux approved these changes Aug 11, 2017

View reviewed changes

fpagnoux merged commit 8598eb5 into openfisca:master Aug 11, 2017

fpagnoux removed the contribution label Aug 11, 2017

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Al primo accession #801

Al primo accession #801

dgombert commented Aug 11, 2017 •

edited

Loading

benjello left a comment •

edited by fpagnoux

Loading

fpagnoux left a comment •

edited

Loading

fpagnoux Aug 11, 2017

fpagnoux Aug 11, 2017

fpagnoux Aug 11, 2017

fpagnoux Aug 11, 2017

fpagnoux Aug 11, 2017

fpagnoux Aug 11, 2017

fpagnoux Aug 11, 2017

fpagnoux Aug 11, 2017

fpagnoux Aug 11, 2017

fpagnoux Aug 11, 2017

fpagnoux Aug 11, 2017

fpagnoux commented Aug 11, 2017

fpagnoux left a comment

fpagnoux Aug 11, 2017

fpagnoux commented Aug 11, 2017

Al primo accession #801

Al primo accession #801

Conversation

dgombert commented Aug 11, 2017 • edited Loading

benjello left a comment • edited by fpagnoux Loading

Choose a reason for hiding this comment

fpagnoux left a comment • 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

fpagnoux commented Aug 11, 2017

fpagnoux left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

fpagnoux commented Aug 11, 2017

dgombert commented Aug 11, 2017 •

edited

Loading

benjello left a comment •

edited by fpagnoux

Loading

fpagnoux left a comment •

edited

Loading