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

Mise à jours des bourses de collège et lycée #707

Merged
merged 4 commits into from
Mar 23, 2017
Merged

Conversation

fpagnoux
Copy link
Member

@fpagnoux fpagnoux commented Mar 16, 2017

Fixes betagouv#73

  • Évolution du système socio-fiscal
  • Périodes concernées : à partir du 01/07/2016.
    • Les changements prennent effet à partir de la rentrée scolaire 2016
  • Zones impactées: prestations/education
  • Détails:
    • Met à jour le mode de calcul des bourses de collège et lycée, entrés en vigueur à la rentrée 2016.
    • Introduit les variables bourse_college_echelon et bourse_lycee_echelon

@fpagnoux fpagnoux force-pushed the maj-bourses branch 2 times, most recently from 593e6d3 to 369a27b Compare March 16, 2017 17:48
@fpagnoux fpagnoux requested review from cbenz and benjello March 16, 2017 17:49
@fpagnoux
Copy link
Member Author

fpagnoux commented Mar 16, 2017

@cbenz @benjello

On doit mettre ça à jour rapidement sur mes-aides, une revue rapide serait très appréciée ;)

Copy link
Member

@cbenz cbenz left a comment

Choose a reason for hiding this comment

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

Nice PR! Sorry for those minor fixes :)

CHANGELOG.md Outdated
* Périodes concernées : à partir du 01/07/2016.
- Les changements prennent effet à partir de la rentrée scolaire 2016
* Zones impactées: `prestations/education`
* Détails:
Copy link
Member

Choose a reason for hiding this comment

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

' ' avant les ':' en français

CHANGELOG.md Outdated
- Les changements prennent effet à partir de la rentrée scolaire 2016
* Zones impactées: `prestations/education`
* Détails:
- Met à jour le mode de calcul des bourses de collège et lycée, entrés en vigueur à la rentrée 2016.
Copy link
Member

Choose a reason for hiding this comment

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

entrés => entré (le mode de calcul)

Copy link
Member

Choose a reason for hiding this comment

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

Ou "pour la rentrée"

label = u"Montant annuel de la bourse de collège"
class bourse_college_echelon(DatedVariable):
column = IntCol
label = u"Echelon de la bourse de collège attribuée"
Copy link
Member

Choose a reason for hiding this comment

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

Accent sur le E de Échelon

label = u"Montant annuel de la bourse de collège"
class bourse_college_echelon(DatedVariable):
column = IntCol
label = u"Echelon de la bourse de collège attribuée"
Copy link
Member

Choose a reason for hiding this comment

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

Accent sur le E de Échelon

class bourse_lycee(Variable):
class bourse_lycee_echelon(Variable):
column = IntCol
label = u"Echelon de la bourse de collège attribuée"
Copy link
Member

Choose a reason for hiding this comment

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

Accent sur le E de Échelon


def function(famille, period, legislation):
"""
Références Legislatives:
Copy link
Member

Choose a reason for hiding this comment

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

Accent sur le e de Législatives

@dated_function(start = date(2016, 07, 01))
def function_apres_2016(famille, period, legislation):
"""
Références legislatives :
Copy link
Member

Choose a reason for hiding this comment

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

Accent sur le e de Législatives

@dated_function(start = date(2016, 07, 01))
def function_apres_2016(famille, period, legislation):
"""
Références Legislatives:
Copy link
Member

Choose a reason for hiding this comment

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

Accent sur le e de Législatives

Copy link
Member

Choose a reason for hiding this comment

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

(2016, 7, 1) plutôt que (2016, 07, 01) ça évitera aux éditeurs de s'énerver


def function(famille, period, legislation):
"""
Références legislatives :
Copy link
Member

Choose a reason for hiding this comment

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

Accent sur le e de Législatives

CHANGELOG.md Outdated
- Les changements prennent effet à partir de la rentrée scolaire 2016
* Zones impactées: `prestations/education`
* Détails:
- Met à jour le mode de calcul des bourses de collège et lycée, entrés en vigueur à la rentrée 2016.
Copy link
Member

Choose a reason for hiding this comment

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

Ou "pour la rentrée"

@dated_function(start = date(2016, 07, 01))
def function_apres_2016(famille, period, legislation):
"""
Références Legislatives:
Copy link
Member

Choose a reason for hiding this comment

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

(2016, 7, 1) plutôt que (2016, 07, 01) ça évitera aux éditeurs de s'énerver

nb_enfants_college = sum(
scolarite == SCOLARITE_COLLEGE for scolarite in scolarites.itervalues()
)
@dated_function(stop = date(2016, 06, 30))
Copy link
Member

Choose a reason for hiding this comment

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

Idem supra

points_de_charge += 2 * (nb_enfants >= 3) + 2 * (nb_enfants >= 4) # 2 points de charge pour les 3ème et 4ème enfants
points_de_charge += 3 * (nb_enfants >= 5) * (nb_enfants - 4) # 3 points de charge pour chaque enfant au-dessus de 4 enfants
points_de_charge += 3 * isole # 3 points de charge en plus si parent isolé
stop_date = date(2016, 07, 01)
Copy link
Member

Choose a reason for hiding this comment

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

idem supra

@@ -82,12 +152,13 @@ class bourse_lycee_nombre_parts(Variable):
label = u"Nombre de parts pour le calcul du montant de la bourse de lycée"
entity = Famille
definition_period = MONTH
stop_date = date(2016, 07, 01)
Copy link
Member

Choose a reason for hiding this comment

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

idem supra

nombre_parts = simulation.calculate('bourse_lycee_nombre_parts', period)
scolarite_holder = simulation.compute('scolarite', period)
valeur_part = simulation.legislation_at(period.start).bourses_education.bourse_lycee.valeur_part
@dated_function(start = date(2016, 07, 01))
Copy link
Member

Choose a reason for hiding this comment

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

idem supra

return nb_enfants_lycee * montant_par_enfant


@dated_function(stop = date(2016, 06, 30))
Copy link
Member

Choose a reason for hiding this comment

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

idem supra

@MattiSG MattiSG changed the title Mise à jours des bourses de collège et lycée Mise à jour des bourses de collège et lycée Mar 21, 2017
@MattiSG MattiSG changed the title Mise à jour des bourses de collège et lycée Mise à jours des bourses de collège et lycée Mar 21, 2017
@fpagnoux fpagnoux requested review from cbenz and benjello March 22, 2017 15:58
def function_apres_2016(famille, period, legislation):
"""
Références Legislatives:
Références législatives:
Copy link
Member

Choose a reason for hiding this comment

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

' ' before :

entity = Famille
definition_period = MONTH

def function(famille, period, legislation):
"""
Références Legislatives:
Références législatives:
Copy link
Member

Choose a reason for hiding this comment

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

' ' before :

@fpagnoux fpagnoux merged commit bab27fe into master Mar 23, 2017
@fpagnoux fpagnoux deleted the maj-bourses branch March 23, 2017 11:02
@fpagnoux fpagnoux removed the support label Mar 23, 2017
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.

4 participants