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

defined_for doesn't work for simulation-defining formulas #74

Open
nikhilwoodruff opened this issue Aug 25, 2022 · 0 comments
Open

defined_for doesn't work for simulation-defining formulas #74

nikhilwoodruff opened this issue Aug 25, 2022 · 0 comments
Labels
bug Something isn't working

Comments

@nikhilwoodruff
Copy link
Contributor

This is a pretty complex edge case I didn't consider when writing the defined_for logic. The way that defined_for works is by intercepting the entity(variable, period) calls inside a subsetted variable's formula and pre-subsetting them, so normal operations on them return the subsetted population results. But no interception happens when a formula creates a new simulation and uses outputs from simulation.calculate.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant