Skip to content

refactor(openfisca): migre la fiscalité dans notre code

BITARD Michaël requested to merge refactor-openfisca into master

Created by: vmaubert

Checklist

  • La fiscalité dans l'onglet Minéraux et métaux de l'hexagone ont les mêmes valeurs que la prod
  • La fiscalité pour les entreprises d'or est correct par rapport à la prod
  • Les matrices sont correctes par rapports aux matrices déjà envoyées

Les gains de cette PR :

  • Découverte de bug d'arrondi dans openFisca (la version de numpy utilisée est trop ancienne et ne respecte pas la notion d'arrondi (Round half even)
  • On peut faire des tests d'intégration correct (sans avoir à mocker un faux serveur openfisca)
  • On n'a plus de communication réseau (api <-> openfisca) à chaque fois qu'on fait des requêtes de fiscalité (gain de temps)
  • On n'a plus de serveur openfisca qui tourne sur les serveurs (1.1G d'image docker, ce qui est énorme, pour comparaison, toute notre API fait 0.449G)
  • On a du code testé, robuste et évolutif dans notre projet directement
  • On n'a plus a builder openfisca pour chacun de nos commits (~30 secondes de resources utilisées pour pas grand chose)

Merge request reports

Loading