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

Issues with Zonal, Meridional statistics preprocessors #443

Open
5 tasks
ledm opened this issue Jan 22, 2020 · 0 comments
Open
5 tasks

Issues with Zonal, Meridional statistics preprocessors #443

ledm opened this issue Jan 22, 2020 · 0 comments
Assignees
Labels
preprocessor Related to the preprocessor

Comments

@ledm
Copy link
Contributor

ledm commented Jan 22, 2020

This issue continues the discussion from #433 (now merged).

zonal_statistics has the following issues:

  • It won't work for irregular grids, but that's not straightforward to fix - it currently raises an error.
  • We may encounter grids that are regular in the latitude/longitude directions, but that vary with depth. It should fail if that is the case as it is not possible to perform this calculation at the moment.
  • Many of the irregular 2D grids (like NEMO's ORCA1) are fairly regular for a lot of the latitude range. A possible improvement would be to add a y-direction option so that we could take the mean in the y direction instead of the latitudinal direction for irregular grids. This would work fine if you're only interested in a small zonal range (like the AMOC RAPID array at 26N, for instance!)

meridional_statistics has the issues:

  • Similarly to zonal_statistics, it won't work for irregular grids - it currently raises an error.
  • The mean 'operator' needs to weight according to fx data.

Also, these preprocessors are in the _area.py file. Is that the best place for them?

@mattiarighi mattiarighi added the preprocessor Related to the preprocessor label Jan 22, 2020
@zklaus zklaus removed their assignment Feb 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
preprocessor Related to the preprocessor
Projects
None yet
Development

No branches or pull requests

3 participants