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.

Sign up for GitHub

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

Jump to bottom

Importe la Garantie Jeunes depuis betagouv/openfisca-france-local. #1347

Merged
merged 8 commits into from
Jun 24, 2019
Merged

Importe la Garantie Jeunes depuis betagouv/openfisca-france-local. #1347

merged 8 commits into from
Jun 24, 2019

Conversation

alexsegura
Copy link
Contributor

  • Changement mineur.
  • Périodes concernées : à partir du 08/08/2016.
  • Zones impactées : N/A.
  • Détails :
    • Ajoute garantie_jeunes

Ces changements (effacez les lignes ne correspondant pas à votre cas) :

  • Ajoutent une fonctionnalité (par exemple ajout d'une variable).

@alexsegura alexsegura requested review from Morendil and guillett June 18, 2019 14:59
@benjello
Copy link
Member

@alexsegura : merci pour ce travail. Personnellement, je ne créerais pas les variables:

  • garantie_jeunes_max
  • garantie_jeunes_eligibilite_age

@Morendil
Copy link
Contributor

@benjello Je suis plutôt de l'avis contraire, suite aux discussions qu'on a eues sur Core (cf. openfisca/openfisca-core#872 et openfisca/openfisca-core#890 pour la partie "extraire les variables de conditions"), ce sont des choses qu'on a découragé historiquement mais le contexte a changé…

Morendil
Morendil requested changes Jun 18, 2019
View reviewed changes
openfisca_france/model/prestations/garantie_jeunes.py Outdated
montant_base = params.montant_de_base_du_rsa
taux_1_personne = params.forfait_logement.taux_1_personne

# Une formule doit retourner un vecteur
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ce n'est plus le cas! Tu peux renvoyer un scalaire maintenant.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

J'ai changé :)

@alexsegura alexsegura requested a review from Morendil June 19, 2019 09:04
@benjello
Copy link
Member

@Morendil @alexsegura : a minima il faut inclure les variables intermédiaires dans openfisca_france/conf/cache_blacklist.py.

Pour le changement de contexte, je dirais qu'il n'a pas encore changé ;-)
J'attends de voir une nette amélioration des performances.

Morendil
Morendil requested changes Jun 23, 2019
View reviewed changes
openfisca_france/model/prestations/garantie_jeunes.py Outdated
value_type = bool
entity = Individu
definition_period = MONTH
label = u"Variable NEET - Not in Employement, Education or Training"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ca serait mieux en français quand même.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

J'ai mis en français et ajouté une référence.

@alexsegura alexsegura requested a review from Morendil June 24, 2019 08:22
@Morendil
Copy link
Contributor

Merci @alexsegura je prends la suite

@Morendil Morendil merged commit f9e6731 into master Jun 24, 2019
@Morendil Morendil deleted the garantie-jeunes branch June 24, 2019 09:59
This was referenced Jun 25, 2019
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Morendil Morendil Morendil approved these changes

@guillett guillett Awaiting requested review from guillett

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@alexsegura @benjello @Morendil