Skip to content
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

Merged
merged 3 commits into from
Dec 6, 2023

Conversation

Allan-CodeWorks
Copy link
Contributor

@Allan-CodeWorks Allan-CodeWorks commented Nov 29, 2023

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

@Allan-CodeWorks Allan-CodeWorks force-pushed the ajoute_revenu_solidarite_jeune_lyon branch 2 times, most recently from 89b67a9 to 60857fc Compare December 4, 2023 17:09
@Allan-CodeWorks Allan-CodeWorks marked this pull request as ready for review December 5, 2023 08:17
@Allan-CodeWorks Allan-CodeWorks requested a review from a team December 5, 2023 08:18
Copy link
Contributor

@Shamzic Shamzic left a 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
Copy link
Contributor

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 ?

Copy link
Contributor

@Shamzic Shamzic Dec 5, 2023

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 ?

Copy link
Contributor Author

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 ?

Copy link
Contributor

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! (:

@yasmine-glitch
Copy link
Contributor

Hello ! Voici mes propositions concernant la description et les conditions textuelles :

Description :
"Le Revenu Solidarité Jeunes est une aide proposée par la Métropole de Lyon pour les 18-24 ans en difficulté financière. Il s'agit d'une aide financière mensuelle et la possibilité d’accéder à des actions d’accompagnement sur des thématiques variées (l’accès au logement, l’accès aux soins, l’accès à ses droits, la définition d’un projet professionnel, la découverte de l’entreprise, la mobilité…)."

conditions:

  • résider dans la Métropole de Lyon depuis au moins 6 mois,
  • ne pas bénéficier du soutien financier des parents ou d’un tiers,

@Allan-CodeWorks
Copy link
Contributor Author

Hello ! Voici mes propositions concernant la description et les conditions textuelles :

Description : "Le Revenu Solidarité Jeunes est une aide proposée par la Métropole de Lyon pour les 18-24 ans en difficulté financière. Il s'agit d'une aide financière mensuelle et la possibilité d’accéder à des actions d’accompagnement sur des thématiques variées (l’accès au logement, l’accès aux soins, l’accès à ses droits, la définition d’un projet professionnel, la découverte de l’entreprise, la mobilité…)."

conditions:

* résider dans la Métropole de Lyon depuis au moins 6 mois,

* ne pas bénéficier du soutien financier des parents ou d’un tiers,

Merci, c'est intégré!

@Allan-CodeWorks Allan-CodeWorks force-pushed the ajoute_revenu_solidarite_jeune_lyon branch from c2929ba to 9424506 Compare December 5, 2023 14:37
@Shamzic
Copy link
Contributor

Shamzic commented Dec 5, 2023

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 TypeError: Cannot read properties of undefined (reading '2023-12')

@Allan-CodeWorks Allan-CodeWorks force-pushed the ajoute_revenu_solidarite_jeune_lyon branch from 9424506 to 9eb3565 Compare December 5, 2023 14:51
Shamzic
Shamzic previously approved these changes Dec 5, 2023
Copy link
Contributor

@Shamzic Shamzic left a 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 (:

@Allan-CodeWorks
Copy link
Contributor Author

Ok pour moi, testé et fonctionnel en local (:

Oui mais ça passe pas la CI 😭

@Shamzic
Copy link
Contributor

Shamzic commented Dec 5, 2023

Ok pour moi, testé et fonctionnel en local (:

Oui mais ça passe pas la CI 😭

Could not parse situation described: {'individus': {'id': {'sous_contrat_engagement_jeune': \"You tried to calculate or to set a value for variable 'sous_contrat_engagement_jeune', but it was not found in the loaded tax and benefit system ([email protected]). Are you sure you spelled 'sous_contrat_engagement_jeune' correctly?

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 ?

@Shamzic Shamzic dismissed their stale review December 5, 2023 16:05

En attente de résolution de la CI

@Allan-CodeWorks
Copy link
Contributor Author

Ok pour moi, testé et fonctionnel en local (:

Oui mais ça passe pas la CI 😭

Could not parse situation described: {'individus': {'id': {'sous_contrat_engagement_jeune': \"You tried to calculate or to set a value for variable 'sous_contrat_engagement_jeune', but it was not found in the loaded tax and benefit system ([email protected]). Are you sure you spelled 'sous_contrat_engagement_jeune' correctly?

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 ;)

@baptou12
Copy link
Contributor

baptou12 commented Dec 5, 2023

@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 :

    generates processable YAML files
      ✕ passes OpenFisca test without extension (3367 ms)
      ✕ passes OpenFisca test with openfisca-paris extension (2969 ms)
      ✓ passes OpenFisca test with openfisca-france-local extension (4504 ms)

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

@Allan-CodeWorks Allan-CodeWorks force-pushed the ajoute_revenu_solidarite_jeune_lyon branch from 9eb3565 to 53f3453 Compare December 6, 2023 08:05
@Allan-CodeWorks
Copy link
Contributor Author

Allan-CodeWorks commented Dec 6, 2023

@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 :

    generates processable YAML files
      ✕ passes OpenFisca test without extension (3367 ms)
      ✕ passes OpenFisca test with openfisca-paris extension (2969 ms)
      ✓ passes OpenFisca test with openfisca-france-local extension (4504 ms)

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

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 ;)

@Allan-CodeWorks Allan-CodeWorks force-pushed the ajoute_revenu_solidarite_jeune_lyon branch from 53f3453 to 033045a Compare December 6, 2023 11:33
@Allan-CodeWorks Allan-CodeWorks self-assigned this Dec 6, 2023
@Allan-CodeWorks Allan-CodeWorks merged commit 8a49923 into master Dec 6, 2023
24 checks passed
@Allan-CodeWorks Allan-CodeWorks deleted the ajoute_revenu_solidarite_jeune_lyon branch December 6, 2023 11:39
@guillett guillett added this to the BC actuel milestone Dec 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants