Skip to content

Add new Commerce EDA USASpending import - #2171

Open
shvngisingh wants to merge 1 commit into
datacommonsorg:masterfrom
shvngisingh:commerce_eda_usaspending_import
Open

Add new Commerce EDA USASpending import#2171
shvngisingh wants to merge 1 commit into
datacommonsorg:masterfrom
shvngisingh:commerce_eda_usaspending_import

Conversation

@shvngisingh

@shvngisingh shvngisingh commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

This PR adds a new import pipeline commerce_eda_usaspending to import EDA (Economic Development Administration) financial investment data from USAspending.

Data Source:

Economic Development Administration (EDA) awards/grants from the USAspending API.

Coverage:

Financial investments aggregated by State/Territory, Year, and EDA Program category (Public Works, Technical Assistance, Regional Innovation Strategies, Planning, Economic Adjustment Assistance, and Total).

PR_check_list: link

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code Review

This pull request introduces a new automated importer for U.S. Economic Development Administration (EDA) investment datasets sourced from USASpending.gov, including preprocessing scripts, configuration files, and documentation. Feedback on the changes suggests improving the robustness of the preprocessing script by raising exceptions on API failures, adding guard checks for date parsing, and removing redundant code that attempts to update a commented-out metadata field. Additionally, it is recommended to explicitly map the 'Total' category in the PV map to properly represent aggregation.

Comment thread statvar_imports/commerce_eda_usaspending/process.py
Comment thread statvar_imports/commerce_eda_usaspending/process.py Outdated
Comment thread statvar_imports/commerce_eda_usaspending/process.py
Comment thread statvar_imports/commerce_eda_usaspending/Investmentpvmap.csv
@shvngisingh
shvngisingh force-pushed the commerce_eda_usaspending_import branch 5 times, most recently from 63a311b to b78097b Compare August 21, 2026 13:19
@shvngisingh
shvngisingh force-pushed the commerce_eda_usaspending_import branch from b78097b to 68b26e6 Compare August 21, 2026 13:24
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.

1 participant