ADD: Support for multiple ARM radars and NEXRAD.#17
Merged
rcjackson merged 11 commits intoARM-Development:mainfrom Mar 3, 2026
Merged
ADD: Support for multiple ARM radars and NEXRAD.#17rcjackson merged 11 commits intoARM-Development:mainfrom
rcjackson merged 11 commits intoARM-Development:mainfrom
Conversation
- Reordered parameters from (site, input_site_dict, rad_time, ...) to (rad_time, site, input_site_dict, ...) - Added missing imports: timedelta, Config, UNSIGNED - Fixed all function calls to match new parameter order in: - src/radclss/core/radclss_core.py (both serial and parallel calls) - tests/test_column_utils.py (both unit and integration tests) - examples/bnf_example.py - Added comprehensive unit tests for get_nexrad_column function - Created AGENTS.md with git commit protocol requiring tests and linting Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
- Added itertools import to radclss_core.py - Modified client.map() call to use itertools.repeat() for site and input_site_dict - This ensures only rad_time varies while site and input_site_dict remain constant across parallel executions - Fixes parallel NEXRAD column extraction to correctly handle constant parameters Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
working correctly.
- Add get_nexrad_column function to fetch NEXRAD radar data from AWS S3 - Integrate NEXRAD data into radclss workflow with parallel and serial modes - Add extensive verbose logging throughout radclss processing pipeline - Create unit tests for NEXRAD column extraction with mocking - Add integration tests for NEXRAD, KASACR, and multiple radar systems - Add tests for different time coordinate modes (nexrad, 5Min intervals) - Fix parameter ordering in get_nexrad_column (rad_time first) - Add helper function _add_station_vars for code reuse - Update example to use new parameter order - Add sorted() to time_list generation for consistent NEXRAD fetching - Replace lambda with named function for pre-commit compliance Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
ADD: Support for multiple ARM radars and NEXRAD.