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

Params/maj last value still valid on ppa #2464

Merged
merged 9 commits into from
Mar 10, 2025

Conversation

clallemand
Copy link
Contributor

  • Changement mineur.
  • Périodes concernées : toutes.
  • Zones impactées : openfisca_france/parameters/prestations_sociales/solidarite_insertion/minima_sociaux/ppa.*
  • Détails :
    • Corrige et ajoute des références pour certains paramètres de la prime d'activité
    • Mets à jour des last_value_still_valid_on pour certains paramètres de la prime d'activité

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

  • Modifient des éléments non fonctionnels de ce dépôt (par exemple modification du README).

2015-10-01:
2016-01-01:
Copy link
Contributor

Choose a reason for hiding this comment

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

En mettant au premier janvier, il n'est pas possible de calculer la PA versée au premier janvier, qui pourtant existait. La valeur a été changée trois fois au cours de l'année écoulée je crois, donc dans tous les cas rajouter une documentation plus claire me semble nécessaire.

Copy link
Contributor

Choose a reason for hiding this comment

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

Et même si tu arrives à me reconvaincre, il faut apporter une modification sur les formules similaire à celle qu'avait apporté Mahdi : https://github.com/openfisca/openfisca-france/pull/2214/files

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oups désolée, je remets la date d'octobre 2015 et je rajoute une note ! Merci pour ton retour

Comment on lines -27 to +29
2019-01-01:
2018-10-01:
Copy link
Contributor

Choose a reason for hiding this comment

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

Et pour 2018 le cas est encore plus clair, étant donné que c'est bien à partir de la PA due au titre d'octobre que la réforme prend effet (y compris avec une dimension rétroactive)

Copy link
Contributor

Choose a reason for hiding this comment

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

J'avais noté dans ma PR précédente : "Fait passer date de réforme à octobre 2018, date où les salaires concernés par les nouveaux calculs de PA débutent (c'est explicite dans le décrets associés).". (C'est amusant de voir que j'avais reviewé en acceptant la PR de Mahdi avant de revenir en arrière dans une PR que tu avais reviewé, avant de vouloir à ton tour revenir sur le sujet : mais l'avantage, c'est que comme on n'est pas beaucoup à le faire, on finir par converger probablement)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ici @sylvainipp j'ai juste modifié la date de la référence pour qu'elle soit la même que celle de la value. Je n'ai pas changé la date de la value, c'est bon pour toi ? Je ne suis pas sûr de comprendre ton commentaire

Copy link
Contributor

@sylvainipp sylvainipp Mar 10, 2025

Choose a reason for hiding this comment

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

Ah oui pardon, j'ai lu rapidement avant de partir et juste après avoir vu le cas d'octobre 2015, mais bien sûr c'est bon pour moi

@clallemand
Copy link
Contributor Author

@sylvainipp si les changements que j'ai fait sont ok pour toi je veux bien que tu me valides la PR :)

Copy link
Contributor

@sylvainipp sylvainipp left a comment

Choose a reason for hiding this comment

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

Tout me va, je propose juste certains articles en 2016 plutôt que leur version ultérieure ! Désolé pour la confusion sur la réforme de 2024.

Co-authored-by: Sylvain Duchesne <[email protected]>
@clallemand clallemand merged commit 447b833 into master Mar 10, 2025
38 checks passed
@clallemand clallemand deleted the params/maj_last_value_still_valid_on_ppa branch March 10, 2025 10:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants