Skip to content

Fix vectorial parameter default lookup#371

Closed
baogorek wants to merge 2 commits intomasterfrom
codex/investigate-valueerror-in-microsimulation-calculation
Closed

Fix vectorial parameter default lookup#371
baogorek wants to merge 2 commits intomasterfrom
codex/investigate-valueerror-in-microsimulation-calculation

Conversation

@baogorek
Copy link
Copy Markdown
Collaborator

Summary

  • fix default creation when keys may be missing
  • test missing key as first index
  • add changelog entry

Testing

  • make format
  • make test (fails: policyengine-core command not found)
  • pytest tests/core/parameters_fancy_indexing/test_fancy_indexing.py::test_wrong_key_first -q (fails: No module named 'numpy')

https://chatgpt.com/codex/tasks/task_e_684894c660808326aa098007bc8cbbd6

@baogorek
Copy link
Copy Markdown
Collaborator Author

Closing this. There could be an opportunity to make this more robust, or at the very least add in some better error messages, but the problem is with missing parameter values.

@baogorek baogorek closed this Jun 10, 2025
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.

1 participant