search

openfisca / openfisca-tunisia

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

Notebooks #63

Closed benjello closed 6 years ago

benjello commented 6 years ago

@sandcha : I think this one is ready for merge too; Thanks !

sandcha commented 6 years ago

@benjello J'ai pushé ce que j'avais en réserve pour test_notebooks.py et la config autour. Peux-tu me dire si c'est bon pour toi ? 🙂

benjello commented 6 years ago

@sandcha. je ne comprends pas le dernier commentaire sur openfisca-pension. Mais tout cela m'a l'air très bien. Let's merge !

sandcha commented 6 years ago

Pour le dernier commentaire : lorsque l'on teste demo_survey_scenario.ipynb, on obtient une CellExecutionError en raison de pkg_resources.get_distribution('tunisia-pension').location,.

Il me semble que ce devrait plutôt être openfisca-tunisia-pension mais malgré tout, il faudrait ajouter une dépendance OpenFisca-Tunisia-Pension dans le setup alors que ce module n'a pas été publié sur Pypi. Travis ne serait pas heureux.

Ceci dit, il y a peut-être une solution de contournement à laquelle je n'aurais pas pensé. En attendant, j'ai sorti les notebooks du make test.

sandcha commented 6 years ago

@benjello Si tu pouvais faire passer les notebooks avant de merger, ce serait top !

benjello commented 6 years ago

@sandcha : j'ai virer le notebook problématique qui n'a d'intérêt qu'avec des données. Et j'ai réintégrer les notebooks dans les tests. Si la façon de faire te convient, je te laisse merger.