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

Calculate the weight for each face on the constant latitude #544

Closed
hongyuchen1030 opened this issue Oct 24, 2023 · 2 comments · Fixed by #555
Closed

Calculate the weight for each face on the constant latitude #544

hongyuchen1030 opened this issue Oct 24, 2023 · 2 comments · Fixed by #555
Assignees
Labels
new feature New feature or request

Comments

@hongyuchen1030
Copy link
Contributor

hongyuchen1030 commented Oct 24, 2023

Describe the feature you'd like added to this project
The function calculates the weights of grid faces at a specific constant latitude, facilitating accurate non-conservative zonal averages in climate and oceanographic data analysis. This addition aims to optimize and simplify user workflows within our library.

Link to original functionality implementation elsewhere, if applicable

Links to other examples, if applicable

Describe the solution you'd like

  • Take a constant latitude as input and compute the perimeter of the corresponding circle.
  • Identify the intervals of longitude that overlap with each face of the grid.
  • Calculate the weight of each face based on these intervals and the total perimeter.
  • Handle any errors or edge cases gracefully, ensuring that the function is robust and user-friendly.

Additional context
Add any other context or screenshots about the feature request here.

@hongyuchen1030 hongyuchen1030 added the new feature New feature or request label Oct 24, 2023
@hongyuchen1030 hongyuchen1030 added this to the Global and zonal means milestone Oct 24, 2023
@hongyuchen1030 hongyuchen1030 self-assigned this Oct 24, 2023
@erogluorhan
Copy link
Member

@hongyuchen1030 can you please provide an actual description of this issue above? The original text doesn't seem to be adding anything to the issue template.

@hongyuchen1030
Copy link
Contributor Author

@hongyuchen1030 can you please provide an actual description of this issue above? The original text doesn't seem to be adding anything to the issue template.

Yes, I will provide a more detailed description here and relate it to my draft PR.

@hongyuchen1030 hongyuchen1030 changed the title Sweep line algorithm in identifying the intersection for each face Calculate the weight for each face on the constant latitude Oct 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new feature New feature or request
Projects
Status: ✅ Done
Development

Successfully merging a pull request may close this issue.

2 participants