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

Revalorisation de la CMUc-ACS au 1er avril 2018 #992

Merged
merged 13 commits into from
Aug 10, 2018

Conversation

frtomas
Copy link
Contributor

@frtomas frtomas commented May 25, 2018

  • Évolution du système socio-fiscal.
  • Périodes concernées : à partir du 01/04/2018.
  • Zones impactées :
  • parameters/cmu
  • prestations/minima_sociaux/cmu
  • prestations/minima_sociaux/rsa
  • Détails :
    • Revalorise les plafonds de ressources pour bénéficier des aides CMU-c et ACS en date du 01/04/2018.
    • Premier lot de la fiabilisation de la CMUc-ACS.
    • Regroupe les modifications d'une autre PR (Fiabilisation CMUc-ACS Lot 1 #1006).

Ces changements :

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

@frtomas frtomas force-pushed the msa_reval_cmuc_acs_lot1 branch from 87078d8 to dd38ebc Compare May 25, 2018 08:53
@guillett
Copy link
Member

@frtomas merci pour cette PR.

Les modifications dans openfisca_france/model/prestations/minima_sociaux/rsa.py (ie. rmi -> rsa dans les paramètres utilisés) sont-elles en lien avec le sujet de cette PR ?
Les tests ajoutés ne semblent pas y faire référence.

@frtomas
Copy link
Contributor Author

frtomas commented May 25, 2018

Bonjour @guillett ,
@marchand-laetitia-msa a identifié que la valeur rmi utilisée correspond en fait a la valeur montant_de_base_du_rsa, excepté qu'elle n'est pas à jour. Elle a donc fait ce basculement sur le RSA.

@benjello
Copy link
Member

@guillett @marchand-laetitia-msa : pourriez-vous vérifiez que cela n'a pas d'impact sur les périodes antérieures qui ne sont pas ou mal testées ? Mais toute clarification est bienvenue, merci pour ce travail !

@jmdallais
Copy link

@frtomas

@guillett @marchand-laetitia-msa : pourriez-vous vérifiez que cela n'a pas d'impact sur les périodes antérieures qui ne sont pas ou mal testées ? Mais toute clarification est bienvenue, merci pour ce travail !

@frtomas
Copy link
Contributor Author

frtomas commented May 25, 2018

Pour plus de clarté sur les deux parameters en questions :
rmi
montant_du_rsa

@benjello c'est peut-être à cause du vendredi après midi mais je ne suis pas sur d'avoir bien compris la question.
En revanche, en regardant un peu le reste du fichier rsa.py, je vois qu'on retrouve cette distinction rmi/rsa un peu partout, gérée de deux manières différentes.
Un test sur la date :
# Les parametres ont changé de nom au moment où le RMI est devenu le RSA
if period.start.date >= date(2009, 6, 1): ... else:

ou une nouvelle méthode dont la date de début de validité correspond :
def formula_2009_06_01(famille, period):
On pourrait donc probablement changer la date de validité de cette nouvelle formule pour correspondre avec le reste de ce qui existe dans ce fichier.

@benjello
Copy link
Member

@frtomas : je n'ai pas dû être très clair ;-)

Sauf si le rmi a survécu au RSA en 2009 (cc'est possible des prestations sont en extinction pendant un certain temps), les deux séries que tu mentionnes doivent coïncider (à vérifier ce n'est pas garanti) après la création du RSA.

Pour éviter cet overlap je propose de virer de rmi les rsa_socle que l'on y avait mis par facilité. Et de vérifier que les formules du passé y survivent.

Est-ce vraiment plus clair ?

@frtomas
Copy link
Contributor Author

frtomas commented May 25, 2018

@jmdallais

@frtomas : je n'ai pas dû être très clair ;-)

Sauf si le rmi a survécu au RSA en 2009 (cc'est possible des prestations sont en extinction pendant un certain temps), les deux séries que tu mentionnes doivent coïncider (à vérifier ce n'est pas garanti) après la création du RSA.

Pour éviter cet overlap je propose de virer de rmi les rsa_socle que l'on y avait mis par facilité. Et de vérifier que les formules du passé y survivent.

Est-ce vraiment plus clair ?

@benjello
Copy link
Member

@frtomas : je n'avais pas lu ta réponse complètement. Tu as compris l'enjeu: rendre un peu de cohérence sans tout casser à la transition rmi rsa.

