Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fiabilisation AL - Mise à jour des paramètres des aides logement #1082

Merged
merged 4 commits into from
Aug 31, 2018

Conversation

frtomas
Copy link
Contributor

@frtomas frtomas commented Aug 21, 2018

  • Évolution du système socio-fiscal.
  • Périodes concernées : à partir du 01/10/2017.
  • Zones impactées : parameters/prestations/aides_logement.
  • Détails :
    • Mise à jour des paramètres utilisés dans le calcul de l'aide au logement.
    • Correction mineure pour éviter un montant d'aide au logement négatif.

Ces changements :

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

@frtomas frtomas added the contrib:msa Identification des sujets MSA label Aug 21, 2018
@frtomas frtomas changed the title Remise à jours des parametres de l'aide au logement Fiabilisation AL - Mise à jour des paramètres des aides logement Aug 21, 2018
@frtomas frtomas requested a review from Anna-Livia August 21, 2018 08:30
@benjello
Copy link
Member

benjello commented Aug 21, 2018

Merci beaucoup @frtomas pour cette mise à jour !
La cohérence temporelle est importante pour openfisca. Je me permets donc quelques requêtes.
Avez-vous vérifié qu'entre les nouvelles valeurs et les plus anciennes, il n'y a pas eu de changement ?
Si ce n'est pas le cas pourriez-vous le faire si cela est simple pour vous ?
Dans le cas où cela vous serait impossible, pourriez-vous signaler les plages temporelles non vérifiées en commentaire dans les fichiers (ou dans tout autre lieu plus adapté que ne manqueront pas de nous indiquer @guillett ou @Anna-Livia ? ;-) ) ?

Par avance merci !

@frtomas
Copy link
Contributor Author

frtomas commented Aug 21, 2018

@benjello il y a définitivement des valeurs manquantes pour les périodes intermédiaires et il nous serait effectivement compliqué de les alimenter également, mais je vais refaire une passe dessus pour les identifier.
Cordialement.

@benjello
Copy link
Member

Merci @frtomas. S'il est possible de les noter dans les fichiers et de les rassembler dans un seul ticket également ce sera une contribution particulièrement bienvenue.

@frtomas frtomas removed the contrib:msa Identification des sujets MSA label Aug 22, 2018
@ThibaultCCMSA
Copy link
Collaborator

Bonjour @benjello. A court terme nous devons nous concentrer sur nos travaux en cours pour suivre nos deadlines mais nous pourrons revenir dessus une fois ces derniers menés. Nous ajoutons un item fiabilisation à notre liste :)

@@ -41,3 +41,5 @@ values:
value: 398.29
2014-09-01:
value: 400.56
2014-09-01:
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Je pense que c'est 2017-10-01 la période :)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@guillet effectivement, merci, il y en a eu un raté

@frtomas frtomas added the contrib:msa Identification des sujets MSA label Aug 23, 2018
Copy link
Member

@guillett guillett left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

J'ai ajouté quelques références qu'il serait bien d'ajouter.

@@ -32,3 +32,5 @@ values:
value: 11.99
2014-10-01:
value: 12.07
2017-10-01:
value: 12.16
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@@ -32,3 +32,5 @@ values:
value: 52.93
2014-10-01:
value: 53.27
2017-10-01:
value: 53.67
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@@ -31,3 +31,5 @@ values:
value: 26.46
2014-10-01:
value: 26.61
2017-10-01:
value: 26.83
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@@ -10,3 +10,5 @@ values:
value: 24.0
2007-01-01:
value: 15.0
2017-10-01:
value: 10.0
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@@ -32,3 +32,5 @@ values:
value: 352.92
2015-10-01:
value: 353.2
2017-10-01:
value: 355.85
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@@ -19,3 +19,5 @@ values:
value: 258.09
2014-10-01:
value: 259.56
2017-10-01:
value: 261.72
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@@ -35,3 +35,5 @@ values:
value: 127.88
2014-10-01:
value: 128.61
2017-10-01:
value: 129.68
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@@ -35,3 +35,5 @@ values:
value: 82.13
2014-10-01:
value: 82.6
2017-10-01:
value: 83.29
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@@ -41,3 +41,5 @@ values:
value: 401.01
2014-09-01:
value: 403.3
2017-10-01:
value: 406.65
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@@ -25,3 +25,5 @@ values:
value: 254.69
2014-09-01:
value: 256.14
2017-10-01:
value: 258.26
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@frtomas
Copy link
Contributor Author

