Skip to content
This repository has been archived by the owner on May 17, 2021. It is now read-only.

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

Merged
merged 7 commits into from
Sep 2, 2020

Conversation

magemax
Copy link

@magemax magemax commented Sep 1, 2020

…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

…arams to take into account these when specified in the request.
…d on the data (i.e. changing them greatly makes the number of eligible communes change in a coherent way)
@LoicPoullain
Copy link

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
Copy link

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
Copy link

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

…e precision (float32 is precise within 10**-6 which is not very classy when amounts are greater that 1 billion)
…recision brought by float64 that changed some results on the potentiel financier
@sandcha sandcha merged commit f9333c3 into master Sep 2, 2020
@sandcha sandcha deleted the params_dsu branch September 2, 2020 16:19
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants