Unifie les formules datées des réductions d'impôt #1195

Morendil · 2018-10-26T17:05:41Z

Amélioration technique.
Périodes concernées : toutes.
Zones impactées : prelevements_obligatoires/impot_revenu/reductions_impot.py.
Détails :
- Unifie en une seule les formules datées de la variable reductions.

Ces changements (effacez les lignes ne correspondant pas à votre cas) :

Corrigent ou améliorent un calcul déjà existant.

Quelques conseils à prendre en compte :

Jetez un coup d'œil au guide de contribution.
Regardez s'il n'y a pas une proposition introduisant ces mêmes changements.
Documentez votre contribution avec des références législatives.
Mettez à jour ou ajoutez des tests correspondant à votre contribution.
Augmentez le numéro de version dans setup.py.
Mettez à jour le CHANGELOG.md.
Assurez-vous de bien décrire votre contribution, comme indiqué ci-dessus

Et surtout, n'hésitez pas à demander de l'aide ! :)

Morendil · 2018-10-26T17:18:37Z

On a vu (#1024, #1193) que les cas de divergences entre OpenFisca et les calculateurs de la DGFIP se concentraient notamment dans les réductions d'impôt.

J'explique cela notamment par le fait que ce seul fichier reductions_impots.py, à lui tout seul plus de 6000 lignes, concentre une partie substantielle de la dette technique présente dans OpenFisca. Le code y est extrêmement répétitif, ce qui le rend vulnérable à certains bugs difficiles à repérer, par exemple:

openfisca-france/openfisca_france/model/prelevements_obligatoires/impot_revenu/reductions_impot.py

Lines 1638 to 1639 in 38c6433

    
           fhal = foyer_fiscal('fhal', period) 
        
           fham = foyer_fiscal('fhal', period)

Certaines de ces formules sont de simples sommes (par exemple doment), une technique similaire à celle appliquée ici permettra de les unifier, on peut attendre un gain très rapide de plusieurs centaines de lignes.

Pour d'autres cela semble un peu plus compliqué et il serait nécessaire de faire émerger des structures latentes (pour reprendre les termes d'une discussion récente). Ce serait aussi la bonne occasion de traiter la problématique des nomenclatures des CERFA de l'impôt sur le revenu comme le souhaitait @claireleroy.

Les solutions actuelles recèlent des bizarreries, par exemple:

openfisca-france/openfisca_france/model/prelevements_obligatoires/impot_revenu/reductions_impot.py

Lines 1329 to 1336 in 38c6433

    
               def formula_2005_01_01(foyer_fiscal, period, parameters): 
        
                   ''' 
        
                   Investissements dans les DOM-TOM dans le cadre d'une entrepise. 
        
                   ''' 
        
                   f7ur = foyer_fiscal('f7ur', period) 
        
                   f7oz = foyer_fiscal('f7oz', period) 
        
                   f7pz = foyer_fiscal('f7pz', period) 
        
                   f7qz = foyer_fiscal('f7qz_2012', period)

…on a du mal à comprendre l'intrusion d'une nomenclature "2012" dans une formule qui couvre en principe l'année 2005.

J'ai l'impression que ces sujets mériteraient du temps présentiel avec au minimum l'IPP et une partie de l'équipe DINSIC, pour mettre en commun nos forces respectives: je peux apporter de l'expertise sur les techniques de refactoring (et pourquoi il est important de les renforcer au sein d'OpenFisca), mais pas sur la fiscalité par exemple.

claireleroy · 2018-10-29T10:28:07Z

@Morendil Juste une petite précision pour que j'y vois plus clair : comment sont gérés les changements de périmètre de la variable reductions dans le temps (ajout d'une nouvelle réduction, suppression d'une réduction qui n'existe plus) suite à cette simplification du code ?

Si je comprends bien :

Côté "start" : les formules de chaque réduction sont des dated_formulas donc elles ont une date de début implicite ?
Ex : la réduction duflot commence en 2013 car avant cela elle n'a pas de formules. Si on essaye de la calculer pour une période avant 2013 elle renvoie 0 ?
Côté "end" : les formules ont un attribut end_date qui empêche de les calculer au-delà de cette date ?
Ex : la réduction sofipe s'arrête au 1er janvier 2011. Si on essaye de la calculer après cette date, elle renvoie 0 ?

Est-ce que c'est bien ça ?

benjello · 2018-10-29T10:41:56Z

Je laisse @Morendil confirmer mais je crois que tu vois juste @claireleroy.

Morendil · 2018-10-29T11:10:06Z

@claireleroy Oui c'est bien ça.

On prend un léger risque par rapport à la situation antérieure, qui était "ceinture et bretelles" par rapport à la fin d'une formule - comme on avait de facto une formule de reductions par année fiscale, si on oubliait de noter le end_date d'une réduction individuelle mais qu'on ne l'appelait plus dans reductions ça donnait le bon résultat. Avec cette nouvelle version tout repose sur les dates de démarrage des formules datées et les end_date des variables concernées. J'ai corrigé un oubli de ce type. Je trouve que le gain de 500 lignes vaut bien ce petit risque. :)

fpagnoux

Pfiew, merci pour ce travail sans doute fastidieux... Le nombre de points qui semblaient incohérent dans l'implem précédente démontre assez clairement l'intérêt de la factorisation 🙂.

fpagnoux · 2018-11-02T14:25:47Z