Skip to content

Conversation

DTrim99
Copy link
Collaborator

@DTrim99 DTrim99 commented Aug 20, 2025

Fixes #6420

@DTrim99 DTrim99 marked this pull request as draft August 20, 2025 16:29
@DTrim99 DTrim99 requested a review from PavelMakarchuk August 20, 2025 16:32
Copy link
Collaborator

@PavelMakarchuk PavelMakarchuk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice! Some comments around format and usage

@DTrim99 DTrim99 requested a review from PavelMakarchuk August 25, 2025 21:19
@PavelMakarchuk PavelMakarchuk marked this pull request as ready for review August 31, 2025 23:21
def formula(person, period, parameters):
p = parameters(period).gov.local.ny.mamdani_income_tax
taxable_income = person("nyc_taxable_income", period)
threshold = p.threshold
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

irrelevant if moving to a scale parameter, but call parameters directly

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

combine into a scale parameter

@@ -0,0 +1,10 @@
description: Zohran Mamdani's proposed this NYC income tax rate on income above a certain threshold.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this style for the scale param

Suggested change
description: Zohran Mamdani's proposed this NYC income tax rate on income above a certain threshold.
description: Zohran Mamdani proposed taxing New York City residents' income above a threshold at this rate.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Mamdani Millionaire Income Tax
3 participants