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

[WIP] - assurance chômage #1481

Closed
wants to merge 92 commits into from
Closed

[WIP] - assurance chômage #1481

wants to merge 92 commits into from

Conversation

lauradelmas8
Copy link
Contributor

Merci de contribuer à OpenFisca ! Effacez cette ligne ainsi que, pour chaque ligne ci-dessous, les cas ne correspondant pas à votre contribution :)

  • Évolution du système socio-fiscal. | Amélioration technique. | Correction d'un crash. | Changement mineur.
  • Périodes concernées : toutes. | jusqu'au JJ/MM/AAAA. | à partir du JJ/MM/AAAA.
  • Zones impactées : chemin/vers/le/fichier/contenant/les/variables/impactées.
  • Détails :
    • Description de la fonctionnalité ajoutée ou du nouveau comportement adopté.
    • Cas dans lesquels une erreur était constatée.

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

  • Modifient l'API publique d'OpenFisca France (par exemple renommage ou suppression de variables).
  • Ajoutent une fonctionnalité (par exemple ajout d'une variable).
  • Corrigent ou améliorent un calcul déjà existant.
  • Modifient des éléments non fonctionnels de ce dépôt (par exemple modification du README).

Quelques conseils à prendre en compte :

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

salaire_de_reference = individu('salaire_de_reference', instant.last_12_months, options=[ADD])


montant_journalier = max_(parameters.ARE.partie_fixe + (parameters.ARE.%_du_SJR_complement * salaire_journalier_de_reference), parameters.ARE.%_du_SJR_seul * salaire_journalier_de_reference)
Copy link
Member

Choose a reason for hiding this comment

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

@lauradelmas8 : éviter le majuscules et les signes spéciaux (%) dans les paramètres et les noms de fichiers.
Au lieu de % utiliser part.
Et j'essaierais d'être plus explicite sur le nom des paramètres.
Par exemple montant_minimal au lie de min
J'éviterais de mettre des mots en anglais sur des paramètres législatifs français (age_limite au lieu de age_threshold).

Copy link
Member

@benjello benjello left a comment

Choose a reason for hiding this comment

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

Une fois les formules nettoyées je pense qu'il faudra rassembler les variable nombre_de_jours_travailles_dans_les_x_derniers_mous_avant/apres_53_ans en une seule variable.

Je pourrais t'aider à le faire.

Je te conseille aussi de faire plus de tests sur les périodes travaillées et le calcul du salaire de référence. Essaie de te mettre sur des cas limites (juste avant et juste après l'éligibilité etc) pour être sûr de bien les gérer.

@@ -31,3 +34,169 @@ class indemnites_chomage_partiel(Variable):
label = "Indemnités de chômage partiel"
definition_period = MONTH
set_input = set_input_divide_by_period


class salaire_de_reference(Variable):
Copy link
Member

Choose a reason for hiding this comment

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

D'après les références législatives résumées dans ton document, le salaire de référence est la somme des salaires des 12 derniers mois avant la fin du contrat de travail divisés par la quotité de travail ce qui n'est pas le cas ici.

Copy link
Member

Choose a reason for hiding this comment

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

@lauradelmas8 : il reste ce point que l'on n'a pas réglé mais on peut le faire plus tard (ou pas).
Essayons de garder nos discussion ici pour faciliter le suivi.

openfisca_france/model/revenus/remplacement/chomage.py Outdated Show resolved Hide resolved
openfisca_france/model/revenus/remplacement/chomage.py Outdated Show resolved Hide resolved
openfisca_france/model/revenus/remplacement/chomage.py Outdated Show resolved Hide resolved
openfisca_france/model/revenus/remplacement/chomage.py Outdated Show resolved Hide resolved
openfisca_france/model/revenus/remplacement/chomage.py Outdated Show resolved Hide resolved
@guillett
Copy link
Member

C'est vraiment top que ces calculs se retrouvent dans OpenFisca, merci beaucoup ! Pour info dans #1503 j'ai récupéré le paramètre ARE_minimal car j'en avais besoin dans un calcul. J'ai fait un commit dédié attribué à @lauradelmas8. (J'en ai profité pour renommer le paramètre pour le rendre plus explicite :) )

@lauradelmas8
Copy link
Contributor Author

Parfait, merci et pas de soucis ! :)

@bfabre01
Copy link

Ce travail n'est plus à l'ordre du jour et date de presque deux ans. Je ferme. Si un jour on veut y revenir, on pourra toujours réouvrir.

@bfabre01 bfabre01 closed this Jan 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants