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

Prise en compte de la baisse de la cotisation AGS au 1er juillet #789

Merged
merged 7 commits into from
Aug 18, 2017
Merged

Prise en compte de la baisse de la cotisation AGS au 1er juillet #789

merged 7 commits into from
Aug 18, 2017

Conversation

Morendil
Copy link
Contributor

@Morendil Morendil commented Jul 2, 2017 •
edited
Loading

  • Évolution du système socio-fiscal.
  • Périodes concernées : à partir du 01/07/2017
  • Zones impactées : openfisca_france/parameters/cotsoc.xml, openfisca_france/parameters/prelevements_sociaux.xml.
  • Détails :
    • Prise en compte de la baisse de la cotisation AGS au 1er juillet.

Ces changements

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

@Morendil Morendil requested a review from fpagnoux July 7, 2017 06:24
@fpagnoux
Copy link
Member

Merci @Morendil pour cette mise à jour !

Aurais-tu une source législative à référencer ?

fpagnoux
fpagnoux reviewed Jul 10, 2017
View reviewed changes
Copy link
Member

@fpagnoux fpagnoux left a comment

Choose a reason for hiding this comment

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

Ok pour le code, 👍 pour le test.

Manque juste une référence législative 🙂

@Morendil
Copy link
Contributor Author

J'ai https://www.ags-garantie-salaires.org/actualites/items/taux-de-cotisation-015.html

Où est-il approprié de le renseigner ?

@MattiSG
Copy link
Member

MattiSG commented Jul 24, 2017

Il n'y a pas pour le moment de manière standard de le faire pour des paramètres législatifs 😞 cela devrait changer bientôt, avec openfisca/openfisca-core#523 🙂

@Morendil Pour le moment, le mieux est d'ajouter un attribut reference au nœud XML ajouté. Il faut essayer d'utiliser une URL stable autant que possible.

@MattiSG MattiSG requested review from benjello and fpagnoux July 24, 2017 15:17
@MattiSG
Copy link
Member

MattiSG commented Jul 24, 2017

Corrections effectuées. J'en ai profité pour ajouter de nombreuses références, merci @Morendil pour la référence originelle !

Cela m'a permis de détecter une erreur dans les barèmes IPP @benjello, et plusieurs dans la valeur apparemment en double côté OpenFisca.

Au passage, quelqu'un sait pourquoi ces valeurs sont en doublon ?!

@benjello
Copy link
Member

benjello commented Jul 24, 2017 •
edited
Loading

@MattiSG : Les valeurs dans les circulaires ne sont pas toujours celles qui sont utilisées en réalité.
Sur les taux de cotisations, notamment, j'aurais tendance à faire confiance à l'IPP.
Et vu les références indiquées ici, il est possible qu'il y ait eu ce genre d'inversion.
Cela ne rend que plus crucial, l'introduction d'un champ référence où l'on pourrait documenter cela.

@MattiSG MattiSG self-assigned this Jul 25, 2017
Anna-Livia
Anna-Livia requested changes Aug 17, 2017
View reviewed changes
CHANGELOG.md Outdated
@@ -1,5 +1,13 @@
# Changelog

### 18.5.6 - [#789](https://github.com/openfisca/openfisca-france/pull/789)
Copy link
Contributor

Choose a reason for hiding this comment

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

Il faut bumper la version et faire un rebase :)

setup.py Outdated
@@ -7,7 +7,7 @@

setup(
name = 'OpenFisca-France',
version = '18.5.3',
version = '18.5.6',
Copy link
Contributor

Choose a reason for hiding this comment

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

bump de version

@Morendil
Copy link
Contributor Author

Les tests échouent pour le moment car le XML est validé par rapport au schema legislation.xsd (qui fait partie de openfisca-core) et celui-ci n'autorise pas les attributs reference dans les balises VALUE.

@MattiSG MattiSG mentioned this pull request Aug 18, 2017
Closed
@MattiSG
Copy link
Member

MattiSG commented Aug 18, 2017

Bloqué par openfisca/openfisca-core#557.

@MattiSG MattiSG requested a review from Anna-Livia August 18, 2017 08:42
@MattiSG
Copy link
Member

MattiSG commented Aug 18, 2017

Ça sera probablement là début de semaine prochaine. Désolé pour le délai, on aurait du le voir venir plus tôt 😞

@MattiSG MattiSG mentioned this pull request Aug 18, 2017
Merged
benjello
benjello requested changes Aug 18, 2017
View reviewed changes
Copy link
Member

@benjello benjello left a comment

Choose a reason for hiding this comment

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

Je resignale ce que j'ai mis en commentaire pour la correction apportée par Matti qui ne me semble pas forcément correcte sans plus ample investigation.

@MattiSG
Copy link
Member

MattiSG commented Aug 18, 2017

Merci infiniment @benjello, je te confirme que la circulaire de l'Unedic est incohérente avec les archives. J'ai donc récupéré la valeur des barèmes de l'IPP, et corrigé la reference correspondante. Cette PR est-elle à présent ok pour toi ? 😃

benjello
benjello approved these changes Aug 18, 2017
View reviewed changes
Copy link
Member

@benjello benjello left a comment

Choose a reason for hiding this comment

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

Super boulot

@Anna-Livia Anna-Livia merged commit 88db88a into openfisca:master Aug 18, 2017
@fpagnoux fpagnoux mentioned this pull request Aug 31, 2017
Closed
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@MattiSG MattiSG MattiSG approved these changes

@benjello benjello benjello approved these changes

@Anna-Livia Anna-Livia Anna-Livia approved these changes

@fpagnoux fpagnoux Awaiting requested review from fpagnoux

Assignees

@MattiSG MattiSG

Labels
meta:legislation
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@Morendil @fpagnoux @MattiSG @benjello @Anna-Livia