@frtomas frtomas force-pushed the msa_reval_cmuc_acs_lot1 branch from dd38ebc to 54d8d4f Compare May 29, 2018 07:57
@frtomas frtomas added the contrib:msa Identification des sujets MSA label May 30, 2018
@frtomas frtomas force-pushed the msa_reval_cmuc_acs_lot1 branch 2 times, most recently from e360713 to 69694b8 Compare June 5, 2018 12:24
@frtomas frtomas requested a review from guillett June 5, 2018 12:25
'retraite_nette',
]

has_ressources_substitution = individu('rsa_has_ressources_substitution', period)
Copy link
Contributor

Choose a reason for hiding this comment

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

Est-il possible de trouver un nom en français pour cette variable ?
En général, il est préférable que les variables soient en français dans OpenFisca-France.
Cela permet de maintenir le plus de cohérence possible :)

@@ -91,6 +91,59 @@ class rsa_base_ressources_individu(Variable):
entity = Individu
definition_period = MONTH
Copy link
Contributor

Choose a reason for hiding this comment

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

Est-il possible de rajouter une référence legislative ?


has_ressources_substitution = individu('rsa_has_ressources_substitution', period)

# Les revenus pros interrompus au mois M sont neutralisés s'il n'y a pas de revenus de substitution.
Copy link
Contributor

Choose a reason for hiding this comment

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

🙌 Merci pour ce commentaire, il m'a bien aidé à comprendre la formule 🙌

has_ressources_substitution = individu('rsa_has_ressources_substitution', period)

# Les revenus pros interrompus au mois M sont neutralisés s'il n'y a pas de revenus de substitution.
revenus_pro = sum(
Copy link
Contributor

Choose a reason for hiding this comment

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

proposition :
Mettre les oprérateurs en début de ligne

revenus_pro = sum(
                individu(type_revenu, period.last_3_months, options = [ADD])
                * not_(
                    (individu(type_revenu, period) == 0)
                    * (individu(type_revenu, period.last_month) > 0)
                    * not_(has_ressources_substitution)
                    )
                for type_revenu in types_revenus_pros
                )

'rsa_indemnites_journalieres_hors_activite',
]

# Les revenus non-pro interrompus au mois M sont neutralisés dans la limite d'un montant forfaitaire,
Copy link
Contributor

Choose a reason for hiding this comment

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

Encore un fois, super commentaires, merci !

@@ -115,3 +115,208 @@
enfants: [2, 3, 4, 5]
output_variables:
cmu_c_plafond: 16574 # 8723 * (1 + 0.25 + 0.15 + 0.3 + 0.2)

# msa_reval_cmuc_acs_lot1 : Cas 1
- name: "CMU-C: CMU-C Ok demandeur plus de 25 ans avec droits ouverts, percevant des salaires pour un motant inférieur à 8810 euros, résident métropole"
Copy link
Contributor

Choose a reason for hiding this comment

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

montant


# msa_reval_cmuc_acs_lot1 : Cas 2 (à revoir cmu_c devrait être false?)
# TODO: cmuc_c: False devrait peut-être passer
- name: "ACS: ACS Ok demandeur plus de 25 avec droits ouverts, percevant des salaires pour un motant supérieur à 8810 euros et inférieur à 11894, résident métropole"
Copy link
Contributor

Choose a reason for hiding this comment

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

montant

cmu_c: True

# msa_reval_cmuc_acs_lot1 : Cas 5
- name: "ACS: ACS Ok demandeur plus de 25 Assuré + de 25 ans avec droits ouverts, percevant des salaires + indemnités chômages
Copy link
Contributor

Choose a reason for hiding this comment

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

j'ai l'impression qu'il y a un double espace entre "+ de 25 ans" et "avec droits ouverts"

cmu_c: True

# msa_reval_cmuc_acs_lot1 : Cas 6
- name: "CMU-C: CMU-C Ok demandeur assuré au RSA ayant perçu des ressources au cours des 12 derniers mois"
Copy link
Contributor

Choose a reason for hiding this comment

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

J'ai l'impression qu'il y a un double espace entre " ressources" et "au cours "

CHANGELOG.md Outdated
* Évolution du système socio-fiscal.
* Périodes concernées : à partir du 01/04/2018.
* Zones impactées :
- `parameters/cmu`
Copy link
Contributor

Choose a reason for hiding this comment

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

Il me semble que le calcul de la rsa_base_ressource_individu a aussi été affecté

