Improve type_menage #695
Improve type_menage #695
Conversation
|
@fpagnoux : this PR should be GTM too |
CHANGELOG.md
Outdated
| ## 13.2.1 - [#695](https://github.com/openfisca/openfisca-france/pull/695) | ||
|
|
||
| * Évolution du système socio-fiscal | ||
| * Périodes concernées : toutes |
There was a problem hiding this comment.
Attention: l'indentation n'est pas cohérente avec les autres blocs du changelog.
CHANGELOG.md
Outdated
| * Évolution du système socio-fiscal | ||
| * Périodes concernées : toutes | ||
| * Zones impactées : `mesures` | ||
| * Amélioration insuffisante mais substantielle de `type_menage` |
There was a problem hiding this comment.
🤔
De quel type d'amélioration s'agit-il ? Une correction du résultat qui était faux dans certains cas ? Seulement une simplification de la syntaxe ?
En tant qu'utilisateur, le fait que ce soit insuffisant mais substantiel m'intéresse moins que de savoir les impacts du changement :)
There was a problem hiding this comment.
C'était complètement inutilisable. C'est mieux maintenant mais pas encore parfait.
There was a problem hiding this comment.
Dois rajouter en bout de ligne "qui était inutilisable" ?
There was a problem hiding this comment.
Inutilisable, ça veut dire que ça produisait une erreur quand tu l'appelais, c'est ça ?
There was a problem hiding this comment.
Que c'était très faux car mal traduit quand le code a évolué et pas utilisé depuis longtemps.
There was a problem hiding this comment.
Dans ce cas je remplacerais
- Amélioration insuffisante mais substantielle de
type_menage
par
- Correction du calcul de
type_menage
- Des erreurs peuvent subsister dans certains cas (lesquels ?)
|
@fpagnoux : changement appliqués ;-) |
|
Merci, le changelog est beaucoup plus clair ! |
|
OK je merge dès que les tests passent ! |
|
Merci de ne pas faire d'upstream merge par contre ;) |
|
Je m'occupe de faire le rebase |
fpagnoux
force-pushed
the
improve-type-menage
branch
from
0439553 to
0641a88
Compare
|
Prêt à merger dès que les tests passent |
|
J'ai pas vraiment compris ce qui a merdé ... |
|
Tu as juste du appuyer sur "Update branch", non ? Je vais voir si on peut pas configurer un peu mieux l'interface graphique. |
| _2_kid = af_nbenf == 2 | ||
| _3_kid = af_nbenf >= 3 | ||
|
|
||
| af_nbenf = menage.personne_de_reference.famille('af_nbenf') |
There was a problem hiding this comment.
Arg, il manquait la période et j'ai rien vu 😭
Il faut vraiment que cet argument devienne obligatoire!
No description provided.