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.

Sign up for GitHub

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

Jump to bottom

Change _lsr to _behavioral_response #5258

Open
MaxGhenis opened this issue Oct 21, 2024 · 0 comments · May be fixed by #5321
Open

Change _lsr to _behavioral_response #5258

MaxGhenis opened this issue Oct 21, 2024 · 0 comments · May be fixed by #5321
Assignees
@Mshuning

Comments

@MaxGhenis
Copy link
Contributor

As we add other behavioral responses, like capital gains and deductions, we will want to name them consistently. We can start by changing, for example, employment_income_before_lsr to employment_income_before_behavioral_response.
Mshuning added a commit to Mshuning/policyengine-us that referenced this issue Oct 31, 2024
@Mshuning
Change _lsr to _behavioral_response
7203d33 
Fixes PolicyEngine#5258
@Mshuning Mshuning mentioned this issue Oct 31, 2024
Closed
Mshuning added a commit to Mshuning/policyengine-us that referenced this issue Oct 31, 2024
@Mshuning
Change _lsr to _behavioral_response
86189dc 
Fixes PolicyEngine#5258
@Mshuning Mshuning linked a pull request Nov 12, 2024 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Mshuning Mshuning

Labels
None yet
Projects
None yet
Milestone
No milestone
2 participants
@MaxGhenis @Mshuning