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.

Sign up for GitHub

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

Jump to bottom

Revalorisation des plafonds des ressources applicables Métropole - PAJE 01/2018 #972

Merged
merged 14 commits into from
Apr 25, 2018
Merged

Revalorisation des plafonds des ressources applicables Métropole - PAJE 01/2018 #972

merged 14 commits into from
Apr 25, 2018

Conversation

frtomas
Copy link
Contributor

@frtomas frtomas commented Apr 17, 2018 •
edited
Loading

  • Évolution du système socio-fiscal.
  • Périodes concernées : à partir du 01/01/2018.
  • Zones impactées :
    • prestations/prestations_familiales/paje/base/apres_2014/taux_partiel
    • prestations/prestations_familiales/paje/base/apres_2014/taux_plein
    • prestations/prestations_familiales/paje
  • Détails :
    • Revalorisation des plafonds de ressources pour la PAJE en date du 01/01/2018
    • Prise en compte du plus jeune enfant dans le calcul de l'éligibilité à la PAJE
    • Mise à jour de la formule de calcul de la PAJE naissance en date du 01/01/2015

Ces changements :

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

@frtomas frtomas changed the title Msa reval paje Revalorisation des plafonds des ressources applicables Métropole - PAJE 01/2018 Apr 18, 2018
@guillett
Copy link
Member

@frtomas top le rebase :-).

Je te laisse en faire un nouveau suite à l'intégration sur master de #971.

Puis-je te laisser modifier la version dans setup.py et le CHANGELOG comme je l'ai fait pour les PRs précédentes ? (AF, CF).

@frtomas
Copy link
Contributor Author

frtomas commented Apr 20, 2018 •
edited
Loading

Pas de soucis @guillett, je m'en occupe.

benjello
benjello reviewed Apr 20, 2018
View reviewed changes
openfisca_france/model/prestations/prestations_familiales/paje.py Outdated
age_plus_jeune_enfant = min(famille.members('age', period))
est_cadet = famille.members('age', period) == age_plus_jeune_enfant

enfant_elig_avant_avril_2014 = famille.any(enfant_eligible_ne_avant_avril_2014() * est_cadet)
Copy link
Member

Choose a reason for hiding this comment

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

Merci @frtomas pour ces nouvelles contributions.
Je suis partisan de garder des noms le moins ambigu possible (et avec le moins d'abbréviations possibles) même s'ils sont parfois très longs.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

C'est noté @benjello, j'ai changé la variable de la condition pour la rendre plus claire et la faire correspondre avec 'age_plus_jeune_enfant'

@frtomas frtomas requested a review from guillett April 24, 2018 12:40
guillett
guillett requested changes Apr 25, 2018
View reviewed changes
Copy link
Member

@guillett guillett left a comment •
edited
Loading

Choose a reason for hiding this comment

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

J'ai fait quelques commentaires. Ping moi à nouveau quand tu as pu regarder ça. Merci beaucoup en tout cas pour cette nouvelle contribution !

openfisca_france/model/prestations/prestations_familiales/paje.py Outdated
plaf_tx = (nbenf > 0) + P.paje.base.avant_2014.taux_majoration_2_premiers_enf * min_(nbenf, 2) + P.paje.base.avant_2014.taux_majoration_3eme_enf_et_plus * max_(nbenf - 2, 0)
majo = isole | biactivite
plaf = P.paje.base.avant_2014.plafond_ressources_0_enf * plaf_tx + (plaf_tx > 0) * P.paje.base.avant_2014.majoration_biact_parent_isoles * majo
elig = (base_ressources <= plaf)
Copy link
Member

Choose a reason for hiding this comment

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

Les trois variables majo, plaf, elig devraient avoir des noms plus explicites.

CHANGELOG.md Outdated
* Zones impactées :
- parameters\prestations\prestations_familiales\paje\base\apres_2014\taux_partiel\
- parameters\prestations\prestations_familiales\paje\base\apres_2014\taux_plein\
- model\prestations\prestations_familiales\paje.py
Copy link
Member

Choose a reason for hiding this comment

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

Le format des zones impactées devrait être le suivant :

  • prestations/prestations_familiales/paje/base/apres_2014
  • prestations/prestations_familiales/paje

@frtomas frtomas requested a review from guillett April 25, 2018 12:54
guillett
guillett approved these changes Apr 25, 2018
View reviewed changes
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.

Merci pour ces modifications. J'ai apporté quelques modifications au CHANGELOG cf. d136927.

CHANGELOG.md Outdated
### 21.8.6 [#972](https://github.com/openfisca/openfisca-france/pull/972)

* Évolution du système socio-fiscal.
* Périodes concernées : à partir du 01/01/2018.
Copy link
Member

Choose a reason for hiding this comment

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

à partir du 01/01/2015

@guillett guillett merged commit 22c82b0 into master Apr 25, 2018
@guillett guillett deleted the msa_reval_paje branch April 25, 2018 13:52
@bonjourmauko bonjourmauko added contrib:msa Identification des sujets MSA kind:evolution and removed contrib:msa Identification des sujets MSA labels Jul 23, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@benjello benjello benjello left review comments

@guillett guillett guillett approved these changes

Assignees
No one assigned
Labels
contrib:msa Identification des sujets MSA kind:evolution
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@frtomas @guillett @benjello @bonjourmauko