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

Update DD - not implemented #69

Merged
merged 12 commits into from
Jun 19, 2024
Merged

Update DD - not implemented #69

merged 12 commits into from
Jun 19, 2024

Conversation

rpassas
Copy link
Collaborator

@rpassas rpassas commented Jun 14, 2024

Summary

  • restructuring "not implemented" columns in the data dictionary
  • adding comments for void / alias rows
  • adding additional sections for coded content

I will squash some commits from the rebase once approved

@rpassas rpassas changed the title Break up dd Update DD - not implemented Jun 14, 2024
@rpassas rpassas requested a review from ptran265 June 18, 2024 15:42
Copy link
Collaborator

@saulakravitz saulakravitz left a comment

Choose a reason for hiding this comment

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

Looks good.
I think the descriptions of the sections of the DD should be extended roughly as follows:
Death Record Section: These fields are included in a VRO's submission of a death record.
Coded Demographics Section: These fields are included in the coded demographic content sent by NCHS to VROs. In total, these are equivalent to the legacy Mortality, Race and Ethnicity (MRE) content.
Coded Cause of Death Section: These fields are included in the coded cause of death content sent by NCHS to VROs. in total, these are equivalent to the Transax (TRX) content.
Coded Industry and Occupation section: These fields are included in the coded industry and occupation content that is produced by the National Institute of Occupational Safety and Health (NIOSH) and delivered to VROs via NCHS.

Copy link
Collaborator

@ptran265 ptran265 left a comment

Choose a reason for hiding this comment

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

You might've done this already, but just checking that changes to the IJE spreadsheet here are also in the master copy

@rpassas
Copy link
Collaborator Author

rpassas commented Jun 19, 2024

You might've done this already, but just checking that changes to the IJE spreadsheet here are also in the master copy

Not yet, once I merge / it is finalized I'll make a master copy PR.

@rpassas rpassas merged commit d8d025e into master Jun 19, 2024
@rpassas rpassas deleted the break-up-DD branch June 19, 2024 19:15
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