Copy link
Contributor Author

Choose a reason for hiding this comment

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

effectivement, zone ajoutée

Copy link
Contributor

Choose a reason for hiding this comment

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

prestations/minima_sociaux/ressources/rsa non ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

en fait c'est plutot prestations/minima_sociaux/rsa, je viens de corriger ça

Copy link
Contributor

Choose a reason for hiding this comment

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

Il faudrait faire un seul version bump pour les deux :

## 22.2.2 [#992](https://github.com/openfisca/openfisca-france/pull/992)

* Évolution du système socio-fiscal.
* Périodes concernées : à partir du 01/04/2018.
* Zones impactées :
  - `prestations/minima_sociaux/cmu`
  - `prestations/minima_sociaux/aah`
  - `parameters/cmu`
* Détails :
  - Revalorise les plafonds de ressources pour bénéficier des aides CMU-c et ACS en date du 01/04/2018.
  - Fiabilise le calcul de la CMUc-ACS en complétant les ressources prises en compte

Copy link
Contributor

Choose a reason for hiding this comment

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

Je ne vois pas de mentions dans le changelog de la mise à jour de la caah. Est-ce normal ?

rev_cap_bar = max_(0, individu.foyer_fiscal('rev_cap_bar', period.last_3_months, options = [ADD]))
rev_cap_lib = max_(0, individu.foyer_fiscal('rev_cap_lib', period.last_3_months, options = [ADD]))
retraite_titre_onereux = individu.foyer_fiscal('retraite_titre_onereux', period.last_3_months, options = [ADD])
revenus_foyer_fiscal = rev_cap_bar + rev_cap_lib + retraite_titre_onereux
Copy link
Contributor

Choose a reason for hiding this comment

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

Est-ce que c'est normal que les revenus comme les loyers ne soient pas mentionnés ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Cette partie n'a pas nécessite de modification et est reprise directement de la formule précédente.

@frtomas frtomas force-pushed the msa_reval_cmuc_acs_lot1 branch 2 times, most recently from d092a2e to 105236b Compare June 6, 2018 13:58
@Anna-Livia
Copy link
Contributor

@frtomas Est-ce que tu as pu trouver une référence legislative ?

@frtomas
Copy link
Contributor Author

frtomas commented Jun 6, 2018

Bonjour @Anna-Livia
j'ai ajouté une référence législative sur le rsa_base_ressources_individu, je n'ai pas réussi a trouver mieux.

CHANGELOG.md Outdated
* Évolution du système socio-fiscal.
* Périodes concernées : à partir du 01/04/2018.
* Zones impactées :
- `parameters/cmu`
Copy link
Contributor

Choose a reason for hiding this comment

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

prestations/minima_sociaux/ressources/rsa non ?

@frtomas frtomas force-pushed the msa_reval_cmuc_acs_lot1 branch 2 times, most recently from 4300a1c to ed83ebb Compare June 7, 2018 06:55

# msa_reval_cmuc_acs_lot1 : Cas 2 (à revoir cmu_c devrait être false?)
# TODO: cmuc_c: False devrait peut-être passer
- name: "ACS: ACS Ok demandeur plus de 25 avec droits ouverts, percevant des salaires pour un montant supérieur à 8810 euros et inférieur à 11894, résident métropole"
Copy link
Contributor

Choose a reason for hiding this comment

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

Dans le test, les salaires sont de 5000/an, or le test décrit des salaires supérieurs à 8810. Est-ce que c'est normal ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ces tests avaient été écrit avec d'anciens formats de fichiers de tests difficile à comprendre, je les ai repris d'après les fichiers plus récents.
Il restera parfois quelques légers écarts entre un montant de l'intitulé et ceux du tests effectifs, mais les résultats de tests n'en sont pas affectés.

# msa_reval_cmuc_acs_lot1 : Cas 4 (à revoir)
# TODO: Résultats différents de ce que nous attendons
- name: "CMU-C: CMU-C Ok demandeur plus de 25 avec droits ouverts, percevant des salaires + indemnités journalières
Foyer composé de 4 personnes (2 enfants + 20 ans et conjoint avec activité percevant des revenus), locataire avec aide au logement (2000 euros), résident métropole"
Copy link
Contributor

Choose a reason for hiding this comment

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

Je ne vois pas apparaitre les 2000 d'aide au logement dans le test.
Est ce que ce sont les 2200 EUR de prestation familiales ?


