Uh oh!
There was an error while loading. Please reload this page.
Mongolia_Education import Fix - #2121
Conversation
Code fix unenergy
There was a problem hiding this comment.
Code Review
This pull request introduces dynamic table path resolution and metadata fetching in common_download_script.py to retrieve and pivot Mongolian education data. It also updates several PV map CSV files with corrected mappings and extended observation dates up to 2030, and adds place resolution to the kindergarten script in run.sh. The code review feedback highlights three key improvements: caching API responses in find_table_path to avoid redundant catalog crawls, adding timeouts to network requests to prevent hanging, and applying defensive programming checks on API responses to avoid potential runtime crashes.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Summary
This PR fixes the download script and pvmappings for the import Mongolia_Education
Key Changes
PR Checklist - link