frtomas commented Aug 29, 2018

@guillet j'ai ajouté les références et corrigé deux petits soucis révélés par les tests

@guillett
Copy link
Member

@frtomas je n'ai pas commenté toutes les absences de référence, peux-tu faire une passe en autonomie ? La référence des nouveaux plafonds est très souvent la même. Merci.

@frtomas
Copy link
Contributor Author

frtomas commented Aug 29, 2018

@guillet je m'en occupe

@guillett
Copy link
Member

Super merci !

@frtomas
Copy link
Contributor Author

frtomas commented Aug 29, 2018

@guillett j'ai ajouté le maximum de références possibles mais il en reste 5 pour lesquelles je ne parviens pas à les trouver :
al_pac.yaml
nr_seuil.yaml
dar_5.yaml
dar_2a.yaml
dar_2b.yaml

@@ -10,3 +10,5 @@ values:
value: 1400.0
2013-01-01:
value: 1500.0
2017-10-01:
value: 1700.0
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

J'ai l'impression que cette revalorisation n'est pas correcte cf ELI: https://www.legifrance.gouv.fr/eli/arrete/2017/12/29/TERL1732837A/jo/article_1

Copy link
Contributor Author

@frtomas frtomas Aug 29, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah effectivement @guillett, s'il n'y a pas d'autre revalorisation visible sur cette donnée (et comme je l'ai dit plus tôt, je ne parviens pas à trouver moi même), c'est une erreur dans la fiche.
Je ne sais pas d'où est venue cette valeur de 1700, je vais faire remonter la question et la supprimer en attendant.

@@ -4,3 +4,6 @@ unit: /1
values:
2002-01-01:
value: 0.0281
2017-10-01:
reference: https://www.legifrance.gouv.fr/affichTexteArticle.do;jsessionid=66B22C3CDDE322749B2BFCEC8CDC94AC.tplgfr31s_3?idArticle=LEGIARTI000035672499&cidTexte=LEGITEXT000006074213&dateTexte=20180829
value: 0.0283
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Je ne vois pas le lien entre la référence et la valeur.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Après un long moment sur legifrance. La source pour les DOM est l'article 5 de l'arrêté du 26 décembre 2000 relatif à l'allocation de logement.

À priori il n'y a pas de revalorisation de ces taux là

https://www.legifrance.gouv.fr/affichTexteArticle.do;jsessionid=1B336613D677A8012B5EC663496EFF09.tplgfr31s_3?idArticle=LEGIARTI000035672548&cidTexte=LEGITEXT000005630325&dateTexte=20180829

Cela étant dit, peux-tu en supprimant les ajouts de cette PR

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@guillett J'ai eu la confirmation qu'il s'agissait d'une erreur, j'ai retiré ces mauvaises valeurs et j'ai ajouté les références aux valeurs existantes des taux pour les DOM.
En revanche je ne suis pas sur de comprendre ce que tu veux dire par :

ajouter les valeurs historisées via
ELI: https://www.legifrance.gouv.fr/eli/arrete/2013/1/14/FAMS1242779A/jo/article_1
et Arrêté 2007-07-10 art. 4 IV, art. 6 JORF 12 juillet 2007 en vigueur le 1er juillet 2007

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Je propose de profiter de tout le travail fait ces dernières heures pour fiabiliser maintenant les informations autours des taux DOM.

Par exemple
https://github.com/openfisca/openfisca-france/pull/1082/files#diff-17b605caa7480e5a2c8798ec31b3e87d

n'ai pas tout à fait correct.

En voilà une proposition :

