Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve geo_tools.find_closest_model_point() #95

Merged
merged 5 commits into from
Feb 15, 2024

Conversation

douglatornell
Copy link
Member

Add tolerance values for HRDPS continental 2.5 km resolution rotated lat/lon grid that became forcing for SalishSeaCast in Feb-2023.

Also clean up code and fix failing unit tests.

Update unit tests that weren't adjusted when 'raiseOutOfBounds' parameter was
added to find_closest_model_point(). This ensures that a ValueError is raised
when raiseOutOfBounds is enabled and latitude and longitude values aren't on the
grid.
The unnecessary import statements for netCDF4 and tidetools modules were removed
from the unit tests of the geo_tools module in the SalishSeaTools library. This
makes the unit tests leaner and reduces dependencies for running the tests.
Add tolerance values for HRDPS continental 2.5 km resolution rotated lat/lon
grid that became forcing for SalishSeaCast in Feb-2023.
@douglatornell douglatornell added bug Something isn't working enhancement New feature or request SalishSeaTools labels Feb 15, 2024
@douglatornell douglatornell self-assigned this Feb 15, 2024
@douglatornell douglatornell added the documentation Improvements or additions to documentation label Feb 15, 2024
@douglatornell douglatornell merged commit 5eeb3fa into main Feb 15, 2024
2 checks passed
@douglatornell douglatornell deleted the improve-find_closest_model_point branch February 15, 2024 22:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working documentation Improvements or additions to documentation enhancement New feature or request SalishSeaTools
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant