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

Corrige la formule APA urgence (institution) #1374

Merged
merged 2 commits into from
Oct 23, 2019

Conversation

benjello
Copy link
Member

@benjello benjello commented Sep 19, 2019

Fix #1364

  • Correction d'un crash.
  • Périodes concernées : à partir du 01/01/2002.
  • Zones impactées : prestations/autonomie.py.
  • Détails :
    • Corrige apa_urgence_institution
    • Corrige une date manquant dans l'appel de l'arbre de paramètres

Ces changements (effacez les lignes ne correspondant pas à votre cas) corrigent ou améliorent un calcul déjà existant.


Quelques conseils à prendre en compte :

Et surtout, n'hésitez pas à demander de l'aide ! :)

@bonjourmauko bonjourmauko changed the title Fix missing instant in parameters Corrige date manquante dans l'appel de l'arbre de paramètres Sep 26, 2019
@bonjourmauko bonjourmauko changed the title Corrige date manquante dans l'appel de l'arbre de paramètres Corrige date manquante dans l'arbre de paramètres Sep 26, 2019
Copy link
Member

@bonjourmauko bonjourmauko left a comment

Choose a reason for hiding this comment

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

Merci @benjello pour ta contribution !

Quelques commentaires et deux suggestions :

  1. Peux-tu ajouter une référence legislative ?
  2. Peux-tu ajouter un test ? À vue d'oeil la formule contient des erreurs.

Une idée de test :

- name: APA établissement - test situation d'urgence
  description: Montant APA forfataire en cas d'urgence
  input:
    dependance_tarif_etablissement_gir_5_6:
      2001-12: 600
      2002-01: 600
  output:
    apa_urgence_institution:
      2001-12: 0
      2002-01: 300 # 600 * 0.5

@benjello
Copy link
Member Author

@maukoquiroga : jai corrigé dans un certain sens mais j'attends la confirmation de notre experte métier @elsaperdrix pour savoir si c'est ok.

@benjello
Copy link
Member Author

benjello commented Sep 27, 2019

@maukoquiroga : a priori c'est ok désormais.

@benjello
Copy link
Member Author

@maukoquiroga : je te laisser rebaser et merger. Tu peux récupérer le changelog dans la description de la PR.

@bonjourmauko
Copy link
Member

bonjourmauko commented Oct 23, 2019

Hello @benjello ! Afin de merger ce PR au plus vite, je me suis permis de supprimer l'ajout suivant, car je n'ai pas réussi à comprendre sa raison d'être, ni par conséquence d'écrire un scénario de test qui fasse du sens. À ta disposition pour qu'on itère dessus si c'est important :

def formula_2002(individu, period):
    gir = individu('gir', period)
    dependance_tarif_etablissement_gir_1_2 = individu('dependance_tarif_etablissement_gir_1_2', period)
    dependance_tarif_etablissement_gir_3_4 = individu('dependance_tarif_etablissement_gir_3_4', period)
    gir_regroupe = [
        (gir == TypesGir.gir_1) + (gir == TypesGir.gir_2),
        (gir == TypesGir.gir_3) + (gir == TypesGir.gir_4),
        ]
    dependance_tarif_etablissement_gir_regroupe = [
        dependance_tarif_etablissement_gir_1_2,
        dependance_tarif_etablissement_gir_3_4,
        ]
    dependance_tarif_etablissement_gir_dependant = select(gir_regroupe, dependance_tarif_etablissement_gir_regroupe)
    return dependance_tarif_etablissement_gir_dependant

@bonjourmauko bonjourmauko changed the title Corrige date manquante dans l'arbre de paramètres Corrige la formule apa_urgence Oct 23, 2019
@bonjourmauko bonjourmauko changed the title Corrige la formule apa_urgence Corrige la formule APA urgence (institution) Oct 23, 2019
@bonjourmauko
Copy link
Member

Merci à toutes et à tous ! @elsaperdrix @nico5655 @Morendil @benjello

@bonjourmauko bonjourmauko merged commit bb09f20 into openfisca:master Oct 23, 2019
@benjello
Copy link
Member Author

@maukoquiroga : il faut le garder car chaque individu un un établissement et chaque établissement à des tarifs pour différent gir. Et il nous faut le tarif déterminé par la formule pour pouvoir calculer l'APA corrigée dans cette PR.

@bonjourmauko
Copy link
Member

@benjello Merci pour l'explication, c'est plus clair. Mettons cela dans une nouvelle PR pendant la semaine.

@benjello
Copy link
Member Author

@maukoquiroga : du coup qui fait quoi ? Je te propose de remettre ce que tu as retiré et de me dire quels tests rajouter et je le ferai volontiers. Cela te convient-il ? Merci !

@benjello benjello deleted the fix-autonomie branch October 28, 2019 14:30
@benjello benjello restored the fix-autonomie branch October 28, 2019 14:30
@bonjourmauko
Copy link
Member

@benjello Oui pardon ma proposition est d’ouvrir une nouvelle PR avec cette partie et un scénario de test, j’avais besoin de plus de contexte pour savoir quoi tester. Je prends !

@benjello benjello deleted the fix-autonomie branch March 24, 2021 07:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Erreur dans la formule APA urgence institution
2 participants