uscensusbureau / recon_replication

Other
2 stars 0 forks source link

requirements.txt is missing #17

Closed simsong closed 1 year ago

simsong commented 1 year ago
rrod515 commented 1 year ago

Python requirements are given in: ec2_dependencies.txt emr_dependencies.txt

Do you intend the makefile to be for use on EMR? If so, @tbeggs-econo can you assist?

simsong commented 1 year ago

the reconstruction code didn't run in EMR; it just used VMs. We may have used the EMR system to spin them up. I recommend putting the python dependencies in a requirements.txt file, which is where they tend to be expected?

tbeggs-econo commented 1 year ago

looks like https://github.com/uscensusbureau/recon_replication/pull/19 got requirements Ill create those make files