[niveditasing]

Summary

Fixes two failures in the WorldBank_IDS (statvar_imports/world_bank/worldbank_ids:WorldBank_IDS) auto-refresh import:

1. Download failure (HTTP 524 origin timeout / AttributeError: 'NoneType' object has no attribute 'rename'):

   • Fetching all 63 years (1970–current_year + 6) for an indicator in a single World Bank API call intermittently triggers Cloudflare HTTP 524 timeouts, causing bblocks (get_indicator_data) to return None and crash in DebtIDS._get_indicator (data.rename(...)).
   • Updated download.py to fetch each indicator individually with bounded retries and exponential backoff (MAX_RETRIES = 3), and fall back to 20-year chunks (YEAR_CHUNK_SIZE = 20) if full-range requests still fail.
   • Removed the unused WorldBankData import from download.py.

2. Validation failure (check_lint_error_count / Sanity_InvalidChars_dcid on input0):

   • Upstream World Bank IDS data contains two counterpart_area values (Africa Finance Corporation and Uganda) that were missing from InputlendingEntity.csv.
   • In currency (input0), configs/currency_pvmap.csv fell back to lendingEntity: {Data}, generating a provisional schema node Node: dcid:Africa Finance Corporation (containing spaces) in currency_stat_vars_schema.mcf and failing genmcf lint validation (Sanity_InvalidChars_dcid).
   • In interest, principal, and disbursed (input1–input3), rows for these two unmapped lenders lacked a lendingEntity property and collided with World StatVars (error-statvar-with-dup-svobs).
   • Added Africa Finance Corporation -> Africa_Finance_Corporation and Uganda -> country/UGA to InputlendingEntity.csv.

Files Changed

• statvar_imports/world_bank/worldbank_ids/download.py
• statvar_imports/world_bank/worldbank_ids/InputlendingEntity.csv

[gemini-code-assist]

Code Review

This pull request adds new entries to the InputlendingEntity.csv mapping and refactors the World Bank data download script to implement a robust retry mechanism with exponential backoff and a fallback to year-based chunking. The feedback recommends narrowing down the exception handling in both retry loops from a broad Exception to specific transient errors, such as requests.exceptions.RequestException, to prevent masking programming or configuration bugs.