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.

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

Mic 4208/1040 metadata #226

Merged
merged 6 commits into from
Jul 19, 2023
Merged

Mic 4208/1040 metadata #226

merged 6 commits into from
Jul 19, 2023

Conversation

albrja
Copy link
Contributor

@albrja albrja commented Jul 18, 2023

Title: Mic 4208/1040 metadata

Description

Adds metadata for 1040 dataset to schema entities.

  • Category: Feature
  • JIRA issue: MIC-4208

-adds additional 1040 columns to schema entities
-adds columns to 1040 dataset

Testing

All tests pass

NOISE_TYPES.make_typos,
),
)
spouse_middle_initial: Column = Column(
Copy link
Contributor

@stevebachmeier stevebachmeier Jul 18, 2023

Choose a reason for hiding this comment

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

We don't need dependent 1-4 middle initial?

edit: nm, I see it in the docs now

COLUMNS.dependent_1_last_name,
COLUMNS.dependent_1_ssn,
COLUMNS.dependent_2_first_name,
COLUMNS.dependent_2_last_name,
Copy link
Contributor

Choose a reason for hiding this comment

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

The docs request "type of GQ" columns as well. Do we need household_type here?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It says for noise functions only. I can verify with RT. Im also not sure about which row types we need since that is not 100% clear to me in the docs.

Base automatically changed from mic-4218/format-1040 to develop July 18, 2023 18:09
@albrja albrja force-pushed the mic-4208/1040-metada branch from 6fd1a8e to 1d0c840 Compare July 18, 2023 21:22
@albrja albrja force-pushed the mic-4208/1040-metada branch from 1d0c840 to 91b9b69 Compare July 19, 2023 17:57
@albrja albrja merged commit 50a6ed2 into develop Jul 19, 2023
@albrja albrja deleted the mic-4208/1040-metada branch July 19, 2023 18:21
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.

3 participants