-
Notifications
You must be signed in to change notification settings - Fork 39
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
Ajoute le Revenu Solidarité Jeune de la Métropole de Lyon #4090
Conversation
89b67a9
to
60857fc
Compare
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.
Ne faudrait-il pas ajouter dans le simulateur la mise en place de la variable sous_contrat_engagement_jeune pour que cette aide s'affiche ?
@@ -0,0 +1,12 @@ | |||
label: Revenu Solidarité Jeunes |
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.
@Allan-CodeWorks du coup comment on utilise la variable sous_contrat_engagement_jeune
? on ne l'utilise pas pour le moment ?
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'imagine qu'il faut envoyer une nouvelle variable sous_contrat_engagement_jeune
à Openfisca qui sera true
si le revenu contrat_engagement_jeune
est > 0 ?
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.
Oui merci!
J'ai fait quelque chose dans ce genre là dans mon dernier commit mais je n'ai aucune idée si c'est bien de cette manière qu'on veut faire ça ?
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.
Oui c'est bien ça! (:
Hello ! Voici mes propositions concernant la description et les conditions textuelles : Description : conditions:
|
Merci, c'est intégré! |
c2929ba
to
9424506
Compare
Il y a probablement un soucis sur la période dans la dernière version de Openfisca-france-local car avec cette PR quand je lance une simulation respectant les critères du RSJ de Lyon, j'ai une erreur sur |
9424506
to
9eb3565
Compare
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.
Ok pour moi, testé et fonctionnel en local (:
Oui mais ça passe pas la CI 😭 |
C'est comme si il arrivait pas à trouver la définition de la variable dans openfisca-france-local.. Du coup il la cherche dans openfisca-france ? |
C'est quelque chose dans ce genre là, j'ai déjà mis les mains dans ce test à Willy il y a quelques temps, ça nous a pris 2 jours pour intégrer la réforme, je reprends la dessus demain avec un cerveau frais ;) |
@Allan-CodeWorks j'ai l'impression que ce test test que le simulateur marche bien sans extension, avec paris et avec france-local. Du coup comme la variable tu l'as rajouté dans france-local les deux premiers tests ne fonctionnent pas :
Ce qui explique pourquoi ça fonctionne en test end-to-end. Car vu du simulateur on utilise que france+paris+france-local. Ça repose la question qu'on s'était posé pour la reforme dynamique : Est-ce que c'est vraiment utile de tester des cas qui ne sont pas en production (ou en reformulant : est-ce qu'on veut que la code base soit toujours viable le jours où on retire une extension). Si tel est le cas, il va falloir rajouter cette variable dans france à la place. cc @guillett |
9eb3565
to
53f3453
Compare
C'est exactement ça, merci @baptou12! La question que tu poses reste intéressante mais grâce à ta remarque, mon cerveau + frais qu'hier semble avoir trouvé une solution très simple pour corriger le test ;) |
53f3453
to
033045a
Compare
Description
La PR oppenfisca 189 ajoute le dispositif du RSJ dans openfisca.
Cependant la modélisation est ne fonctionne pas avec notre besoin, une PR qui corrige ce problème est actuellement en review est bloque le merge de la présente PR