search

openfisca / openfisca-tunisia

Tunisian tax and benefit system for OpenFisca
http://www.openfisca.tn
15 stars 7 forks source link

Enum #58

Closed benjello closed 6 years ago

benjello commented 6 years ago

@sandcha : j'ai un test qui ne passe pas que je ne comprends pas. Et je ne suis pas sûr d'avoir tout bien fait (je ne maîtrise pas les enums)

sandcha commented 6 years ago

@benjello L'erreur de formulas/revenu_assimile_salaire_apres_abattements.yaml semble venir du set_input_divide_by_period de salaire_en_nature (revenus/activite/salarie.py). Du moins, le test passe si on transforme la période de cette variable en annuelle. 😶

benjello commented 6 years ago

@sandcha : je crois qu'on tient un bug. Le set_input_divide_by_period arrondi au dinar près. C'est pas top. Saurais-tu trouver dans openfisca-core si on peut contrôler le nombre de décimales ? Merci !

benjello commented 6 years ago

@fpagnoux : t'as peut-être une idée aussi sur les set_input_divide ?

benjello commented 6 years ago

@sandcha @fpagnoux : le problème vient du fait que la variable est déclarée comme int ....

benjello commented 6 years ago

C'est corrigé: j'ai passé en float les variables qui sont susceptibles d'être divisées !

benjello commented 6 years ago

@sandcha : review et je merge !

benjello commented 6 years ago

@sandcha : j'ai fait un peu de nettoyage mais j n'ai pas tout traiter. Si tu as le temps n'hésite pas à pusher sur cette PR. Sinon je m'y remettrai bientôt ie quand je pourrai ;-)

benjello commented 6 years ago

@sandcha : modifications done !

sandcha commented 6 years ago

@benjello Quid du notebook demo-Copy1.ipynb ? Futur demo_survey_scenario.ipynb ?

benjello commented 6 years ago

Je vais merger et nnettoyer le notebook a posteriori