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

Add script and methods to save YAML parameter tree on disk #1023

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

benjello
Copy link
Member

Thanks for contributing to OpenFisca! Remove this line, as well as any other in the following that don't fit your contribution :)

New features

  • Introduce a script to unfold and normalize parameters

@benjello
Copy link
Member Author

benjello commented May 31, 2021

@Sasha-Laniece @sandcha @eraviart : here is @cbenz script to unfold an normalize parameters.
It would be nice to be able to use it for the convergence openfisca-france <-> barèmes IPP but it may be useful for larger purposes. But please review it to see if it is sufficient or can be improved.

It would be nice to have a complementary tool to fold parameters at a fixed directory depth/path length too.

This was used for exemple in openfisca/openfisca-france#1414.

@MattiSG
Copy link
Member

MattiSG commented Aug 9, 2021

Thanks @benjello @cbenz for sharing this script which, if I understand correctly, generates a dump of all parameters in their full name in a YAML file!

In order to better assess how this could be useful to the wider community, could you please:

  1. Confirm what this does (I also see mentions of maximum name length, so maybe it does more than just dumping a YAML list?).
  2. Clarify what are the use cases (how does it help in converging two separate sets of parameters?).
  3. Provide an example output of this script (if possible on the Country Template, so that we have a clear and concise reference).

Thank you! 🙂

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.

2 participants