DRAFT: Reductions

[aaronzedwick]

Closes #714, #715, #716, #717

Adds 4 new reduction methods to topological_aggregations

Overview

Expected Usage

import uxarray as ux

grid_path = "/path/to/grid.nc"
data_path = "/path/to/data.nc"

uxds = ux.open_dataset(grid_path, data_path)

# this is how you use this function
some_output = uxds.some_function()

# this is another way to use this function
other_output = uxds.some_function(some_param = True)

PR Checklist

General

• An issue is linked created and linked
• Add appropriate labels
• Filled out Overview and Expected Usage (if applicable) sections

Testing

• Adequate tests are created if there is new functionality
• Tests cover all possible logical paths in your function
• Tests are not too basic (such as simply calling a function and nothing else)

Documentation

• Docstrings have been added to all new functions
• Docstrings have updated with any function changes
• Internal functions have a preceding underscore (_) and have been added to docs/internal_api/index.rst
• User functions have been added to docs/user_api/index.rst

Examples

• Any new notebook examples added to docs/examples/ folder
• Clear the output of all cells before committing
• New notebook files added to docs/examples.rst toctree
• New notebook files added to new entry in docs/gallery.yml with appropriate thumbnail photo in docs/_static/thumbnails/

[review-notebook-app]

Check out this pull request on 

See visual diffs & provide feedback on Jupyter Notebooks.

---

Powered by ReviewNB

[philipc2]

@aaronzedwick

What's the state of this PR? It would be nice to include this in our Friday release.

[aaronzedwick]

@aaronzedwick

What's the state of this PR? It would be nice to include this in our Friday release.

It needs a lot of work; I can start working on it again this week, now that get_face_containing_point is finished, but I don't know if I can get it done by Friday. Depending on how difficult it ends up being, our connectivity for edges is incorrectly constructed, so that is the main fix.

[rajeeja]

@aaronzedwick can you try to resolve conflicts here?

[philipc2]

Closing this. Planning to tackle these reductions and other aggregations in a future PR with a cleaner approach, either through a .agg() method or aggregate accessor