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

Improve set_input_dispath error message #1107

Open
benjello opened this issue Feb 8, 2022 · 0 comments
Open

Improve set_input_dispath error message #1107

benjello opened this issue Feb 8, 2022 · 0 comments

Comments

@benjello
Copy link
Member

benjello commented Feb 8, 2022

Hi there!

I really enjoy OpenFisca, but I recently encountered an issue.

Here is what I did:

I tried to initialize a Simulation using a TaxBenefitSystem with conflicting set_input and definition_periodattributes

Here is what I expected to happen:

Detect the error at the TaxBenefitSystem creation level or at least show an error message indicating the name of variable of concern and its definition_period.

Here is what actually happened:

I got this unclear error message

clearer error message then the one I got

'set_input_dispatch_by_period can be used only for yearly or monthly variables.'

Context

I identify more as a:

  • Developer (I create tools that use the existing OpenFisca code).
  • Producer (I make computations on individual situations).
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

No branches or pull requests

1 participant