-
Notifications
You must be signed in to change notification settings - Fork 16
Support upload of raceeth microdata in Makefile #1211
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
Conversation
Would you add the new keys to |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
Should I wait for Alex on this or go ahead and merge once it finishes building? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I approve provided @krivard has looked at the Make and agrees that the raceeth files will end up in SFTP under protected-race-ethnicity-data/, and won't accidentally end up elsewhere
Also note that whatever script runs |
I haven't seen any scripts or make targets that do the monthly rollup. @krivard is that committed to the repo somewhere? |
Here's the monthly rollup builder: It's called by a local script that we probably should check into the repo. Recommend addressing posting monthly rollups of raceeth data in a separate PR. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All Makefile changes are legit and won't put raceeth data anywhere except on purpose.
Waiting until Monday to merge this just in case we need to do an emergency deploy of |
Description
Put #1183 into production.
Changelog
protected-race-ethnicity-data/
dir. Runs as part ofpipeline
target.individual_raceeth_dir
andproduce_individual_raceeth
default settings to the template production params file.produce_individual_raceeth
in the params make target.