-
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
Fix chèque energie implementation #960
Conversation
@benjello je viens de t'ajouter en reviewer. Si tu peux jeter un coup d'oeil c'est super et ça permet à @Anna-Livia de ce focaliser sur les PRs de l'IPP. Les changements de cette PR sont mineurs. Merci ! |
uc = parameters(period).cheque_energie.unites_consommation | ||
nb_personnes = menage.nb_persons() | ||
gardes_alternees = menage.sum(menage.members('garde_alternee', period.first_month)) | ||
|
||
nb_personnes_ajuste = nb_personnes - 0.5 * gardes_alternees | ||
return uc.premiere_personne + uc.deuxieme_personne * (nb_personnes_ajuste > 1) * (nb_personnes_ajuste - 1) + uc.autres_personnes * (nb_personnes_ajuste > 2) * (nb_personnes_ajuste - 2) | ||
return ( | ||
+ uc.premiere_personne |
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.
Bizarre cette ligne vide et ce + en début de ligne pour le premier terme de l'addition
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.
La première ligne (ie. (
) ça me semble assez classique. Pour le +
en début de ligne c'est pour harmoniser la première ligne avec les lignes suivantes.
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.
J'ai crû voir une ligne complètement vide entre la parenthèse et le premier p+ mais peut être était-ce dû à mon browser ou simplement à ma vue déficiente.
0k pour mettre un + en début de ligne si cela passe.
prestations/cheque_energie
Supersedes #960.