Fix Landais-Piketty-Saez reform #927
Conversation
CHANGELOG.md
Outdated
| * Correction d'un crash. | ||
| * Zones impactées : openfisca_france/reforms/landais_piketty_saez.py. | ||
| * Détails : | ||
| - La réforme ne fonctionnait plus à la suite d'un changement de syntaxe intempestif. |
There was a problem hiding this comment.
Quelle était l'erreur obtenue ? Non reproduite avec tests/reforms/test_landais_piketty_saez.py.
Mettre à jour le test pour qu'il détecte l'erreur ?
There was a problem hiding this comment.
Changement de syntaxe ou renommage de variable qui était à propager ?
There was a problem hiding this comment.
Il y a eu un changement de syntaxe et la persone qui l'a fait a fait une erreur sur le nom d'une variable
| prestations_sociales_i = menage.members.famille('prestations_sociales', period) * individu.has_role(Famille.DEMANDEUR) | ||
| prestations_sociales = menage.sum(prestations_sociales) | ||
| prestations_sociales_i = menage.members.famille('prestations_sociales', period) * menage.members.has_role(Famille.DEMANDEUR) | ||
| prestations_sociales = menage.sum(prestations_sociales_i) |
|
@sandcha ; j'ai rajouté un test et j'ai rebasé sur master mais j'obtiens une PR bizarre ... |
benjello
force-pushed
the
fix-landais-piketty-saez-reform
branch
from
ad24742 to
30493b9
Compare
|
@sandcha: tout est rentré dans l'ordre grâce à @Anna-Livia ! |
|
Je retire @sandcha qui est en vacances pour demander ton approbation @Anna-Livia si tu n'y vois pas d'inconvénients |
|
Merci @sandcha . |
Contexte: faire fonctionner ce notebook
openfisca_france/reforms/landais_piketty_saez.py.Ces changements :