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

Move random elements to policyengine-uk-data #960

Open
MaxGhenis opened this issue Oct 15, 2024 · 2 comments
Open

Move random elements to policyengine-uk-data #960

MaxGhenis opened this issue Oct 15, 2024 · 2 comments
Labels
enhancement New feature or request

Comments

@MaxGhenis
Copy link
Collaborator

MaxGhenis commented Oct 15, 2024

As we have done in the US for stability and keeping the rules engine deterministic. Includes benefit take-up, private school attendance, and so on.

@nikhilwoodruff
Copy link
Collaborator

I think this is causing a bug where take up is varying before and after the consumer-incident taxes, so I'll work on this now.

@nikhilwoodruff nikhilwoodruff self-assigned this Oct 18, 2024
@nikhilwoodruff nikhilwoodruff added the enhancement New feature or request label Oct 18, 2024
@nikhilwoodruff
Copy link
Collaborator

Fixed in terms of the actual data package having the take up values. But the logic is still in PE-UK, so leaving open- we should move it to the data package.

@nikhilwoodruff nikhilwoodruff removed their assignment Oct 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants