-
Notifications
You must be signed in to change notification settings - Fork 99
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
Harmonisation des références et des short labels avec les barèmes IPP + raccourci short label trop longs #2408
base: master
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Quelques remarques pour un premier passage
openfisca_france/parameters/impot_revenu/calcul_impot_revenu/pv/actions_gratuites/taux3.yaml
Outdated
Show resolved
Hide resolved
openfisca_france/parameters/impot_revenu/calcul_impot_revenu/pv/actions_gratuites/taux4.yaml
Outdated
Show resolved
Hide resolved
openfisca_france/parameters/impot_revenu/calcul_impot_revenu/pv/pvce/index.yml
Outdated
Show resolved
Hide resolved
...mpot_revenu/calcul_reductions_impots/emploi_salarie_domicile/plafond_maximum_1ere_annee.yaml
Outdated
Show resolved
Hide resolved
...mpot_revenu/calcul_reductions_impots/emploi_salarie_domicile/plafond_maximum_1ere_annee.yaml
Outdated
Show resolved
Hide resolved
Bonjour, cette PR n'est elle pas en doublon avec la #2407 ? |
Hello! @clallemand je te répond ici car c'est cette PR qu'il faut suivre et pas la précédente. On s'est rendu compte suite à une tentative d'harmonisation des barèmes IPP avec OF-france que les short label étaient bcp trop longs (rendant les tableaux barèmes illisibles) d'où cette PR qui en réduit certains. En revanche, je n'ai pas touché aux restructurations des paramètres ni aux clarifications dans la description du paramètre simplement il faudrait qu'on discute du fait que le short label doit effectivement ne pas être long. |
@Lolajossipp @clallemand @eraviart : peut-être que cela vaudrait le coup de définir une longueur maximale et de la tester. |
Merci @Lolajossipp pour cette explication, je comprends mieux ! Effectivement ça peut être top de nous dire si vous identifier une longueur maximum à ne pas dépasser ! Après je suppose qu'il y aura toujours des cas spécifiques s'il s'agit d'une contrainte visuelle mais comme ça ça nous donnera un ordre d'idée ! |
Je pense que dans l'idée 100 caractères ça pourrait être bien pour les barèmes, tu penses que pour vous ça serait assez long? Qu'en penses tu @clallemand ? |
@Lolajossipp je pense qu'on peut s'adapter oui ! Est ce qu'on peut vous faire un retour d'ici une semaine sur les changements de cette PR ? |
4cb4eb4
to
d8b2d63
Compare
openfisca_france/parameters/impot_revenu/calcul_reductions_impots/dons/dons_cultuels/index.yaml
Outdated
Show resolved
Hide resolved
@Lolajossipp En effet fixer une limite du nombre de caractères serait me semble une idée tout à fait pertinente ! J'ai l'impression que certaines modifications seraient des simples retours en arrière par rapport aux changements des PR récentes, sans gain du nombre de caractères ; par exemple dans openfisca_france/parameters/impot_revenu/calcul_impot_revenu/plaf_qf/index.yaml, où la description et l'order qui sont modifiés, mais pas les shorts_labels nii les références (cf capture ci-dessous). |
Hello @b-michaud! Effectivement, j'ai corrigé les modifications en question dans le dernier commit. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Bonjour @Lolajossipp @sylvainipp,
Je viens de rentrer de congés, j'en profite pour vous souhaiter une belle nouvelle année 🎉
J'ai commencé à effectuer une relecture et mes retours vont surtout porter sur la discussion "Short_label" trop longs.
On avait bien conscience avec @b-michaud de proposer parfois des labels très longs : ces cas sont présents dans nos changements récents (sauf oubli) uniquement pour des cas très spécifiques, où raccourcir le label introduit des confusions.
J'ai essayé de vous détailler nos choix, là où vos propositions de changements me semblent introduire à nouveau de potentiels quiproquos pour les usagers.
Globalement, je dirai que je pense qu'effectivement, un label ne devrait pas faire plus de 100 caractères, voire dans la majorité des cas, pas plus de 60, mais je pense que dans certains cas précis (et plus fréquent pour l'IR qui est complexe), il est mieux que le label soit plus long s'il y a un risque d'incompréhension.
J'ai vu par ailleurs, que vous avez retiré la case fiscale des short_labels.
Je suis partagée, je pense que la case est un repère pour aider les utilisateur, mais c'est vrai que si celle-ci est mentionnée dans la description, c'est peut-être suffisant.
Avant de poursuivre ma relecture en détail sur tous les champs, je suis preneuse d'avoir déjà votre avis sur les quelques cas mentionnés ici.
...nce/parameters/impot_revenu/calcul_impot_revenu/pv/abattement_option/droit_commun/index.yaml
Outdated
Show resolved
Hide resolved
openfisca_france/parameters/impot_revenu/calcul_impot_revenu/pv/abattement_option/index.yaml
Outdated
Show resolved
Hide resolved
...rance/parameters/impot_revenu/calcul_impot_revenu/pv/abattement_option/majoration/index.yaml
Outdated
Show resolved
Hide resolved
openfisca_france/parameters/impot_revenu/calcul_impot_revenu/pv/actions_gratuites/index.yaml
Outdated
Show resolved
Hide resolved
openfisca_france/parameters/impot_revenu/calcul_impot_revenu/pv/actions_gratuites/taux2.yaml
Outdated
Show resolved
Hide resolved
openfisca_france/parameters/impot_revenu/calcul_impot_revenu/pv/actions_gratuites/taux3.yaml
Outdated
Show resolved
Hide resolved
openfisca_france/parameters/impot_revenu/calcul_impot_revenu/pv/actions_gratuites/taux4.yaml
Outdated
Show resolved
Hide resolved
openfisca_france/parameters/impot_revenu/calcul_impot_revenu/pv/bspce/taux_moins_3_ans.yaml
Outdated
Show resolved
Hide resolved
...ca_france/parameters/impot_revenu/calcul_impot_revenu/pv/bspce/taux_plus_3_ans_pre_2018.yaml
Outdated
Show resolved
Hide resolved
Bonne année @DorineLam !
On peut utiliser le champ documentation pour préciser les choses.
La case fiscale peut changer d'une année sur l'autre. Il vaut mieux me semble-t-il donner cette information dans la documentation (et la mettre à jour ...). |
Hello @DorineLam, bonne année à toi!! Un premier point est peut être que je ne sais pas comment vous utiliser les short_label en termes de visualisation? De notre côté, ils sont utilisés dans des tableaux avec plusieurs paramètres en même temps ce qui signifie qu'on visualise toujours le short_label de l'index principal puis de chaque sous index et chaque catégorie précise donc on a souvent bcp de redites avec les versions actuelles d'où certaines de mes suppressions qui peuvent sembler étrange. Je te met un exemple ici car ce n'est peut être pas très clair https://www.ipp.eu/baremes-ipp/impot-sur-le-revenu/impot_revenu.calcul_impot_revenu.pv/table/. Je suis d'accord sur la perte de précision et donc la possibilité de mauvaise compréhension du paramètre avec les raccourcis mais par exemple dans plus values (l'exemple au dessus), les short labels étaient tellement longs qu'on ne pouvait plus voir les paramètres et cela bloquait l'affichage des tableaux donc je n'ai pas eu le choix de les raccourcir et donc perdre l'information (qui est conservé dans description si nécessaire). Pour la case fiscale, c'était justement pour ma part un choix de ne pas l'afficher dans le short label pour éviter les confusions car la case peut changer d'une année à l'autre et cela m'étonnerait qu'on pense à les réactualiser mais je les avais laissé dans la description pour ne pas casser votre travail. |
@DorineLam de ce que je comprend la plupart des changements problématiques portent sur les plus values où j'ai bcp raccourci car cela rendait les tableaux illisibles donc je vais essayer de rajouter le maximum de détails tout en ayant un tableau lisible et je vous remet une "proposition" ici |
Bonjour @Lolajossipp et @benjello, merci pour vos retours. @Lolajossipp, je n'ai pas l'impression que tes changements soient déjà en ligne. On se propose avec @b-michaud de les attendre pour refaire une relecture. Et peut-être pourrons-nous organiser une visio pour échanger sur les points vraiment bloquants :) |
Hello @DorineLam! Pardon pour l'attente un peu longue mais je m'occupe de ça cette semaine et je reviens vers vous pour qu'on parle des points bloquants si il y en a! :) |
Bonjour Lola, aucun problème, n'hésite pas ! :) |
2a50ea1
to
046df62
Compare
Hello @DorineLam! J'ai fini pour ma proposition, j'ai fais pas mal de changements sur les plus-values en rajoutant ce que tu suggérais et en restructurant un peu actions_gratuites qui était à la fois peu compréhensible et trop long pour les tableaux, dites moi ce que vous en pensez. |
2ce4ff8
to
72c0685
Compare
Bonjour @Lolajossipp, merci je fais une relecture cette semaine :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Lolajossipp voici des premiers retours, je me suis arrêtée environ à la moitié des changements, car j'ai l'impression qu'on ferait mieux d'échanger, il y a des éléments récurrents sur lesquels nous semblons diverger et je suis sûre que ce sera plus simple de trouver une solution commune à l'oral.
Que penses-tu qu'on prenne un temps pour passer en revue mes retours et la suite de la PR demain après-midi (jeudi 29 janvier) ? avec pour objectif de merger cette PR ^^ ?
title: | ||
href: https://www.legifrance.gouv.fr/codes/article_lc/LEGIARTI000006302198 | ||
2011-01-01: | ||
title: Article 30, loi n° 2011-900 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
title: Article 30, loi n° 2011-900 | |
title: Article 30 de la Loi n° 2011-900 (LFR pour 2011) |
@@ -8,3 +8,10 @@ values: | |||
value: null | |||
metadata: | |||
unit: /1 | |||
reference: | |||
2007-01-01: | |||
title: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
title: | |
title: Article 1 du Code général des impôts |
@@ -0,0 +1,6 @@ | |||
description: Imposition des gains de levée d'options sur titre et gains d'acquisition d'actions gratuites attribuées avant le 28.09.2012 si cession avant 2 ans | |||
metadata: | |||
short_label: Taux si cession avant 2 ans |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
short_label: Taux si cession avant 2 ans | |
short_label: Taux si cession des titres avant 2 ans |
Spécifier qu'il s'agit de la cession de titres ?
@@ -0,0 +1,6 @@ | |||
description: Imposition des gains de levée d'options sur titre et gains d'acquisition d'actions gratuites attribuées avant le 28.09.2012 si cession après 2 ans | |||
metadata: | |||
short_label: Taux si cession après 2 ans |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
short_label: Taux si cession après 2 ans | |
short_label: Taux si cession des titres après 2 ans |
Idem
+ pv.pv_cession_valeurs_mobilieres_pv_professionnelles.taux * max_(0, f3vg - f3vh) | ||
+ pv.actions_gratuites.taux2 * glo_taxation_ir_forfaitaire_taux2 | ||
+ pv.actions_gratuites.cession_plus_2.taux_tranche_inferieure * glo_taxation_ir_forfaitaire_taux2 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Super idée ! ça permet en effet de rendre beaucoup plus clairs les noms des paramètres, merci :)
@@ -7,7 +7,7 @@ values: | |||
2017-01-01: | |||
value: null | |||
metadata: | |||
short_label: Plafond des dépenses, avant majorations, la première année | |||
short_label: Plafond 1ère année |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
short_label: Plafond 1ère année | |
short_label: Plafond des dépenses la 1ère année |
@@ -11,7 +11,7 @@ values: | |||
2017-01-01: | |||
value: null | |||
metadata: | |||
short_label: Plafond des dépenses, avant majorations, pour les contribuables invalides ou ayant à charge une personne invalide | |||
short_label: Plafond si invalidité |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
short_label: Plafond si invalidité | |
short_label: Plafond des dépenses si invalidité |
@@ -7,7 +7,7 @@ values: | |||
2017-01-01: | |||
value: null | |||
metadata: | |||
short_label: Plafond des dépenses, après majorations | |||
short_label: Plafond avec majoration |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
short_label: Plafond avec majoration | |
short_label: Plafond des dépenses, avec majorations |
@@ -7,7 +7,7 @@ values: | |||
2017-01-01: | |||
value: null | |||
metadata: | |||
short_label: Plafond des dépenses, après majorations, la première année | |||
short_label: Plafond 1ère année avec majoration |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
short_label: Plafond 1ère année avec majoration | |
short_label: Plafond des dépenses, avec majoration, la 1ère année |
@@ -7,7 +7,7 @@ values: | |||
2001-01-01: | |||
value: 61 | |||
metadata: | |||
short_label: Montant de la réduction par enfant au collège (case 7EA) | |||
short_label: Collège |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ici, et pour les paramètres suivants, il me semble qu'on ne peut pas savoir, à l'aide des parents du paramètre qu'il s'agit du montant de la réduction, et du montant par enfant.
Bonjour @DorineLam, Lola est en vacances pour un mois. Je ne sais pas si tu préfères essayer d'en finir assez vite et d'en parler avec moi ou attendre son retour ? |
Bonjour Sylvain, c'est comme vous préférez de votre côté. On peut attendre le retour de Lola :) |
parameters/impot_revenu
.Quelques conseils à prendre en compte :
pyproject.toml
.CHANGELOG.md
.Et surtout, n'hésitez pas à demander de l'aide ! :)