PolicyEngine / PolicyEngine/policyengine-us

Change the type of age variable to int

Open
#3,357 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Python
Stars
163
Forks
213
Avg merge
3d 3h
Merged PRs (30d)
96

Description

Currently, the type is float: https://github.com/PolicyEngine/policyengine-us/blob/master/policyengine_us/variables/household/demographic/age/age.py. This causes age to be displayed as a float in the app: https://github.com/PolicyEngine/policyengine-app/issues/928. On the other hand, float ages seem to be needed in variables like ccdf_age_group.

Contributor guide

Open the contributing guide

Research direction

Start with policyengine_us/variables/household/demographic/age/age.py and review how the age variable is defined. Then inspect the ccdf_age_group relationship mentioned in the issue and the linked app issue for the display impact. Done means age is displayed as an integer while variables that need fractional ages continue to work.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
backend
Issue type
Refactor
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.