-
Notifications
You must be signed in to change notification settings - Fork 101
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 double comparaison incompatible avec Numpy #1365
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Merci @nico5655.
Quelques suggestions de correction de typo et c'est bon pour moi
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Merci beaucoup @nico5655 !
Pour éviter ces type de bugs, c'est toujours une bonne idée d'ajouter un test qui reproduit l'erreur. Dans ce cas précis, il n'y a probablement pas de tests qui s'assurent du bon fonctionnement des formules avec de typos.
Pourrais-tu en ajouter ?
On regardant de plus près, je vois qu'il y a déjà de tests ici, mais qu'on n'a pas intercepté l'erreur... |
C'est donc le problème des tests avec un seul scénario qu'on a déjà vu avant (PAJE...). Je rebase et j'accepte, on devra trouver une solution pérenne à ce problème probablement sur core. |
@benjello C'est bon pour moi. |
@maukoquiroga je te laisse merger |
Co-Authored-By: Mahdi Ben Jelloul <mahdi.benjelloul@gmail.com>
Fix #1363
Fix #1359
openfisca_france/model/prestations/autonomie.py
numpy
Ces changements :
Quelques conseils à prendre en compte :
setup.py
.CHANGELOG.md
.