Bls_jolts_refresh_import #1153
Bls_jolts_refresh_import #1153
Conversation
scripts/us_bls/jolts/BLSJolts.csv
Outdated
There was a problem hiding this comment.
Is this 4.83 MB file necessary for the import? if yes, please use Git LFS to check-in this file.
There was a problem hiding this comment.
File not needed, it's an output file. Removed output csv and mcf files.
There was a problem hiding this comment.
There is no logging module used in the script
There was a problem hiding this comment.
Please implement the retry module to download all the files.
There was a problem hiding this comment.
Please user retry module to implement this. Please refer to https://pypi.org/project/retry/
There was a problem hiding this comment.
Added, along with modes download or process.
There was a problem hiding this comment.
Please implement mode flag to control the execution flow of the script like download or process.
No description provided.