search

leximpact / leximpact-server

Simuler l'impact des réformes socio-fiscales en moins d'une minute
https://leximpact.an.fr
GNU Affero General Public License v3.0
4 stars 0 forks source link

Includes DSU params in dictionnary that translate request into OFDL p… #31

Closed magemax closed 4 years ago

magemax commented 4 years ago

…arams to take into account these when specified in the request.

Devrait résoudre le bug des params pas pris en compte pour la DSU

LoicPoullain commented 4 years ago

Sans modifications, il y a des différences entre les résultats du code existant et ceux de l'amendement Capture d’écran 2020-09-02 à 08 56 39 Capture d’écran 2020-09-02 à 08 57 40

LoicPoullain commented 4 years ago

C'est bon, j'ai trouvé ce qui clochait : c'est la valeur pourcentageRangSeuilHaut.

A priori, tu utilises 66,6 comme valeur et j'avais 67 de mon côté.

Dans le cas où la valeur est 66, on a 6 communes qui ne sont plus éligibles. Dans le cas où la valeur est 67, on a 4 communes qui sont éligibles en plus.

@magemax est-ce que les différences avec la DGCL viendraient de cette interprétation des "deux premiers tiers" ? Laquelle de ces trois valeurs réduit la marge d'erreur ?

LoicPoullain commented 4 years ago

Pour une valeur popMinSeuilHaut égale à 8000, j'obtiens des dotationMoyenneParHab très proches de zéro mais non nulles. Je trouve cela curieux pour des moyennes (0+0=tête à toto) d'autant plus que 10000 renvoie bien des valeurs strictement nulles. Y a-t-il une erreur de ce côté ?

Capture d’écran 2020-09-02 à 09 38 07