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

Misc ipp fix #1340

Merged
merged 10 commits into from
Jun 13, 2019
Merged

Misc ipp fix #1340

merged 10 commits into from
Jun 13, 2019

Conversation

benjello
Copy link
Member

@benjello benjello commented Jun 9, 2019

  • Périodes concernées : toutes.
  • Zones impactées :
    • model\prelevements_obligatoires\impot_revenu\ir.py
    • model\revenus\
    • parameters\prelevements_sociaux\contributions\csg\remplacement
    • setup.py
  • Détails :
    • Supprime la variable cbnc_assc redondante avec f5qm
    • Corrige deux erreurs mineurs dans les cases 1TZ et 3VA
    • Mets à jour les paramètres relatifs à la CSG sur revenus de remplacement
    • Ajout d'une option d'installation casd-dev pour les activités de développement de l'IPP au CASD

Ces changements

  • Modifient l'API publique d'OpenFisca France (par exemple renommage ou suppression de variables).
  • Corrigent ou améliorent un calcul déjà existant.
  • Modifient des éléments non fonctionnels de ce dépôt (par exemple modification du README).

@benjello benjello self-assigned this Jun 9, 2019
@benjello
Copy link
Member Author

benjello commented Jun 9, 2019

@claireleroy : quelles sont les versions des dépendances commentées qu'on utilise au CASD ?

@benjello benjello requested a review from claireleroy June 10, 2019 13:36
@claireleroy
Copy link
Contributor

@benjello On utilise :

  • autopep8 version 1.3.2
  • flake8 on utilise pas
  • flake8-print non plus
  • pycodestyle version 2.4.0
  • pytest version 3.8.0
  • scipy version 1.1.0
  • yammlint on utilise pas

J'ai pas trop compris la question puisqu'en fait comme on commente des paquets c'est justement qu'on les utilise pas ....

@benjello
Copy link
Member Author

@claireleroy : j'essaie de faire des setup.py qui passent partout pour éviter de doublonner les branches.

@benjello benjello changed the title WIP: Misc ipp fix Misc ipp fix Jun 11, 2019
@sandcha
Copy link
Contributor

sandcha commented Jun 11, 2019

Cette PR n'est pas un changement mineur puisqu'elle supprime une variable (aka cbnc_assc).
Elle n'est pas rétro-compatible.

@benjello benjello requested a review from Morendil June 13, 2019 08:53
@benjello benjello merged commit 214bd2e into master Jun 13, 2019
@benjello benjello deleted the misc-ipp-fix branch June 19, 2019 19:01
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.

4 participants