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

Ipp update charges deductibles #907

Merged
merged 49 commits into from
Mar 5, 2018
Merged

Conversation

Anna-Livia
Copy link
Contributor

Modifications faites pas @ClaireLeroyIPP
Les modifications de cette PR sont comparées à la branche ipp-update-bareme (#906 )
Elle est à rebaser une fois 906 approved

  • Évolution du système socio-fiscal
  • Périodes concernées : à partir du 01/01/2009.
  • Zones impactées : openfisca_france/model/prelevements_obligatoires/impot_revenu/charges_deductibles.py.
  • Détails :
    • Amélioration du calcul des dépenses de grosses réparations

Ces changements Ajoutent une fonctionnalité (par exemple ajout d'une variable) et Corrigent ou améliorent un calcul déjà existant.

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

* Évolution du système socio-fiscal.
* Périodes concernées : 2009 - 2016
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ajouter : Zones impactées : openfisca_france/model/prelevements_obligatoires/impot_revenu/charges_deductibles.py.

@claireleroy
Copy link
Contributor

Pour les tests @Anna-Livia, est-ce que je peux créer une issue commune à toutes les PR ? (Pour l'instant j'ai pas de tests vraiment pertinents en stock à rajouter - j'ai surtout des tests qui testent chaque cases une à une - et j'ai pas le temps tout de suite d'en créer, j'y reviendrais par la suite)

@fpagnoux fpagnoux force-pushed the ipp-update-baremes branch 2 times, most recently from 10f924e to 879d6f0 Compare February 27, 2018 00:15
@claireleroy claireleroy changed the base branch from ipp-update-baremes to master February 28, 2018 09:19
@Anna-Livia
Copy link
Contributor Author

@ClaireLeroyIPP pour les tests, OK pour une seule issue qui liste avec des checkbox tous les tests à livrer.

@Anna-Livia
Copy link
Contributor Author

@ClaireLeroyIPP Il reste deux remarques qui n'ont pas de réponse pour le moment il me semble :

  • Lister les variables créées dans le changelog
  • Ajouter une référence aux charges déductibles

@claireleroy
Copy link
Contributor

@Anna-Livia

  • Pour la liste des variables je t'ai répondu mais je sais pas pourquoi c'est caché dans la PR .. Je te remets ma réponse ici :

Anna-Livia 5 days ago Owner
Ajouter les noms des variables crées :)

ClaireLeroyIPP 5 days ago Member

Tu veux dire les variables de type "f6hn" etc. qui sont les nouvelles cases de reports ?
Dans ce cas précis, je n'en ai rajouté que 6 ou 7 donc ça parait ok mais si je fais le total sur toutes mes PR, j'ai rajouté la plupart des cases qui sont apparues dans les déclarations fiscales entre 2014 et 2016 et ça va faire quelque chose comme plus de 100 cases à rajouter à la main dans chaque Changelog, je t'avoue que ça me parait un peu long de rajouter chaque inputs variables, et le Changelog va devenir illisible..
Par contre lorsqu'il s'agit de variables intermédiaires (ex: la réduction d'impôt 'rpinel'), ça me parait plus pertinent de les rajouter oui.

  • A l'inverse pour la référence sur les charges déductibles, j'ai ajouté mais la PR ne l'affiche pas comme "outdated" il me semble

@Anna-Livia
Copy link
Contributor Author

@ClaireLeroyIPP Pour les noms des variables créées ok pour mentionner seulement les variables intermédiaires. Est-ce que ça aurait du sens de mettre quelque chose comme Ajout de variables d'input pour les cases de l'IR correspondantes. ?

@claireleroy
Copy link
Contributor

@Anna-Livia @benjello Je pense qu'on peut merger cette PR ?

@Anna-Livia Anna-Livia merged commit 7eda77c into master Mar 5, 2018
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.

3 participants