Compute labor supply response revenue_change instead of preserving legacy zero
I maintainer di solito rispondono entro 1 giorno
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 5/5
- Tempo stimato
- Più di una settimana
- Idoneità per principianti
- 35/100
- Tipo di issue
- Funzionalità
- Chiarezza
- Abbastanza chiara
- Stato di attività
- Tranquilla
- Stack tecnologico
- python
- Ambito
- backend-api-design
Direzione di ricerca
Inizia con calculate_labor_supply_response(...) e esamina la PR #360 per comprendere l'output esistente di LaborSupplyResponse e il comportamento legacy di budgetary_impact_lsr. Definisci il significato e la convenzione dei segni per revenue_change, quindi identifica gli output di simulazione disponibili necessari per calcolarlo. Il lavoro è completato quando le esecuzioni attive producono effetti non nulli verificati, le esecuzioni inattive rimangono pari a zero e qualsiasi divergenza legacy è documentata.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Context
PR #360 (Add labor supply response macro output) adds a 4.x-compatible LaborSupplyResponse output that preserves the legacy macro output shape.
As part of that parity work, revenue_change is intentionally set to 0.0 because the legacy API initialized budgetary_impact_lsr to zero and never populated it. That preserves existing public behavior, but it leaves a real fiscal-effect gap: callers may reasonably interpret labor_supply_response.revenue_change as the tax/revenue impact caused by labor-supply responses.
Current behavior
calculate_labor_supply_response(...) computes LSR behavioral quantities such as:
income_lsrsubstitution_lsrtotal_changerelative_lsr- decile average/relative breakdowns
- hours effects where supported
But revenue_change remains hard-coded to 0.0 for both active and inactive LSR paths.
Desired behavior
Determine and implement the correct fiscal calculation for LaborSupplyResponse.revenue_change, rather than preserving the legacy zero.
Acceptance criteria
- Define the intended meaning and sign convention for
revenue_change. - Compute
revenue_changefor active LSR runs using available simulation outputs or clearly documented additional required outputs. - Preserve zero behavior for inactive LSR runs.
- Add focused tests for non-zero revenue effects, zero/inactive behavior, and sign convention.
- Document any intentional divergence from legacy
budgetary_impact_lsrbehavior.
Related PR: #360
- Lingua principale
- Python
- Stelle
- 8
- Fork
- 9
- Merge medio
- 15h 28m
- PR unite (30g)
- 13
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Nessun modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di PolicyEngine/policyengine.py
-
Raise the simulation cache memory warning to 16 GiBForse già presa @anth-volk l’ha presa 3 giorni fa. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
PolicyEngine/policyengine.py#549 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
PolicyEngine/policyengine.py#473 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
PolicyEngine/policyengine.py#436 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
Certify populace-uk as the UK certified defaultForse di nuovo libera Una pull request per questa issue è stata chiusa senza essere unita. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
PolicyEngine/policyengine.py#408 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 48/100
PolicyEngine/policyengine.py#551 ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di PolicyEngine/policyengine.py
Issue simili
-
Layers of type "3D Unknown" and "3D Measured Unknown" are refused, so FileGDB multipatch layers can't be readForse già presa @aidannewsome l’ha presa oggi. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
-
quantization
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
vllm-project/vllm#59904 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
mikf/gallery-dl#9791 ·
-
Speaker custom-fields CSV export returns 404 instead of downloading dataForse già presa @techvoyager-varun l’ha presa oggi. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 88/100
fossasia/eventyay#6151 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
P4: low tooling
Difficoltà 2/5 1-3 ore Idoneità per principianti 88/100
jeffknupp/association#318 ·