# msa_reval_cmuc_acs_lot1 : Cas 5
- name: "ACS: ACS Ok demandeur plus de 25 Assuré + de 25 ans avec droits ouverts, percevant des salaires + indemnités chômages
Foyer composé de 6 personnes (3 enfants - 20 ans, 1 enfant +20ans et conjoint avec activité percevant des revenus et du chômage), propriétaire avec aide au logement (2000 euros), résident métropole"
Copy link
Contributor

Choose a reason for hiding this comment

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

Je ne vois pas apparaitre les 2000 d'aide au logement dans le test.
Est ce que ce sont les 1350 EUR de prestation familiales ?

@frtomas frtomas force-pushed the msa_reval_cmuc_acs_lot1 branch 3 times, most recently from d418f02 to 52b0d75 Compare June 7, 2018 12:24
@Anna-Livia
Copy link
Contributor

@frtomas Je n'arrive pas à mettre la main sur l'autre PR qui avait les mêmes modifications.
Il faudrait la mentionner dans la decription de cette PR et mettre à jour la descrption de la PR avec le nouveau contenu.

D'autre part, j'ai vu que certains de mes anciens commentaires étaient sans réponse. n"hésite pas à me dire si tu n'es pas d'accord avec les demandes de modifications.

@Anna-Livia
Copy link
Contributor

@frtomas Aurais-tu du temps la semaine prochaine pour faire un point ? Peut-être pairer pour mettre ces modifications sur master ?

@frtomas frtomas force-pushed the msa_reval_cmuc_acs_lot1 branch 3 times, most recently from 398a310 to 398b48c Compare August 7, 2018 14:24
@frtomas frtomas requested a review from Anna-Livia August 7, 2018 14:26

# Revenus du foyer fiscal que l'on projette sur le premier invidividus
revenus_capitaux = (
max_(0, individu.foyer_fiscal('rev_cap_bar', period.last_3_months, options = [ADD]))
Copy link
Contributor

Choose a reason for hiding this comment

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

En V22, rev_cap_bar est devenu revenus_capitaux_prelevement_bareme
rev_cap_lib --> revenus_capitaux_prelevement_liberatoire

@@ -0,0 +1,654 @@
# Les tests sont en erreurs car la PR concernant les revalorisation CMUC-ACS n'est pas encore mergée.
Copy link
Contributor

Choose a reason for hiding this comment

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

Est-ce toujours d'actualité ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Non effectivement, c'est un oubli de commentaire

@Anna-Livia
Copy link
Contributor

@frtomas --> retraite_titre_onereux en rente_viagere_titre_onereux :)

@frtomas
Copy link
Contributor Author

frtomas commented Aug 8, 2018

@Anna-Livia il reste un test en erreur, un test de mes-aides. Mais cette erreur est pour moi normale, parmi les modifications effectuées, l'une d'elles concerne l'abattement chômage pour lequel l'éligibilité n’était pas complète. Ce test portant précisément sur l'éligibilité n'est donc plus d'actualité.

@Anna-Livia
Copy link
Contributor

@frtomas Tu as raison. Voudrais-tu vérifier la valeur attendue pour ce test mes-aides (Le test runner dit 350, est-ce la bonne valeur ?) et corriger le fichier yaml ?

@frtomas
Copy link
Contributor Author

frtomas commented Aug 8, 2018

@Anna-Livia oui, ce montant est cohérent, il correspond bien au montant ACS pour une personne âgée de 50 à 59 ans (ici 51 ans), la personne est rendue éligible à l'"abattement chômage" par sa perception d'indemnité journalières de maladie, et cet abattement fait basculer la personne en dessous du plafond acs.
Je vais changer la description et le résultat du test pour refléter la réalité du cas tout de suite.

@Anna-Livia
Copy link
Contributor

🎉 🎉 🎉 Un dernier rebase sur la dernière version de master et c'est bon :)

@frtomas frtomas force-pushed the msa_reval_cmuc_acs_lot1 branch from 80a0ef8 to 0385a37 Compare August 10, 2018 12:14
@frtomas
Copy link
Contributor Author

frtomas commented Aug 10, 2018

@Anna-Livia c'est fait ^^

@guillett guillett merged commit e5c9f40 into master Aug 10, 2018
@guillett guillett deleted the msa_reval_cmuc_acs_lot1 branch August 10, 2018 13:07
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.

7 participants