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

Refactor la gestion de la condition taux_incapacite dans la réforme aides jeune #174

Merged
merged 5 commits into from
Jul 31, 2023

Conversation

Allan-CodeWorks
Copy link
Contributor

Description

La condition taux_incapacite peut être gérée de la même façon que la condition age.
Voici une proposition pour homogénéiser cette gestion.

@Allan-CodeWorks Allan-CodeWorks requested a review from a team July 26, 2023 14:25
@Allan-CodeWorks Allan-CodeWorks force-pushed the Reforme_aides-jeune_refacto_taux_incapacite branch from 5b5fe93 to 6745a58 Compare July 27, 2023 12:27
@Allan-CodeWorks Allan-CodeWorks self-assigned this Jul 27, 2023
@Allan-CodeWorks Allan-CodeWorks added the enhancement New feature or request label Jul 27, 2023
Copy link
Contributor

@Shamzic Shamzic left a comment

Choose a reason for hiding this comment

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

Ok pour moi, je n'ai pas de remarques sur le code, la logique similaire à l'âge me semble ok et les tests passent 👍

Je viens de tester le dernier test dans ce fichier en mettant un taux_d'incapacite à 2 avec un output à 1 pour test_condition_taux_incapacite et le test passe, il n' y a pas de vérification sur un taux d'incapacité erroné j'ai l'impression ?

@Allan-CodeWorks
Copy link
Contributor Author

Allan-CodeWorks commented Jul 31, 2023

Ok pour moi, je n'ai pas de remarques sur le code, la logique similaire à l'âge me semble ok et les tests passent 👍

Je viens de tester le dernier test dans ce fichier en mettant un taux_d'incapacite à 2 avec un output à 1 pour test_condition_taux_incapacite et le test passe, il n' y a pas de vérification sur un taux d'incapacité erroné j'ai l'impression ?

@Shamzic
Du point de vue d'OpenFisca, il semblerai que la variable taux_incapacite soit de type float. Techniquement, tout float est valide et le fait qu'on utilise des valeurs comprises entre 0 et 1 ne serait qu'une convention.
Toute valeur supérieur à 2 est donc considérée > à 100%
Le test en question utilise une aide accordée en cas de taux_incapacite > 0.5 donc 2 est supérieur, l'aide est accordée.

@Allan-CodeWorks Allan-CodeWorks merged commit 7a00fea into master Jul 31, 2023
9 checks passed
@Allan-CodeWorks Allan-CodeWorks deleted the Reforme_aides-jeune_refacto_taux_incapacite branch July 31, 2023 09:07
@guillett guillett added this to the BC 1406243900 milestone Oct 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants