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

Update bareme irpp 2017 2018 #50

Merged
merged 21 commits into from
Jan 7, 2018
Merged

Conversation

benjello
Copy link
Member

@benjello benjello commented Jan 1, 2018

Et plein de mises à jour de paramètres

@benjello benjello requested a review from sandcha January 1, 2018 22:14
Copy link
Contributor

@sandcha sandcha left a comment

Choose a reason for hiding this comment

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

Un doublon dans un yaml de deductions (et typos).

end = '2016-12-31'

def formula_2014(foyer_fiscal, period, parameters):
# Les éligibles ne doivent percevori que des salaires et des pensions
Copy link
Contributor

Choose a reason for hiding this comment

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

Typo "percevoir".

@@ -641,6 +656,12 @@ def calcule_impot_revenu_brut(salaire_mensuel, deduction_famille_annuelle, perio
revenu_assimile_salaire_apres_abattement = max_(revenu_assimile_salaire * (1 - tspr.abat_sal) - smig * tspr.smig, 0)
bareme = parameters(period.start).impot_revenu.bareme

return - bareme.calc(
non_exonere = revenu_assimile_salaire_apres_abattement >= 0
Copy link
Contributor

Choose a reason for hiding this comment

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

N'est-il pas prématuré d'avoir une méthode auxiliaire pour un seul usage sachant qu'en plus, elle n'apparaîtra pas dans legislation explorer ?

Copy link
Member Author

Choose a reason for hiding this comment

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

J'ai pas trouvé mieux pour l'instant.

@@ -10,53 +10,63 @@ brackets:
value: 0
2017-01-01:
value: 0
2018-01-01:
value: 0.01
Copy link
Contributor

Choose a reason for hiding this comment

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

Avons-nous des références pour les valeurs mises à jour ?

Copy link
Member Author

Choose a reason for hiding this comment

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

http://www.paie-tunisie.com/423/fr/173/publications/la-loi-de-finances-tunisie-2018-principales-dispositions.aspx

Loi de Finances 2018 art. 27 mais je ne sais pas si c'est l'article voté ou le projet.
@trojette saurais-tu ?

Copy link
Member Author

Choose a reason for hiding this comment

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

Cela n'a pas l'air d'être cela ...

@@ -0,0 +1,44 @@
description: Contributions au budget de l'Etat (montant de la contribution en jour de salaire ou pension brut par tranche de revenu annuel net)
brackets:
Copy link
Contributor

Choose a reason for hiding this comment

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

De même, références pour ces valeurs ?

Copy link
Member Author

Choose a reason for hiding this comment

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

Loi de finances sûrement

@@ -0,0 +1 @@
description: Exonration d'impôt sur le revenu pour les revenus net inférieur à un certain seuil
Copy link
Contributor

Choose a reason for hiding this comment

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

Typos : Exonération d'impôt sur le revenu pour les revenus nets inférieurs à un certain seuil

Copy link
Contributor

Choose a reason for hiding this comment

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

@benjello Premier mot bien corrigé mais ne voudrait-on pas du pluriel sur la suite ? 🤔

@@ -3,3 +3,7 @@ unit: currency
values:
1990-01-01:
value: 300
2011-01-01: # TODO à corriger si besoin
value: 600
1990-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.

300 ou 1000 ? Date en doublon.

@@ -1,25 +0,0 @@
# -*- coding: utf-8 -*-
Copy link
Contributor

Choose a reason for hiding this comment

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

👍




- name: "Salarié 5000 TND chef de famille"
Copy link
Contributor

Choose a reason for hiding this comment

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

Si chef de famille est un élément de distinction entre les deux tests, le faire apparaître directement ?

Copy link
Member Author

Choose a reason for hiding this comment

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

Le salaire non netté de la déduction lié au statut de chef de famille est supérieur au seuil de 5000

Copy link
Contributor

@sandcha sandcha Jan 7, 2018

Choose a reason for hiding this comment

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

Ok pour cette PR mais ça me semble à clarifier pour le futur parce que le name décrit la situation en entrée mais ne dit rien de ce qu'on vérifie.

Ce que je comprends :
Chef de famille est un élément qui distingue ce test.
5000 TND est un seuil et c'est pour ça que la valeur testée est > 5000 malgré le nom du test.
On vient tester une exonération mais ce terme n'apparaît pas dans le nom du test ; c'est du corps et par comparaison avec l'autre test de ce fichier qu'on voit que, par exemple, deduction_famille nous intéresse.

@trojette
Copy link
Contributor

trojette commented Jan 5, 2018 via email

@trojette
Copy link
Contributor

trojette commented Jan 5, 2018 via email

@benjello
Copy link
Member Author

benjello commented Jan 5, 2018

@trojette: Je cherche la modification du barème lié à l'introduction de la "CSG" tunisienne de 1%

@benjello
Copy link
Member Author

benjello commented Jan 6, 2018

@sandcha : pourrait-on merger rapidement quitte à ouvrir des issues sur les points soulever.
Je veux faire un notebook qui s'appuie sur les récentes modifications ?

@sandcha
Copy link
Contributor

sandcha commented Jan 7, 2018

Référence mise à jour pour impot_revenu/deductions/fam/infirme.yaml d'après :

الترفيع في الطرح بعنوان الأبناء المعاقين
من الفصل 40 من مجلة الضريبة على دخل الأشخاص الطبيعيين والضريبة III الفصل 55  يرفع المبلغ الوارد بالمطة الثانية من الفقرة
على الشركات من 1200 دينار إلى 2000 دينار
- و1200 دينار لكل طفل معاق مهما كان سنه أو رتبته (نقّحت بالفصل 50 ق.م عدد 90 لسنة 2004 المؤرخ في 31/12/2004 وبالفصل  40-2 ق.م عدد 71 لسنة 2009 المؤرخ في 21/12/2009 و بالفصل 94 ق.م عدد54 لسنة 2013 المؤرخ في 2013/12/30).

Ceci indiquerait-il que la dernière référence serait en 2013 et non en 2017 ? Pas de certitude donc non modifié dans le yaml. A moins qu'on ne référence l'en-tête de la loi de finance 2018 :
.قانون عدد 66 لسنة 2017 مؤرخ في 18 ديسمبر 2017 يتعلق بقانون المالية لسنة 2018

@sandcha
Copy link
Contributor

sandcha commented Jan 7, 2018

@benjello @trojette Si on traduit bien "CSG" par
مساهمة اجتماعية تضامنية لفائدة الصناديق الاجتماعية,
ceci est évoqué par الفصل 53 de la Loi de Finance 2018 (page 4287).

@sandcha
Copy link
Contributor

sandcha commented Jan 7, 2018

@benjello Si ma modification sur impot_revenu/deductions/fam/infirme.yaml te convient, il reste la version est à mettre à jour pour le module et c'est ok pour moi.
Il y a bien des améliorations que j'aurais aimé voir (cf. commentaires) mais rien de critique.

@benjello benjello merged commit 208bdff into master Jan 7, 2018
@benjello benjello deleted the update-bareme-irpp-2017-2018 branch January 10, 2018 21:29
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