Skip to content

Montants de la DSR

Benoît Courty requested to merge montant-dsr-bc into master

Created by: magemax

La nouvelle PR à la mode !!!

Elle inclut trois parties :

  • Une qui consiste à redesigner un peu les tests dotations, pour qu'ils soient plus fonctionnels, mais aussi pour pouvoir supprimer le check d'égalité parfaite de 2 dictionnaires complexes qui rendrait relou tout changement dans le futur : j'ai tenté de traduire les vraies choses qu'on essaye de checker, puis je retire le test d'égalité des dictionnaires. On peut en parler si tu veux qu'on aille plus avant sur les vrais tests mais je pense qu'on couvre déjà bien.

  • A partir du commit "Bumps required OFDL version", on plonge dans le monde onirique du calcul des montants hors garanties de la DSR. ses trois fractions n'auront plus de secret pour vous (enfin si, vu que c'est OFDL qui fait tout le boulot sous le capot). Attention, utilise une version d'OFDL pour les connaisseurs, qui n'est pas encore sortie (mais dont on subodore qu'elle sera la version 0.3.0)

  • A partir du commit "Adds a function that extracts precomputed results from DGCL", refactore la fonction qui compare avec la DGCL et compare désormais des colonnes numériques

Elle est prête à être reviewed, mais elle vient bien sûr après celle d'OFDL.

En supposant qu'on merge la PR22 en l'état, je reporte ici les éléments de la todolitre restant :

  • implémenter les montants
  • optimisation du temps de calcul (par exemple par du preload)
  • fonction de recherche / des villes
  • lots of documentation
  • lots of additional tests
  • lots of typing

Merge request reports