Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Corrige le calcul de l'ASF #1043

Merged
merged 2 commits into from
Oct 1, 2018
Merged

Corrige le calcul de l'ASF #1043

merged 2 commits into from
Oct 1, 2018

Conversation

alexsegura
Copy link
Contributor

@alexsegura alexsegura commented Jul 23, 2018


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

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


return asf_elig * montant
return asf_elig * (montant - pensions_alimentaires_percues)
Copy link
Contributor

@Morendil Morendil Sep 22, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Et si la pension alimentaire est supérieure au montant de l'ASF ? Plusieurs des KO remontés par l'intégration continue semblent être liés à la non prise en compte de ce cas, y compris un effet de bord sur le RSA.

J'utiliserais numpy.maximum pour bloquer à 0 le montant différentiel, ou bien on peut comparer ASF et pensions dans la variable d'éligibilité (moins efficace en termes de performances mais peut-être plus intéressant en termes d'explication de la non-éligibilité).

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Un max ira très bien. C'est généralement ce que l'on utilise pour les prestations différentielles.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Il faut utiliser la fonction max_ (qui pointe vers numpy.maximum) de la façon suivante

return asf_elig * max_(montant - pensions_alimentaires_percues, 0)

@alexsegura alexsegura self-assigned this Sep 24, 2018

return asf_elig * montant
return asf_elig * (montant - pensions_alimentaires_percues)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Il faut utiliser la fonction max_ (qui pointe vers numpy.maximum) de la façon suivante

return asf_elig * max_(montant - pensions_alimentaires_percues, 0)

@alexsegura alexsegura force-pushed the fix-asf branch 3 times, most recently from bfd7bea to 615e075 Compare September 26, 2018 12:40
@benjello
Copy link
Member

Le message dans le YAML du test qui plante est confusant. On dirait qu'il a été forcé à zéro.
@guillett saurais-tu nous en dire plus ?

@alexsegura
Copy link
Contributor Author

Je suis complètement paumé, je ne comprends pas comment est calculé le chiffre de 98.15519

@alexsegura
Copy link
Contributor Author

C'est la formule qui ne marche pas, non ?

Dans ce cas de figure les pensions alimentaires (103) sont supérieures au montant de l'ASF (fixé à 100.58 en juin 2015)

@alexsegura
Copy link
Contributor Author

Ha, non. Il y a deux enfants, donc ça fait 201.15

@Morendil
Copy link
Contributor

Peut-être que le test est faux!

@alexsegura
Copy link
Contributor Author

Je ne suis pas du tout sûr, mais ces tests test_mes_aides_* ne sont pas des cas réels d'utilisation constatés ?

@Morendil
Copy link
Contributor

Oui c'est ce que je comprends aussi, et dans ce cas présent le descriptif du test ne me semble pas tout à fait confiant, ni complètement clair (même si c'est mieux que beaucoup d'autre tests avec zéro commentaire). Ton raisonnement me paraît correct alors que je ne lis pas dans le test une bonne raison pour laquelle l'ASF devrait être à 0.

(Et je ne sais pas si "L'ASF n'est pas perçue" décrit le comportement attendu, ou le comportement observé incorrect; d'autres tests qui datent de la même PR #358 déclarent par exemple "L'ASPA n'est pas perçue" en attendant une valeur non nulle pour l'ASPA.)

@guillett est en congé mais @monbocal a peut-être des billes.

@Morendil Morendil dismissed benjello’s stale review October 1, 2018 08:57

Corrections effectuées

@Morendil
Copy link
Contributor

Morendil commented Oct 1, 2018

Vu avec @monbocal ce matin

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants