Skip to content

Conversation

@nate-kean
Copy link
Contributor

@nate-kean nate-kean commented Aug 5, 2025

Description

This is a small change that will make the codebase more consistent. RAiDER uses several packages for interfacing with datasets in different places, and one is pydap for GMAO and MERRA2 data. This change changes pydap out for xarray, which is already used in many other places.

For GMAO, pydap is no longer used at all. For MERRA2, since it requires authentication, pydap is still used under the hood through xarray's pydap engine.

Motivation and Context

How Has This Been Tested?

All tests still pass. I realized the path for fetching GMAO data before T0 was not tested in test_downloaders, so I added a test for it.

Screenshots (if appropriate):

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

  • I have added an explanation of what your changes do and why you'd like us to include them.
  • I have written new tests for your core changes, as applicable.
  • I have successfully ran tests with your changes locally.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.

nate-kean and others added 10 commits July 31, 2025 14:00
- `utilFcns.write_weather_vars_to_ds`: annotate number of dimensions
- types: Add generic version of the numpy array helper types so you can annotate an array of booleans as `Array1D[np.bool]`
- Add type annotations to `models.hrrr.get_bounds_indices`
@nate-kean nate-kean force-pushed the xarray-for-opendap branch from 4b09d36 to 8de37e7 Compare August 5, 2025 17:27
@nate-kean nate-kean mentioned this pull request Aug 22, 2025
8 tasks
@jlmaurer jlmaurer mentioned this pull request Aug 23, 2025
8 tasks
@nate-kean nate-kean marked this pull request as draft September 9, 2025 17:01
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