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

Rangement du complement_are #2000

Merged
merged 6 commits into from
Feb 2, 2023
Merged

Rangement du complement_are #2000

merged 6 commits into from
Feb 2, 2023

Conversation

Sasha-Laniece
Copy link
Contributor

@Sasha-Laniece Sasha-Laniece commented Jan 19, 2023

  • Correction d'un crash.
  • Périodes concernées : toutes.
  • Zones impactées : parameters/chomage.
  • Détails - L'ajout du complement_are n'a pas été fait en respectant les bonnes pratiques de modélisation:
    • plusieurs paramètres dans un même yaml
    • l'arborescence de rangement n'est pas respectée
    • le formatage du paramètre est à revoir (ux_name, references)
    • il y a un doublon avec un autre paramètre

Ces changements :

  • Modifient l'API publique d'OpenFisca France (par exemple renommage ou suppression de variables).
  • Corrigent ou améliorent un calcul déjà existant.

@Sasha-Laniece Sasha-Laniece marked this pull request as ready for review January 20, 2023 08:24
@Sasha-Laniece
Copy link
Contributor Author

Sasha-Laniece commented Jan 20, 2023

Salut @LucasDetre , je me suis permis de te demander en revue car je suis tombée sur le complement_are et je vois que tu n'as pas respecté les règles de modélisation.
Pour t'aider, elles sont décrites ici: https://github.com/openfisca/openfisca-france/wiki/Bonnes-pratiques-de-mod%C3%A9lisation
Elles vont légèrement changer dans les prochaines semaines, mais l'essentiel restera.

Je te laisse observer (et valider) mes corrections. Je suis à ta disposition si tu as des questions !

@Sasha-Laniece Sasha-Laniece linked an issue Jan 20, 2023 that may be closed by this pull request
5 tasks
…cient_allocation_journaliere` vers `chomage.allocations_assurance_chomage.alloc_base_taux.pourcentage_sjr`
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.

Ranger et harmoniser complement_are
2 participants