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

Subset global field with arbitrary (rectangular) lat / lon corners #472

Open
jpolton opened this issue Jun 23, 2022 · 2 comments
Open

Subset global field with arbitrary (rectangular) lat / lon corners #472

jpolton opened this issue Jun 23, 2022 · 2 comments
Assignees

Comments

@jpolton
Copy link
Collaborator

jpolton commented Jun 23, 2022

Story:
Lazy load in a global sea surface temperature data set with the fancy COPERNICUS read method.
Cut out domain of interest with input bounding coords.
Tricky bit: deal with wrapped coordinates

lat = [50,60], lon=[-10,5] # UK
lat = [50,60], lon = [350,5] # UK
lat = [50,60], lon = [5,350] # zonal band excluding the UK

Deal with dateline:
lat = [-10,10], lon = [170, -170] # somewhere in the Pacific

Steve's work address much of the wrapped coordinates issue, but the implementation was incomplete

AC:

  • subset + plot on battery of example subdomains for COPERNICUS
  • subset zos across longitude join for SENEMO_1ts_20210101_20211231_grid_T.nc in /projectsa/COAsT/SENEMO at latitude=0
  • subset zos across longitude join for SENEMO_1ts_20210101_20211231_grid_T.nc in /projectsa/COAsT/SENEMO at latitude=85
  • unit_testing updated and works

Crazy global grid

@jpolton
Copy link
Collaborator Author

jpolton commented Jun 24, 2022

Ticket 327 is probably a good place to start. It briefly describes what Steve tried to do and the code he sought to replace.
It has an associated Pull request. Which I didn't like (it didn't work) and so created the following ticket.

Ticket 425 discussed a issue found in Steve's implementation that manifest in example_files/subsetting_by_lon_lat.py but was I think a result of his method coast.general_utils.subset_indices_lonlat_box() being to limited in its scope relative to the original code. (It appears that Steve enforces that the subset must be a square rather than a rectangle. Maybe there are other issues too)

The pull request is also missing Unit_testing so that is also missing from Steve work.

@lukegorman
Copy link
Contributor

issue part of #425

@MCazaly MCazaly moved this from BODC - Todo to BODC - In Progress in COAsT_dev Sep 1, 2022
@MCazaly MCazaly self-assigned this Sep 1, 2022
@roje-bodc roje-bodc moved this from BODC - In Progress to BODC - Todo in COAsT_dev Sep 8, 2022
@MCazaly MCazaly moved this from BODC - Todo to BODC - In Progress in COAsT_dev Sep 13, 2022
@thogar-computer thogar-computer moved this from BODC - In Progress to BODC - Todo in COAsT_dev Oct 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: BODC - Todo
Development

No branches or pull requests

3 participants