-
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
Fusion de la CMU-C et de l'ACS #1369
Conversation
@MehdiBenHnin Pour info. |
@frtomas Merci Frédéric. |
openfisca_france/parameters/cmu/complementaire_sante_solidaire/cmu_30_49_ans.yaml
Outdated
Show resolved
Hide resolved
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 fait des commentaires pour les tests générés.
Il y a beaucoup de tests qui ne sont pas unitaires et qui mélange la complexité de plusieurs formules ce qui les rend peu exploitables.
Un indicateur de cette complexité embarqué c'est le niveau de descente entre le calcul des valeurs attendues dans output
et les valeurs mises en input
. En lançant les tests tu peux ajouter -v
pour avoir les détails des calculs et voir comment limiter cette complexité si c'est pertinent.
@guillett en regardant, la plus grande part de complexité vient sans surprise du calcul des ressources. |
@guillet est-ce qu'il reste encore des points à revoir sur cette PR ? |
@maukoquiroga très bien, merci beaucoup, j'attends tes retours. |
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.
Pas d'objection.
Néanmoins je vas me permettre une remarque d'ordre général.
Il est assez bizarre de devoir déclarer les Enum deux fois.
Mais je n'arrive pas à trouver de formulation alternative simple.
@benjello Je me demande s'ils sont vraiment nécessaires ces enums. |
…nte_solidaire_montant_i, complementaire_sante_solidaire_montant et complementaire_sante_solidaire_plafond
…nte_solidaire_montant_i, complementaire_sante_solidaire_montant et complementaire_sante_solidaire_plafond
49aa516
to
91e4d7b
Compare
Les changements demandés on été faits.
@maukoquiroga @guillett Voici les références législatives pour la réforme : https://www.legifrance.gouv.fr/affichTexte.do?cidTexte=JORFTEXT000038670887&categorieLien=id https://www.legifrance.gouv.fr/affichTexte.do?cidTexte=JORFTEXT000038670723&categorieLien=id Source : Légifrance |
definition_period = MONTH | ||
|
||
def formula(famille, period, parameters): | ||
css_participation_forfaitaire_i = famille.members('css_participation_forfaitaire_i', period) |
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.
@frtomas la variablecss_participation_forfaitaire_i
ne semble pas exister. Son bon nom serait css_participation_forfaitaire_montant_i
.
Sur quelle branche fonctionne le PNDS à l'heure actuelle ?
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.
@guillett Nous sommes sur une branche local ne prenant pas en compte les renommage qui ont été fait par @maukoquiroga (ils présentaient un impact sur PNDS que nous n'avions pas le temps de gérer).
De notre coté, ce problème n’apparaît donc pas et ne nous impacte pas encore.
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.
@MehdiBenHnin tu as peut-être des éléments. Désolé, nos messages se sont croisés.
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 @frtomas !
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.
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.
@maukoquiroga merci, le retour au master ne devrait pas être problématique.
L'impact concernait effectivement le nommage des variables dans PNDS qui, bien que léger, représentait un risque supplémentaire étant donné le planning serré.
@maukoquiroga merci pour ta proposition d'aide. Je vois que les déploiements automatiques sur master échouent, penses-tu pouvoir regarder ça ? |
@guillett Je suis dessus, semble une anomalie introduite sur https://github.com/openfisca/openfisca-core |
C'est une incompatibilité entre Prochaines étapes :
|
parameters/cmu/complementaire_sante_solidaire
parameters/cmu/complementaire_sante_solidaire_regime_alsace_moselle
model/prestations/minima_sociaux/cmu
Ces changements :