-
Notifications
You must be signed in to change notification settings - Fork 174
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
South Carolina Use Tax #4545
base: master
Are you sure you want to change the base?
South Carolina Use Tax #4545
Conversation
policyengine_us/parameters/gov/states/sc/tax/income/use_tax/main_rate.yaml
Outdated
Show resolved
Hide resolved
policyengine_us/parameters/gov/states/sc/tax/income/use_tax/main_rate.yaml
Outdated
Show resolved
Hide resolved
policyengine_us/parameters/gov/states/sc/tax/income/use_tax/main_rate.yaml
Outdated
Show resolved
Hide resolved
policyengine_us/parameters/gov/states/sc/tax/income/use_tax/group_3_rate.yaml
Outdated
Show resolved
Hide resolved
policyengine_us/parameters/gov/states/sc/tax/income/use_tax/county_group_3.yaml
Outdated
Show resolved
Hide resolved
policyengine_us/parameters/gov/states/sc/tax/income/use_tax/county_group_3.yaml
Outdated
Show resolved
Hide resolved
policyengine_us/parameters/gov/states/sc/tax/income/use_tax/county_group_3.yaml
Outdated
Show resolved
Hide resolved
policyengine_us/variables/gov/states/sc/tax/income/subtractions/sc_use_tax/is_group_1_county.py
Outdated
Show resolved
Hide resolved
policyengine_us/variables/gov/states/sc/tax/income/subtractions/sc_use_tax/is_group_1_county.py
Outdated
Show resolved
Hide resolved
policyengine_us/variables/gov/states/sc/tax/income/subtractions/sc_use_tax/is_group_1_county.py
Outdated
Show resolved
Hide resolved
...variables/gov/states/sc/tax/income/subtractions/sc_use_tax/sc_use_tax_in_group_one_county.py
Outdated
Show resolved
Hide resolved
policyengine_us/parameters/gov/states/sc/tax/income/use_tax/county_group/one.yaml
Show resolved
Hide resolved
policyengine_us/variables/gov/states/sc/tax/income/subtractions/sc_use_tax/sc_use_tax.py
Outdated
Show resolved
Hide resolved
policyengine_us/variables/gov/states/sc/tax/income/subtractions/sc_use_tax/sc_use_tax.py
Outdated
Show resolved
Hide resolved
policyengine_us/variables/gov/states/sc/tax/income/subtractions/sc_use_tax/sc_use_tax.py
Outdated
Show resolved
Hide resolved
...variables/gov/states/sc/tax/income/subtractions/sc_use_tax/sc_use_tax_in_group_one_county.py
Outdated
Show resolved
Hide resolved
policyengine_us/tests/policy/baseline/gov/states/sc/tax/income/sc_use_tax/sc_use_tax_group.yaml
Outdated
Show resolved
Hide resolved
policyengine_us/variables/gov/states/sc/tax/income/subtractions/sc_use_tax/sc_use_tax.py
Outdated
Show resolved
Hide resolved
policyengine_us/variables/gov/states/sc/tax/income/subtractions/use_tax/sc_use_tax.py
Show resolved
Hide resolved
policyengine_us/parameters/gov/states/sc/tax/income/use_tax/county_group/one.yaml
Show resolved
Hide resolved
policyengine_us/parameters/gov/states/sc/tax/income/use_tax/group_two.yaml
Outdated
Show resolved
Hide resolved
policyengine_us/parameters/gov/states/sc/tax/income/use_tax/county_group/one.yaml
Outdated
Show resolved
Hide resolved
...tests/policy/baseline/gov/states/sc/tax/income/use_tax/sc_use_tax_in_group_one_counties.yaml
Show resolved
Hide resolved
policyengine_us/parameters/gov/states/sc/tax/income/use_tax/county_group/one.yaml
Outdated
Show resolved
Hide resolved
...tests/policy/baseline/gov/states/sc/tax/income/use_tax/sc_use_tax_in_group_two_counties.yaml
Outdated
Show resolved
Hide resolved
...tests/policy/baseline/gov/states/sc/tax/income/use_tax/sc_use_tax_in_group_two_counties.yaml
Outdated
Show resolved
Hide resolved
...sts/policy/baseline/gov/states/sc/tax/income/use_tax/sc_use_tax_in_group_three_counties.yaml
Outdated
Show resolved
Hide resolved
...tests/policy/baseline/gov/states/sc/tax/income/use_tax/sc_use_tax_in_group_one_counties.yaml
Outdated
Show resolved
Hide resolved
policyengine_us/variables/household/expense/out_of_state_purchase.py
Outdated
Show resolved
Hide resolved
policyengine_us/variables/household/expense/out_of_state_purchase.py
Outdated
Show resolved
Hide resolved
policyengine_us/variables/gov/states/sc/tax/income/subtractions/use_tax/sc_use_tax.py
Outdated
Show resolved
Hide resolved
…s/use_tax/sc_use_tax.py Co-authored-by: Pavel Makarchuk <[email protected]>
…ase.py Co-authored-by: Pavel Makarchuk <[email protected]>
Lint and test failures |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #4545 +/- ##
==========================================
+ Coverage 98.98% 99.00% +0.01%
==========================================
Files 2517 2533 +16
Lines 36615 36833 +218
Branches 197 196 -1
==========================================
+ Hits 36245 36467 +222
+ Misses 339 337 -2
+ Partials 31 29 -2 ☔ View full report in Codecov by Sentry. |
policyengine_us/parameters/gov/states/sc/tax/income/use_tax/main.yaml
Outdated
Show resolved
Hide resolved
…in.yaml Co-authored-by: Pavel Makarchuk <[email protected]>
…young/policyengine-us into keepjojoyoung/issue4514
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
could we instead map counties to county groups, then have a breakdown parameter for the % for each county group? similar to state_group
@@ -0,0 +1,11 @@ | |||
description: South Carolina levies a local use tax of group 1 counties at this percent of out of state purchase. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
and others
description: South Carolina levies a local use tax of group 1 counties at this percent of out of state purchase. | |
description: South Carolina levies a local use tax of group 1 counties at this percent of out of state purchases. |
Fixes #4514