description: Taux pour 1 adulte
reference:
  - Article D755-28 du code de la Sécurité Sociale
  - https://www.legifrance.gouv.fr/affichCodeArticle.do?cidTexte=LEGITEXT000006073189&idArticle=LEGIARTI000006738605&dateTexte=&categorieLien=cid
  - Article 5 de l'arrêté du 26 décembre 2000 relatif à l'allocation de logement
  - https://www.legifrance.gouv.fr/affichTexteArticle.do;jsessionid=16DF0CDF8FFBDA212DAB06A60F299208.tplgfr31s_3?idArticle=LEGIARTI000035672548&cidTexte=LEGITEXT000005630325&dateTexte=20180830
unit: /1
values:
  2001-01-01:
    reference: https://www.legifrance.gouv.fr/affichTexteArticle.do;jsessionid=16DF0CDF8FFBDA212DAB06A60F299208.tplgfr31s_3?idArticle=LEGIARTI000006791749&cidTexte=JORFTEXT000000388784&categorieLien=id&dateTexte=20070630
    value: 0.0268
  2013-01-01:
    reference: https://www.legifrance.gouv.fr/eli/arrete/2013/1/14/FAMS1242779A/jo/article_1
    value: 0.0281

Copy link
Contributor Author

@frtomas frtomas Aug 30, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok, je vois, personnellement je n'y vois pas de soucis, mais à voir dans un premier temps avec @jmdallais / @ThibaultCCMSA / @JenniferTelep.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Vu avec @guillett par tél. : On n'embarque pas cette demande pour la V2.1 PNDS (besoin de cas de tests entre autres).
@JenniferTelep et @ThibaultCCMSA , pouvez-vous créer l'ajouter à notre backlog et créer une issue que nous traiterons post V2.1 svp ?

@@ -4,3 +4,6 @@ unit: /1
values:
2002-01-01:
value: 0.0299
2017-10-01:
reference: https://www.legifrance.gouv.fr/affichTexteArticle.do;jsessionid=66B22C3CDDE322749B2BFCEC8CDC94AC.tplgfr31s_3?idArticle=LEGIARTI000035672499&cidTexte=LEGITEXT000006074213&dateTexte=20180829
value: 0.0315
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Je ne vois pas le lien entre la référence et la valeur.

@@ -4,3 +4,6 @@ unit: /1
values:
2002-01-01:
value: 0.0238
2017-10-01:
reference: https://www.legifrance.gouv.fr/affichTexteArticle.do;jsessionid=66B22C3CDDE322749B2BFCEC8CDC94AC.tplgfr31s_3?idArticle=LEGIARTI000035672499&cidTexte=LEGITEXT000006074213&dateTexte=20180829
value: 0.027
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Je ne vois pas le lien entre la référence et la valeur.

@@ -4,3 +4,6 @@ unit: /1
values:
2002-01-01:
value: 0.0217
2017-10-01:
reference: https://www.legifrance.gouv.fr/affichTexteArticle.do;jsessionid=66B22C3CDDE322749B2BFCEC8CDC94AC.tplgfr31s_3?idArticle=LEGIARTI000035672499&cidTexte=LEGITEXT000006074213&dateTexte=20180829
value: 0.0238
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Je ne vois pas le lien entre la référence et la valeur.

@frtomas
Copy link
Contributor Author

frtomas commented Aug 31, 2018

@guillett avec la partie discutée avec Jean-Marie décalée à plus tard, reste-t-il d'autres choses à revoir sur cette PR ?

@guillett
Copy link
Member

Je regarde

* Périodes concernées : à partir du 01/10/2017.
* Zones impactées : `parameters/prestations/aides_logement`.
* Détails :
- Mise à jour des paramètres utilisés dans le calcul de l'aide au logement.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Il faudrait ajouter :

  • Correction mineure pour éviter un montant d'aide au logement négatif.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@guillet c'est fait

@guillett
Copy link
Member

@frtomas je te laisse rebase et je merge ensuite.

@frtomas frtomas force-pushed the msa_fiab_param_al branch from 8e1deb5 to 3686b00 Compare August 31, 2018 12:49
@frtomas
Copy link
Contributor Author

frtomas commented Aug 31, 2018

@guillet rebase terminé.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
contrib:msa Identification des sujets MSA
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants