-
Notifications
You must be signed in to change notification settings - Fork 9
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
Reform dynamique : ajoute la gestion de la condition attached_to_institution
#191
Reform dynamique : ajoute la gestion de la condition attached_to_institution
#191
Conversation
a0a872c
to
e124d1a
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.
Je te l'ai review comme si c'était good commit par commit, si tu veux les changer.
Sinon on peut ignorer.
7be7351
to
59b7355
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.
Top juste la question sur les regions et une proposition de simplification de code
59b7355
to
cd7770d
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.
LGTM, merci pour les rebases
cd7770d
to
170d33b
Compare
Description
La condition
attached_to_instititution
n'as pas été développée jusqu'a maintenant car elle n'est pas explicitée dans le fichier yaml d'une aide.Cette condition est une condition géographique dont le périmètre est régi par l'institution en question.
Les aides avec cette condition ont un champ
institution
dans lequel réside une chaine de caractères qui correspond a nom d'un fichier dans un autre répertoire sans leur extension.yml
(pas compatible avec.yaml
pour le moment)Cette particularité technique implique que la condition ne peut pas être traitée comme les autres car il faudrait intégrer toutes les institutions dans openfisca, ce qui ne semble pas souhaitable.
Cette PR propose donc d'agir avant l'importation des aides dans le tax and benefit system, juste après avoir lu les fichiers d'aides et les avoir transformés en
dict
, on :attached_to_institution
desconditions_generales
institution
institution
dans les conditions généralesModifié en même temps :
Dans le
boy scout spirit
je me suis dit qu'il était ok de faire quelques changements :KeyError
àNotImplementedError
Added
attaches_to_institution