Uh oh!
There was an error while loading. Please reload this page.
Branch 1 - #2131
Conversation
Added a newline at the end of the file for proper formatting.
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
Added a robust session setup for HTTP requests with retries and connection pooling. Updated functions to utilize the new session for API calls.
There was a problem hiding this comment.
Code Review
This pull request refactors the Brazil SIDRA IBGE import script to fetch data directly via the IBGE REST API instead of using Selenium, adds validation configurations and golden data for both the Brazil SIDRA and US FARS crash datasets, and updates the US FARS crash data mappings to include 2024 data. Feedback on the changes suggests converting period IDs to strings to prevent comparison type errors, adding explicit guard checks for nested API responses to improve robustness, and raising an exception on critical failures in the main execution block to ensure automated pipelines fail loudly.
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.
70cb2c5 to
c56efe0Compare
No description provided.