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

Fix scale units #1107

Merged
merged 6 commits into from
Nov 8, 2018
Merged

Fix scale units #1107

merged 6 commits into from
Nov 8, 2018

Conversation

benjello
Copy link
Member

@benjello benjello commented Sep 6, 2018 •
edited by Morendil
Loading

  • Amélioration technique.
  • Périodes concernées : toutes.
  • Zones impactées : certains barèmes dont les seuils sont en unité monétaire dans les paramètres de la législation.
  • Détails :
    • Utilisation des champs rate_unit et threshold_unit dans les méta-données (champ metadata) des paramètres qui sont des barèmes.
    • Correction du script d'investigation des unités

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

  • Corrigent ou améliorent un calcul déjà existant.

En relation avec #1019

@benjello benjello requested review from fpagnoux and claireleroy and removed request for fpagnoux September 6, 2018 06:17
@claireleroy
Copy link
Contributor

@benjello Du coup la solution que j'avais bricolé au CASD ne s'appliquera plus, il faudra faire attention la prochaine fois qu'on actualise les repo que tes changements écrasent bien les miens.

@benjello
Copy link
Member Author

@claireleroy : je peux attendre avant de merger hein si c'est plus simple.

Morendil
Morendil reviewed Sep 14, 2018
View reviewed changes
openfisca_france/scripts/parameters/explore_parameters_unit.py Outdated
@@ -11,11 +11,12 @@

def get_parameters_by_unit(parameter, parameters_by_unit = None):
"""
Build a dictionnary collectiing the legislation aprameters according to their units
Build a dictionnary collectiing the legislation aprameters according to their units
Copy link
Contributor

Choose a reason for hiding this comment

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

collectiing -> collecting (la typo est d'origine mais tant que tu y es…)

@Morendil
Copy link
Contributor

Morendil commented Oct 6, 2018 •
edited
Loading

Je ferme temporairement pour éviter que la PR soit mergée accidentellement, j'ai mal arbitré entre le "Approve" de @claireleroy et l'avertissement sur le fonctionement au CASD. Réouverture programmée la semaine du lundi 15/10.

@Morendil Morendil closed this Oct 6, 2018
@Morendil Morendil reopened this Oct 14, 2018
@Morendil
Copy link
Contributor

@benjello On fait quoi finalement pour cette PR ?

@benjello
Copy link
Member Author

Je suis en train de faire le mette côté CASD et je rappatrie tout ici. Ce sera mergé cette semaine ou la prochaine.

@Morendil
Copy link
Contributor

Morendil commented Nov 7, 2018

@benjello Up ?

@benjello
Copy link
Member Author

benjello commented Nov 7, 2018

C'est en cours avec @claireleroy mais cela va mettre un peu plus de temps que prévu ...

@benjello
Copy link
Member Author

benjello commented Nov 7, 2018

@Morendil : @claireleroy reprend el flambeau pour rebaser et merger !

@benjello benjello assigned claireleroy and unassigned benjello Nov 7, 2018
@claireleroy
Copy link
Contributor

@sandcha Merci ! Je merge alors !

@claireleroy claireleroy merged commit 4901bce into master Nov 8, 2018
@Morendil Morendil deleted the fix-scale-units branch November 8, 2018 19:01
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

@claireleroy claireleroy claireleroy approved these changes

@fpagnoux fpagnoux Awaiting requested review from fpagnoux

Assignees

@claireleroy claireleroy

Labels
kind:evolution
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@benjello @claireleroy @Morendil @bonjourmauko @sandcha