Amélioration des calculs de prime d'activité #1232
Conversation
| definition_period = ETERNITY | ||
|
|
||
|
|
||
| class ppa_versee_offset_total(Variable): |
There was a problem hiding this comment.
Le nom "offset" me paraît peu clair, on pourrait peut-être éviter l'anglicisme? A vue de nez c'est le nombre de mois écoulés depuis la demande, est-ce que ça pourrait s'appeler ppa_mois_depuis_demande ?
guillett
changed the title
| return (datetime64(period.start).astype('datetime64[M]') - ppa_mois_demande.astype('datetime64[M]')).astype('int') | ||
|
|
||
|
|
||
| class ppa_versee_offset(Variable): |
There was a problem hiding this comment.
Contrairement au reste de la PR (par exemple sur la décomposition de ppa_fictive) ça ne saute pas aux yeux que ça a de la valeur d'introduire une deuxième variable plutôt que faire tout le calcul ici, qu'est-ce qui t'a orienté vers ce choix ?
Je ne sais pas encore trop comment elle pourrait s'appeler, il y a une notion de passage d'une base trimestrielle à mensuelle ?
|
Merci beaucoup Laurent pour tes retours effectivement j'avais coupé en deux la logique du calcul de ces nombres de mois car les deux parties n'ont pas été évidente à coder (qui pourrait le croire qu'en on voit que tout cela tient en deux petites lignes). Tu as raison, ça n'a plus de sens à séparer la logique. |
|
@guillett Je prends le rebase, setup/changelog et merge. |
|
Argh. Comme il n'y a pas eu besoin de rebase, les commits avec "fixup!" etc. sont restés tels quels. |
prestations/minima_sociaux/ppa.