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 college metric. #450

Open
wants to merge 1 commit into
base: version2025
Choose a base branch
from
Open

Update college metric. #450

wants to merge 1 commit into from

Conversation

jwalsh28
Copy link
Collaborator

Back-fill yeras 2014 and 2016. Apply new AWS d…ata storing approach. Bring in final evaluation tests.

Mobility metric pull request template

Please include the following points in your PR:

  1. A link to the issue that this PR relates to. You can bring up a list of suggested issues and pull requests within the repository by typing.

#403

  1. A description of the content in this pull request.
  • What was changed?
    I back-updated for 2014 and 2016. The format of the program has changed to make it easier to update individual years and run validation and tests collectively at the end. This program also now utlizes AWS for intermediate file storage.
  • What should the reviewer be focusing on?
    The AWS step is new and the biggest conscern is that this file storage would introduce some threat to the data content.
  • Is there a logical order to review the files in?
    Look at the API extract functions first:
  1. extract_ipums_aws.R
    2)extract_repwt_college_aws.R
    Then I would start with the place file college_metrics_place.qmd

  2. Detail on any issues or flags that the metric reviewer/data-team should be aware of.

You will need to have access to the AWS keys for this review to function best. Please slack or email me and I will provide those.

…ata storing approach. Bring in final evaluation tests.
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.

2 participants